Re: Review Request 43999: Use relative path to create libraries symbolic link.

2016-02-26 Thread Jacob Janco
rpm builds look good, thanks! - Jacob Janco On Feb. 26, 2016, 8:18 a.m., Zhiwei Chen wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request 46416: remove hardcoding for MethodNotAllowed for http

2016-04-20 Thread Jacob Janco
://reviews.apache.org/r/46416/diff/ Testing --- Thanks, Jacob Janco

Review Request 46415: construct http reponse body in libprocess

2016-04-20 Thread Jacob Janco
/ Testing --- Thanks, Jacob Janco

Review Request 46505: Remove MethodNotAllowed error string creation

2016-04-21 Thread Jacob Janco
://reviews.apache.org/r/46505/diff/ Testing --- Thanks, Jacob Janco

Review Request 46504: Construct error string in MethodNotAllowed

2016-04-21 Thread Jacob Janco
--- Thanks, Jacob Janco

Re: Review Request 46505: Removed MethodNotAllowed error string creation.

2016-04-22 Thread Jacob Janco
a9cb99a92ff5a783e719df5e5cfb6e8301241df9 src/slave/http.cpp 3908e33ed5b233387790276f6f5d884452087d2c Diff: https://reviews.apache.org/r/46505/diff/ Testing --- Thanks, Jacob Janco

Re: Review Request 46505: Removed MethodNotAllowed error string creation.

2016-05-10 Thread Jacob Janco
9b558862e025c5caa71e05fc5eeba783c0ad6fd5 Diff: https://reviews.apache.org/r/46505/diff/ Testing (updated) --- make check visual check by sending a (`PUT`) request to an endpoint that does not support it (`/quota`) Thanks, Jacob Janco

Re: Review Request 46505: Removed MethodNotAllowed error string creation.

2016-05-02 Thread Jacob Janco
/slave/http.cpp 9b558862e025c5caa71e05fc5eeba783c0ad6fd5 Diff: https://reviews.apache.org/r/46505/diff/ Testing (updated) --- make check Thanks, Jacob Janco

Re: Review Request 46504: Constructed error string in MethodNotAllowed.

2016-05-02 Thread Jacob Janco
/ Testing (updated) --- make check Thanks, Jacob Janco

Re: Review Request 46504: Constructed error string in MethodNotAllowed.

