Hi Sage, I wanted to touch base on this Java bindings patch series to make sure this can become a solid foundation for the Hadoop shim clean-up. Were there any specific issues with this, other than not yet having a major consumer?
-Noah On Thu, Sep 6, 2012 at 11:02 AM, Noah Watkins <[email protected]> wrote: > On Thu, Sep 6, 2012 at 8:42 AM, Sage Weil <[email protected]> wrote: >> >> Also, I noticed the automake patch removes CephException.java, added in >> the previous patch... probably an accident? > > I left CephException out intentionally because it could be replaced by > generic exceptions in Java like IOException or FileNotFoundException. > My belief is that CephException (and specializations of this) will > need to be re-introduced as more error cases are covered. For example, > I think we'll want a CephBadFileDescriptor, but currently libcephfs > doesn't appear to return -EBADF (the client is crashing do to fd_map > assert). So, the Java wrappers should be as stable as the C API, but > both seem to need some loose ends tied up. > >> Can't wait to pull this in! BTW, Noah, do you know if this is still an >> issue? >> >> http://tracker.newdream.net/issues/2778 > > It does appear that -ENOENT is now returned. I added a unit test that > expects FileNotFoundException and that works out nicely. > > - Noah -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
