Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95548 --- Ship it! Definitely a band-aid but given that we keep tracking this

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-14 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95460 --- Ship it! Per Greg, the underlying issue will be tracked here: http

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95405 --- Patch looks great! Reviews applied: [37415] All tests passed. - M

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/ --- (Updated Aug. 14, 2015, 4:56 a.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Marco Massenzio
> On Aug. 13, 2015, 4:22 p.m., Till Toenshoff wrote: > > src/examples/java/TestFramework.java, line 275 > > > > > > Why 500 - not 200 or 1000? If I am not mistaken, then using 500 worked > > fine on your test-machin

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Greg Mann
> On Aug. 13, 2015, 4:22 p.m., Till Toenshoff wrote: > > src/examples/java/TestFramework.java, line 275 > > > > > > Why 500 - not 200 or 1000? If I am not mistaken, then using 500 worked > > fine on your test-machin

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95299 --- Patch looks great! Reviews applied: [37414, 37415] All tests passe

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Joseph Wu
> On Aug. 13, 2015, 9:22 a.m., Till Toenshoff wrote: > > src/examples/java/TestFramework.java, line 275 > > > > > > Why 500 - not 200 or 1000? If I am not mistaken, then using 500 worked > > fine on your test-machin

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Greg Mann
> On Aug. 13, 2015, 4:22 p.m., Till Toenshoff wrote: > > src/examples/java/TestFramework.java, line 275 > > > > > > Why 500 - not 200 or 1000? If I am not mistaken, then using 500 worked > > fine on your test-machin

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/ --- (Updated Aug. 13, 2015, 4:49 p.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95264 --- src/examples/java/TestFramework.java (line 275)

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/#review95203 --- Patch looks great! Reviews applied: [37415] All tests passed. - M

Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-12 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/ --- Review request for mesos, Joris Van Remoortere and Till Toenshoff. Bugs: MESOS-