2016-05-09 Thread Jacob Janco
) --- Constructed error string in MethodNotAllowed. Diffs (updated) - 3rdparty/libprocess/include/process/http.hpp 8f4eabcbb71ead1f5c28e1d8a2dd40db7af1f297 Diff: https://reviews.apache.org/r/46504/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-25 Thread Jacob Janco
=1 GTEST_FILTER="*BENCHMARK_Test.SuppressOffers*" make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-25 Thread Jacob Janco
; > ++suppressCount; > > } > > > > ... > > } > > ``` I agree this is easier to understand and simpler if we loop by the expected counts. - Jacob --- T

Re: Review Request 49616: Add suppression benchmark.

2016-07-26 Thread Jacob Janco
https://reviews.apache.org/r/49616/diff/ > > > Testing > --- > > MESOS_BENCHMARK=1 GTEST_FILTER="*BENCHMARK_Test.SuppressOffers*" make check > > > Thanks, > > Jacob Janco > >

Re: Review Request 49616: Add suppression benchmark.

2016-07-26 Thread Jacob Janco
/tests/hierarchical_allocator_tests.cpp bb6947fcecb5b78047e98d10fc1278c612a69548 Diff: https://reviews.apache.org/r/49616/diff/ Testing --- MESOS_BENCHMARK=1 GTEST_FILTER="*BENCHMARK_Test.SuppressOffers*" make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-28 Thread Jacob Janco
/#review143901 --- On July 28, 2016, 8:42 p.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apac

Re: Review Request 49616: Add suppression benchmark.

2016-07-28 Thread Jacob Janco
ut of 6000 frameworks suppressing offers allocate() took 17.318823secs to make 0 offers with 6000 out of 6000 frameworks suppressing offers ... Thanks, Jacob Janco

Review Request 50268: Target shutdownFramework to associated agents.

2016-07-20 Thread Jacob Janco
shutdownFramework to associated agents. Diffs - src/master/master.cpp 78a8889313179b4509a6657e6c61d9f6d3bb99c0 src/slave/slave.cpp da643e6e50b2f313705d2f862c961291aa5d2f22 Diff: https://reviews.apache.org/r/50268/diff/ Testing --- Thanks, Jacob Janco

Re: Review Request 50268: Target shutdownFramework to associated agents.

2016-07-20 Thread Jacob Janco
frameworks on 1 agent with 2 agents in cluster pre: ShutDownFrameworkMessage sent to both agents. post: ShutdownFrameworkMessage sent to agent with executors associated with framework. Thanks, Jacob Janco

Re: Review Request 50268: Target shutdownFramework to associated agents.

2016-07-20 Thread Jacob Janco
: https://reviews.apache.org/r/50268/diff/ Testing --- 5 frameworks on 1 agent with 2 agents in cluster pre: ShutDownFrameworkMessage sent to both agents. post: ShutdownFrameworkMessage sent to agent with executors associated with framework. Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-04 Thread Jacob Janco
in batches. Diffs - src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49617/diff/ Testing --- make check Thanks, Jacob Janco

Review Request 49616: Add suppression benchmark.

2016-07-04 Thread Jacob Janco
, Jacob Janco

Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-04 Thread Jacob Janco
Diff: https://reviews.apache.org/r/49617/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-05 Thread Jacob Janco
0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49616/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-08 Thread Jacob Janco
/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49616/diff/ Testing (updated) --- MESOS_BENCHMARK=1 GTEST_FILTER="*BENCHMARK_Test.SuppressFrameworks*" make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-08 Thread Jacob Janco
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49616/#review141095 --- On July 8, 2016, 5:55 p.m., Jacob Janco wrote: > > --

Re: Review Request 49616: Add suppression benchmark.

2016-07-08 Thread Jacob Janco
nerated e-mail. To reply, visit: https://reviews.apache.org/r/49616/#review141251 ------- On July 8, 2016, 5:55 p.m., Jacob Janco wrote: > > --- > This i

Re: Review Request 49616: Add suppression benchmark.

2016-07-08 Thread Jacob Janco
test. Agreed, I'll add this. - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49616/#review141253 ---

Re: Review Request 49616: Add suppression benchmark.

2016-07-08 Thread Jacob Janco
/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49616/diff/ Testing (updated) --- MESOS_BENCHMARK=1 GTEST_FILTER="*BENCHMARK_Test.SuppressOffers*" make check Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
) { > > allocator->recoverResources( > > offer.frameworkId, > > offer.slaveId, > > offer.resources, > > None()); > > } > > > > offers.clear(); > > } > >

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
frameworks. - In this scenario, frameworks are offered resources and suppressed in batches. Diffs (updated) - src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49617/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
frameworks. - In this scenario, frameworks are offered resources and suppressed in batches. Diffs (updated) - src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49617/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
are offered resources and suppressed in batches. Diffs (updated) - src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49617/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-07 Thread Jacob Janco
/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49616/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-07 Thread Jacob Janco
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49616/#review141095 ------- On July 8, 2016, 3:48 a.m., Jacob Janco wrote: > >

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
ition for (unsigned i = batchStart; i < batchEnd && i < frameworkCount; i++) { to account for all frameworkCount cases. Also, I recover offers there because only framework 1 of the batch will get an offer, so we will get 1005 offers for the whole of a test run with 1000 agents the 5 being

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
, Jacob Janco

Review Request 49784: Increase framework numbers to allocator benchmarks.

2016-07-07 Thread Jacob Janco
- src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49784/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-07-07 Thread Jacob Janco
) { > > allocator->recoverResources( > > offer.frameworkId, > > offer.slaveId, > > offer.resources, > > None()); > > } > > > > offers.clear(); > > } > >

Re: Review Request 49616: Add suppression benchmark.

2016-07-05 Thread Jacob Janco
an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49616/#review140714 ------- On July 5, 2016, 6:17 p.m., Jacob Janco wrote: > > --- > T

Re: Review Request 49616: Add suppression benchmark.

2016-07-05 Thread Jacob Janco
/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 49616: Add suppression benchmark.

2016-07-29 Thread Jacob Janco
frameworks suppressing offers allocate() took 17.318823secs to make 0 offers with 6000 out of 6000 frameworks suppressing offers ... Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-08-16 Thread Jacob Janco
allocator settled after 7us [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.FrameworkFailover/24 (6332 ms) Thanks, Jacob Janco

Review Request 51028: Fix tests with rapidly triggered allocations.

2016-08-16 Thread Jacob Janco
--- Thanks, Jacob Janco

Review Request 51027: Track allocation candidates to bound allocator.

2016-08-16 Thread Jacob Janco
/master/allocator/mesos/hierarchical.cpp 234ef98529964a0b6d3f132426a6c8ccbb1263ee Diff: https://reviews.apache.org/r/51027/diff/ Testing --- Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-08-16 Thread Jacob Janco
or settled after 7us [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.FrameworkFailover/24 (6332 ms) Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-17 Thread Jacob Janco
allocator settled after 25.525654secs [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.FrameworkFailover/22 (220137 ms) Thanks, Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-08-16 Thread Jacob Janco
://reviews.apache.org/r/51028/diff/ Testing (updated) --- make check Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-16 Thread Jacob Janco
, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-16 Thread Jacob Janco
filters depends on https://reviews.apache.org/r/51028 Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-02-03 Thread Jacob Janco
utomatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/#review164111 --- On Jan. 31, 2017, 1:46 a.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://rev

Re: Review Request 56296: Fix to potential dangling pointer in `batch()`.

2017-02-03 Thread Jacob Janco
cpp 5f540569043df9d2bb75416c8c36bb4dd7bd68a1 Diff: https://reviews.apache.org/r/56296/diff/ Testing (updated) --- make check. Thanks, Jacob Janco

Review Request 56296: Fix to potential dangling pointer in `batch()`.

2017-02-03 Thread Jacob Janco
ing --- Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-02-03 Thread Jacob Janco
t; > Jiang Yan Xu wrote: > Ah I see what you mean now, thanks! I guess we can copy > `allocationInterval` in. > > Jacob Janco wrote: > ^I'll submit a patch with the above fix. https://reviews.apache.org/r/56296 - Jacob -

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-02-01 Thread Jacob Janco
check Thanks, Jacob Janco

Re: Review Request 52534: Dispatch filter expiration twice.

2017-01-31 Thread Jacob Janco
/ Testing --- With https://reviews.apache.org/r/51027/: GTEST_FILTER="-*SmallOfferFilter*" make check -j8 Thanks, Jacob Janco

Re: Review Request 52534: Dispatch filter expiration twice.

2017-01-31 Thread Jacob Janco
--- On Jan. 31, 2017, 1:47 a.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52534/ > -

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-31 Thread Jacob Janco
check Thanks, Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-31 Thread Jacob Janco
check Thanks, Jacob Janco

Re: Review Request 55893: Fixed OversubscriptionTest.RescindRevocableOfferWithIncreasedRevocable.

2017-01-31 Thread Jacob Janco
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55893/#review163723 --- Ship it! Ship It! - Jacob Janco On Jan. 31, 2017, 9:03 a.m

Re: Review Request 55791: Rework clipboard functionality in UI.

2017-01-22 Thread Jacob Janco
880e64ee7614e224660c6616b4bbb1ee5fab8ec9 Diff: https://reviews.apache.org/r/55791/diff/ Testing (updated) --- make check + verification of clipboard functionality Thanks, Jacob Janco

Review Request 55889: Fixes to comments in review 54216.

2017-01-24 Thread Jacob Janco
to comments in review 54216. Diffs - src/webui/master/static/js/controllers.js b6364fa636a6dc4b8d314285bec7dc19eb1c9c3b src/webui/master/static/pailer.html 2f48d2393088bd1eceab0f25174c0366cdb41694 Diff: https://reviews.apache.org/r/55889/diff/ Testing --- Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-01-26 Thread Jacob Janco
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/#review162229 --- On Jan. 12, 2017, 6:55 p.m., Jacob Janco wrote: > >

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-01-26 Thread Jacob Janco
, RescindRevocableOfferWithIncreasedRevocable), fix in 51621 Thanks, Jacob Janco

Re: Review Request 52534: Dispatch filter expiration twice.

2017-01-26 Thread Jacob Janco
/ Testing --- With https://reviews.apache.org/r/51027/: GTEST_FILTER="-*SmallOfferFilter*" make check -j8 Thanks, Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-26 Thread Jacob Janco
yea I like that better - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51028/#review162722 --- On Jan. 27, 2

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-26 Thread Jacob Janco
eviews.apache.org/r/51028/#review162237 --- On Jan. 27, 2017, 2:33 a.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-26 Thread Jacob Janco
--- make check Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-01-26 Thread Jacob Janco
d, was waiting for that commit =D - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/#review162529 --- On Jan. 27, 2

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-30 Thread Jacob Janco
://reviews.apache.org/r/51028/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 55852: Fixed MasterAllocatorTest/1.RebalancedForUpdatedWeights.

2017-01-30 Thread Jacob Janco
> On Jan. 30, 2017, 11:51 p.m., Jacob Janco wrote: > > Ship It! Makes sense, with batched allocation recovery and allocation can be interleaved causing flakiness in this test. - Jacob --- This is an automatically generat

Re: Review Request 55893: Fixed OversubscriptionTest.RescindRevocableOfferWithIncreasedRevocable.

2017-01-30 Thread Jacob Janco
108546b97 testing::internal::HandleExceptionsInMethodIfSupported<>() @0x108546a68 testing::UnitTest::Run() @0x107642ee1 RUN_ALL_TESTS() @0x10763ec1d main @ 0x7fffc2091255 start Segmentation fault: 11 - Jacob Janco On Jan. 24, 2

Re: Review Request 55852: Fixed MasterAllocatorTest/1.RebalancedForUpdatedWeights.

2017-01-30 Thread Jacob Janco
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55852/#review163594 --- Ship it! Ship It! - Jacob Janco On Jan. 23, 2017, 7:08 p.m

Re: Review Request 55852: Fixed MasterAllocatorTest/1.RebalancedForUpdatedWeights.

2017-01-30 Thread Jacob Janco
> On Jan. 30, 2017, 11:51 p.m., Jacob Janco wrote: > > Ship It! > > Jacob Janco wrote: > Makes sense, with batched allocation recovery and allocation can be > interleaved causing flakiness in this test. Passed --gtest_

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-01-30 Thread Jacob Janco
, RescindRevocableOfferWithIncreasedRevocable), fix in 51621 Thanks, Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-30 Thread Jacob Janco
/51028/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 52534: Dispatch filter expiration twice.

2017-01-30 Thread Jacob Janco
://reviews.apache.org/r/52534/diff/ Testing --- With https://reviews.apache.org/r/51027/: GTEST_FILTER="-*SmallOfferFilter*" make check -j8 Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2017-01-30 Thread Jacob Janco
(OversubscriptionTest, RescindRevocableOfferWithIncreasedRevocable), fix in 51621 Thanks, Jacob Janco

Re: Review Request 55874: Added a simple AllocatorBacklog benchmark.

2017-01-30 Thread Jacob Janco
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55874/#review163611 --- Ship it! Ship It! - Jacob Janco On Jan. 24, 2017, 9:46 p.m

Re: Review Request 54410: Made the style of the cluster name consistent with others.

2017-01-25 Thread Jacob Janco
> On Dec. 9, 2016, 7:51 p.m., Benjamin Mahler wrote: > > If it looks the same as the other navbar tabs, it seems clickable? At least > > when it is greyed out I wouldn't think I can click it. > > haosdent huang wrote: > Got it, use `btn disabled` to make it grey now. Hey, responding to

Review Request 55791: Rework clipboard functionality in UI.

2017-01-20 Thread Jacob Janco
d45551d6c9f75c275032219768549f76865b9cd3 src/webui/master/static/js/zeroclipboard-1.1.7.min.js 32535fddf1baab255fcbf5d3d776a0a9ac1447e8 src/webui/master/static/obj/zeroclipboard-1.1.7.swf 880e64ee7614e224660c6616b4bbb1ee5fab8ec9 Diff: https://reviews.apache.org/r/55791/diff/ Testing --- Thanks, Jacob

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2017-01-18 Thread Jacob Janco
/diff/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 55691: Fix XSS vulnerability in pailer invocation.

2017-01-18 Thread Jacob Janco
19e0981143bd7e8372b49f4f036867e9dd05727a Diff: https://reviews.apache.org/r/55691/diff/ Testing --- make -j8 + test framework + checking pailer representation of files in sandbox Thanks, Jacob Janco

Review Request 55691: Fix XSS vulnerability in pailer invocation.

2017-01-18 Thread Jacob Janco
framework + checking pailer representation of files in sandbox Thanks, Jacob Janco

Re: Review Request 55691: Fix XSS vulnerability in pailer invocation.

2017-01-19 Thread Jacob Janco
w162303 --- On Jan. 18, 2017, 11:40 p.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-03 Thread Jacob Janco
25.525654secs [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.FrameworkFailover/22 (220137 ms) Thanks, Jacob Janco

Re: Review Request 49617: Add benchmark for failover of many frameworks.

2016-09-03 Thread Jacob Janco
> > > > allocation += createPorts(ranges.get()); > > ``` I took this out to simplify the test. - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49617/#review146625 --- On

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-01 Thread Jacob Janco
following > > event in the queue, are we still going to get the correct result? > > ``` > > batch -> addFramework(f1) -> addFramework(f2) > > ``` > > Jacob Janco wrote: > insert in a loop? > > Klaus Ma wrote: > No; if multiple fr

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
on. - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51028/#review147697 --- On Aug. 23, 2016, 8:47 a.m., Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
ly, visit: https://reviews.apache.org/r/51028/#review147809 --- On Aug. 23, 2016, 8:47 a.m., Jacob Janco wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
- Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51028/#review147697 --- On Aug. 23, 2016, 8:47 a.m., Jacob Janco wrote

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-08 Thread Jacob Janco
> > finished after `addSlave` finished. > > > > Shall we do a `while` loop in the benchmark to wait till all > > allocations are got? > > Guangya Liu wrote: > Perhaps we can simply remove the `ASSERT_EQ` for offer and agent count in > `Hierarchic

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-30 Thread Jacob Janco
25.525654secs [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.FrameworkFailover/22 (220137 ms) Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-30 Thread Jacob Janco
(220137 ms) Thanks, Jacob Janco

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-08-30 Thread Jacob Janco
generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/#review146919 --- On Aug. 23, 2016, 8:49 a.m., Jacob Janco wrote: > > --- > This is an automatically

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-07 Thread Jacob Janco
------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/#review147722 --- On Sept. 3, 2016, 6:05 a.m., Jacob Janco wrote: > >

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-07 Thread Jacob Janco
> ``` Yea, I think preserving the original behavior was the idea here so the metrics reflect that. - Jacob --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5102

Re: Review Request 51027: Track allocation candidates to bound allocator.

2016-09-08 Thread Jacob Janco
> > finished after `addSlave` finished. > > > > Shall we do a `while` loop in the benchmark to wait till all > > allocations are got? > > Guangya Liu wrote: > Perhaps we can simply remove the `ASSERT_EQ` for offer and agent count in > `Hierarchic

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
this test may > > be failed as r/51027 > > Jacob Janco wrote: > Since there is an `AWAIT_READY(allocation)` beforehand for the previous 2 > calls to the allocator, I believe we should be OK here. > > Guangya Liu wrote: > Sorry, I posted this in the wrong place, it

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
/ Testing --- make check Thanks, Jacob Janco

Re: Review Request 51028: Fix tests with rapidly triggered allocations.

2016-09-08 Thread Jacob Janco
this test may > > be failed as r/51027 > > Jacob Janco wrote: > Since there is an `AWAIT_READY(allocation)` beforehand for the previous 2 > calls to the allocator, I believe we should be OK here. > > Guangya Liu wrote: > Sorry, I posted this in the wrong place, it

  1   2   3   >