I fixed the latest marathon master and submitted a pull request for
chronos. If you want to use chronos with 0.14-rc3, please use the branch
referenced in the pull request: https://github.com/airbnb/chronos/pull/115.

Let me know if it worked!

Thanks,

--Flo


On Tue, Sep 10, 2013 at 5:40 PM, Benjamin Hindman <[email protected]> wrote:

>
>
> > 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
> >
> >
>
>

Reply via email to