On Mar 7, 2012, at 9:37 AM, Sage Weil wrote: > Hey Noah- > > This looks great! I'd love to merge this for v0.44, but need your > Signed-off-by: on the patches. Also, if you want to switch to com.ceph > while you're at it, that'd be good too. > > I have a trivial makefile fix in wip-java you might want to look at. Sounds great. I'll fold in your patches, fix-up the com.ceph path, and add the sign-off. When are you shipping v0.44? I'd like to include some more unit tests... > Has any of the hadoop side work been done to use this yet? Is there > anything we can do help get that in a state where we can send it upstream? I've kept an eye on how much things are deviating from the current implementation and its pretty minimal, so I think producing a new version to use the generic libcephfs interface will happen quickly. As for producing an upstream patch, I think the two biggest concerns are: 1. Reporting hostname/racks rather than IPs for object location. 2. Address (or at least document) the time synchronization issue. These are largely orthogonal to getting the code ready for upstream, but may be a "political" road block? I can put together the details of these two issues in a separate thread… Thanks, Noah > > Thanks! > sage > > > On Fri, 2 Mar 2012, Noah Watkins wrote: > >> Generic libcephfs Java wrappers. >> >> This is the first of a patch series that can eventually replace the current Java wrappers that are Hadoop specific. This group implements enough of the libcephfs API to support Hadoop functionality, includes Debian packaging, autoconf magics that look for default Java installations, and some basic JUnit tests. >> >> The patches are available in the git repository at: >> >> git://github.com/noahdesu/ceph.git wip/java-cephfs >> >> Noah Watkins (3): >> java: add Java and C++ source files >> java: setup autotools to build cephfs-java >> debian: add libcephfs-java package >> >> configure.ac | 74 ++ >> debian/.gitignore | 1 + >> debian/control | 8 +- >> debian/libceph1-java.install | 2 + >> debian/rules | 1 + >> src/Makefile.am | 15 +- >> src/java/.gitignore | 4 + >> src/java/Makefile.am | 49 + >> src/java/README | 43 + >> src/java/build.xml | 67 ++ >> .../java/net/newdream/ceph/fs/CephConstants.java | 38 + >> .../java/net/newdream/ceph/fs/CephDirectory.java | 84 ++ >> .../java/net/newdream/ceph/fs/CephException.java | 21 + >> .../ceph/fs/CephInvalidStateException.java | 11 + >> src/java/java/net/newdream/ceph/fs/CephMount.java | 360 +++++++ >> .../net/newdream/ceph/fs/CephNativeLoader.java | 16 + >> src/java/java/net/newdream/ceph/fs/CephProxy.java | 217 ++++ >> src/java/java/net/newdream/ceph/fs/CephStat.java | 15 + >> .../java/net/newdream/ceph/fs/CephStatVFS.java | 14 + >> src/java/java/net/newdream/ceph/fs/CephStruct.java | 33 + >> src/java/native/libcephfs_jni.cc | 1037 ++++++++++++++++++++ >> src/java/test/CephMountCreateTest.java | 76 ++ >> src/java/test/CephMountTest.java | 72 ++ >> 23 files changed, 2256 insertions(+), 2 deletions(-) >> create mode 100644 debian/libceph1-java.install >> create mode 100644 src/java/.gitignore >> create mode 100644 src/java/Makefile.am >> create mode 100644 src/java/README >> create mode 100644 src/java/build.xml >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephConstants.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephDirectory.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephException.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephInvalidStateException.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephMount.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephNativeLoader.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephProxy.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephStat.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephStatVFS.java >> create mode 100644 src/java/java/net/newdream/ceph/fs/CephStruct.java >> create mode 100644 src/java/native/libcephfs_jni.cc >> create mode 100644 src/java/test/CephMountCreateTest.java >> create mode 100644 src/java/test/CephMountTest.java >> >> -- >> 1.7.5.4 >> -- >> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
