> On Oct. 1, 2020, 10:19 p.m., Benjamin Mahler wrote: > > src/java/src/org/apache/mesos/MesosSchedulerDriver.java > > Lines 416-418 (patched) > > <https://reviews.apache.org/r/72922/diff/1/?file=2240239#file2240239line416> > > > > Use javadoc style here? maybe make this clear that it's a note for us > > and not the user looking at the interface?
Changed style. Starts with `/*` and not with `/**`, so that javadoc does not generate documentation, regardless of whatever flags governing doc generation for private methods. Also seems to trigger different highlighting in some editors. Given that the method is private, it should be more or less clear that this is not for user. Now I've moved the public 3-argument `updateFramework()` higher up, next to the deprecated 2-parameter one, hopefully this will help avoid confusing users. - Andrei ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72922/#review221984 ----------------------------------------------------------- On Oct. 2, 2020, 2:09 p.m., Andrei Sekretenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72922/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2020, 2:09 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-10189 > https://issues.apache.org/jira/browse/MESOS-10189 > > > Repository: mesos > > > Description > ------- > > This patch converts the implementation of the obsolete 2-parameter > `SchedulerDriver.updateFramework(frameworkInfo, suppressedRoles)` > from a wrapper around the new signature back into a JNI method > that 930c7e98d17e71192dae1d49b4b2217cc2dbd8b2 attempted to remove. > > This is needed to keep compatibility between older versions of > `mesos.jar` and newer versions of `libmesos-java.so`. > > > Diffs > ----- > > src/java/jni/org_apache_mesos_MesosSchedulerDriver.cpp > 4efde3083bb62253f3ad512b40a291fd7f7f7a78 > src/java/src/org/apache/mesos/MesosSchedulerDriver.java > 4fdae338491709574775c52438e41d03b39917bd > > > Diff: https://reviews.apache.org/r/72922/diff/2/ > > > Testing > ------- > > `make check` > > > Thanks, > > Andrei Sekretenko > >
