----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36897 -----------------------------------------------------------
src/Makefile.am <https://reviews.apache.org/r/18946/#comment68091> Ben Mahler just reminded me that this change is going to necessitate a deploy order dependency. In particular, if a JVM scheduler is using an old jar and new libmesos it won't have access to the JNI code correct? We should figure out a way to not impose this constraint if possible. How about not stripping JNI code from libmesos in this release cycle but strip it in a later release after deprecation. - Vinod Kone On March 12, 2014, 12:56 a.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18946/ > ----------------------------------------------------------- > > (Updated March 12, 2014, 12:56 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-855 > https://issues.apache.org/jira/browse/MESOS-855 > > > Repository: mesos-git > > > Description > ------- > > Introduced a new environment variable, MESOS_NATIVE_JAVA_LIBRARY. That > variable points towards libmesos_java. libmesos_java contains the JNI- > specific code (formally part of libmesos) and dynamically links against > libmesos. > > A typical java-based framework relies on mesos.jar to do the loading > but may use some extra logic in its startup to make sure > MESOS_NATIVE[_JAVA]_LIBRARY is set/valid. That extra-logic would need > to be adapted to use the new environment variable instead of the old > one. > > Note: > This patch does not remove the MESOS_NATIVE_LIRBRAY variable. It is > kept for frameworks that are not JVM based but need to bind to the > self-contained Mesos library. MESOS_NATIVE_LIRBRAY points towards > libmesos. > MESOS_NATIVE_JAVA_LIBRARY however is the new variant to be used for > JVM based frameworks. > > > Diffs > ----- > > bin/mesos-slave-flags.sh.in dc73aef > src/Makefile.am 384b312 > src/java/generated/org/apache/mesos/MesosNativeLibrary.java.in 231d1e2 > src/slave/containerizer/containerizer.cpp d0a1023 > src/tests/environment.cpp 5c92fcd > > Diff: https://reviews.apache.org/r/18946/diff/ > > > Testing > ------- > > make check and functional testing with external, java based frameworks > > > Thanks, > > Till Toenshoff > >
