> On Sept. 10, 2013, 10:41 p.m., Al Tobey wrote: > > src/java/jni/org_apache_mesos_state_ZooKeeperState.cpp, line 58 > > <https://reviews.apache.org/r/12441/diff/1/?file=319802#file319802line58> > > > > I don't understand the error completely but it looks like this breaks > > cross-version compatibility. Specifically, I'm trying to run Marathon > > against a Mesos 0.14.0-rc3 package I built locally. Marathon is pulling > > down a Mesos jar from Mesosphere's Maven that doesn't seem to be compatible > > with an actual 0.14.0 release. > > > > https://gist.github.com/tobert/6516404 > >
Yes, the libmesos.so from 0.14.0-rc3 now expects to look for certain things in a super class which isn't present in the JAR that is shipping with Marathon (and/or Chronos). This should have been caught in this review ... apologies. Unfortunately, the JNI doesn't look in super classes automagically if it isn't found in the specified class. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12441/#review26026 ----------------------------------------------------------- On July 10, 2013, 7:39 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12441/ > ----------------------------------------------------------- > > (Updated July 10, 2013, 7:39 a.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/Makefile.am 29337df8f844068bcc1b20a66cd8ade71d2f6a45 > src/java/jni/org_apache_mesos_state_AbstractState.cpp PRE-CREATION > src/java/jni/org_apache_mesos_state_ZooKeeperState.cpp > 84174d564be2700ffdc750cdc1e966df3cc6cec9 > src/java/src/org/apache/mesos/state/AbstractState.java PRE-CREATION > src/java/src/org/apache/mesos/state/ZooKeeperState.java > 999d593e26c435132c1ed1e02dd2f432a20bd132 > > Diff: https://reviews.apache.org/r/12441/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
