Re: [GIT PULL v5] java: add libcephfs Java bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 5 Sep 2012, Noah Watkins wrote:
> Latest version of Java bindings. There is ~full coverage of the
> libcephfs API, and nearly a complete set of unit tests covering
> success, and common failure cases. In addition to the basic review, I
> think the Debian integration needs review.
> 
> Patches are here:
> 
>   git://github.com/noahdesu/ceph.git wip-java-cephfs

The debian bits look okay to my untrained eye.  Laszlo, do you mind taking 
a quick look?

Also, I noticed the automake patch removes CephException.java, added in 
the previous patch... probably an accident?

Can't wait to pull this in!  BTW, Noah, do you know if this is still an 
issue?

	http://tracker.newdream.net/issues/2778

Thanks!
sage

> 
> ----------------------------------------------------------------
> Noah Watkins (4):
>       libcephfs: return error when not mounted
>       java: add Java and C++ source files
>       automake: setup autotools to build cephfs-java
>       debian: add libcephfs-java package
> 
> Sage Weil (2):
>       do_autogen.sh: add -j flag to enable java stuff
>       Makefile: add src/include to #include path for libcephfs_jni_la
> 
>  configure.ac                                       |   74 +
>  debian/.gitignore                                  |    1 +
>  debian/control                                     |    7 +-
>  debian/libceph1-java.install                       |    2 +
>  debian/rules                                       |    1 +
>  do_autogen.sh                                      |    5 +-
>  src/Makefile.am                                    |   15 +-
>  src/common/config_opts.h                           |    1 +
>  src/include/cephfs/libcephfs.h                     |    7 +
>  src/java/.gitignore                                |    4 +
>  src/java/Makefile.am                               |   43 +
>  src/java/README                                    |   48 +
>  src/java/build.xml                                 |   67 +
>  src/java/java/com/ceph/fs/CephMount.java           |  645 ++++++
>  src/java/java/com/ceph/fs/CephNativeLoader.java    |   35 +
>  .../java/com/ceph/fs/CephNotMountedException.java  |   42 +
>  src/java/java/com/ceph/fs/CephStat.java            |   45 +
>  src/java/java/com/ceph/fs/CephStatVFS.java         |   33 +
>  src/java/native/libcephfs_jni.cc                   | 2383 ++++++++++++++++++++
>  src/java/test/CephMountCreateTest.java             |   88 +
>  src/java/test/CephMountTest.java                   |  797 +++++++
>  src/java/test/CephUnmountedTest.java               |  143 ++
>  src/libcephfs.cc                                   |  134 +-
>  23 files changed, 4614 insertions(+), 6 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/com/ceph/fs/CephMount.java
>  create mode 100644 src/java/java/com/ceph/fs/CephNativeLoader.java
>  create mode 100644 src/java/java/com/ceph/fs/CephNotMountedException.java
>  create mode 100644 src/java/java/com/ceph/fs/CephStat.java
>  create mode 100644 src/java/java/com/ceph/fs/CephStatVFS.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
>  create mode 100644 src/java/test/CephUnmountedTest.java
> --
> 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


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux