Re: Review Request 43635: Changed scalar resources to use fixed-point internally.

value was big. Scalar is a general type, there maybe used with a big value, e.g. total size of distributed filesystem. src/tests/resources_tests.cpp (line 23) <https://reviews.apache.org/r/43635/#comment180731> Se

Re: Review Request 43635: Changed scalar resources to use fixed-point internally.

> On Feb. 17, 2016, 8:30 a.m., Klaus Ma wrote: > > src/common/values.cpp, line 67 > > <https://reviews.apache.org/r/43635/diff/1/?file=1252182#file1252182line67> > > > > Let's add check on overflow; it will be helpful if scalar value was > > big

Re: Review Request 41306: MESOS-1718: use command line executor to launch tasks.

66898e914c7b4ab83c4580be67530f355cfb05ca src/slave/slave.cpp f0be0d5bf4b853952caf06e2e262c2903d79ead3 Diff: https://reviews.apache.org/r/41306/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41305: MESOS-1718: move getExecutorInfo from slave to master.

/slave.cpp f0be0d5bf4b853952caf06e2e262c2903d79ead3 Diff: https://reviews.apache.org/r/41305/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

, Klaus Ma

Re: Review Request 41302: MESOS-1718: add slave's configuration into SlaveInfo.

proto e24d3e03a7dc7c6bfd07f34531cb593fe4925646 include/mesos/v1/mesos.proto d909e60ddfd8e3ba2075f82c372edde04cd99d54 Diff: https://reviews.apache.org/r/41302/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41305: MESOS-1718: move getExecutorInfo from slave to master.

gt; does not have this chain of patches? > > - Have you considered completely converting a `CommandInfo` to > > `ExecutorInfo`? If we do that, it may be possible to remove all the other > > places with special logic for command tasks. > > Klaus Ma wrote: > * It&#

Review Request 43754: WIP: Enhance allocator performance.

frameworks Added 100 frameworks in 1846us Added 1 slaves in 4.953045secs Updated 1 slaves in 4.338845secs [ OK ] SlaveAndFrameworkCount/HierarchicalAllocator_BENCHMARK_Test.AddAndUpdateSlave/14 (10309 ms) ``` Thanks, Klaus Ma

Re: Review Request 43635: Changed scalar resources to use fixed-point internally.

> On Feb. 17, 2016, 8:30 a.m., Klaus Ma wrote: > > src/common/values.cpp, line 60 > > <https://reviews.apache.org/r/43635/diff/1/?file=1252182#file1252182line60> > > > > Regarnding `lround`, it maybe overcommit resources, if framework keep > > lau

Review Request 43802: Wrapped TASK_LOST with `` in authorization.md

--- Wrapped TASK_LOST with `` in authorization.md Diffs - docs/authorization.md bbb4f2a Diff: https://reviews.apache.org/r/43802/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 43802: Wrapped TASK_LOST with `` in authorization.md.

://reviews.apache.org/r/43802/diff/ Testing --- Thanks, Klaus Ma

Review Request 43807: Replaced with /Users/klaus in authentication.md.

Mac OS. Failed when using `~`: I0221 13:08:41.717308 4284416 credentials.hpp:35] Loading credentials for authentication from '~/credentials' Failed to read credentials file '~/credentials': Failed to open file '~/credentials': No such file or directory (see --credentials flag) Thanks, Klaus Ma

Re: Review Request 43807: Replaced with `~` in authentication.md.

27; Failed to read credentials file '~/credentials': Failed to open file '~/credentials': No such file or directory (see --credentials flag) Thanks, Klaus Ma

Re: Review Request 43718: Added fs::supported() function.

718/#comment181438> Would you add comments on expected format in `/proc/systems`? It'll help other contributors to understand magic number `1` & `2`. - Klaus Ma On Feb. 21, 2016, 3:46 p.m., Shuai Lin wrote: > > ---

Re: Review Request 37168: Add an example framework using dynamic reservation.

ttps://reviews.apache.org/r/37168/diff/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 37168: Added example for dynamic reservation features.

://reviews.apache.org/r/37168/diff/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 37531: Fix master CHECK failure if a framework uses duplicated task id.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37531/#review120073 --- ping @jieyu/vinodkone. - Klaus Ma On Jan. 13, 2016, 10:06 p.m

Re: Review Request 43807: Replaced with `~` in authentication.md.

ly, visit: https://reviews.apache.org/r/43807/#review120099 --- On Feb. 21, 2016, 2:42 p.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 39939: Make docker_socket option support different protocols.

g/r/39939/#comment181473> I think we need to handle feature interaction with `docker_mesos_image`. Currently, we mount unix socket into container; but it'll failed when using TCP socket. - Klaus Ma On Feb. 22, 2016, 11:35 a.m., haosdent

Re: Review Request 43821: Updated the HA framweork guide for TASK_KILLING.

aviour if executor generate `TASK_KILLING` without framework's capability: ignore, crash or undefined? - Klaus Ma On Feb. 22, 2016, 1:19 p.m., Ben Mahler wrote: > > --- > This is an automatically generated e-mail. To reply, vi

Re: Review Request 43838: Added note about not implemented requestResources call.

tps://reviews.apache.org/r/43838/#comment181681> It seems scheduler did not send `ResourceRequestMessage` to the master; so other allocator also can not handle it. I'd like to say Master ignore this call. - Klaus Ma On Feb. 23, 2016, 12:54 a.m., Joerg

Review Request 43864: Fix typo of roles doc.

--- Fix typo of roles doc. Diffs - docs/endpoints/master/roles.json.md d67779c246cceae2209f2611f32ada4493ae6f83 docs/endpoints/master/roles.md 976a9b7891a17652289126ec7e7ee73cea0c2e35 Diff: https://reviews.apache.org/r/43864/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 43816: Updated `/frameworks` master endpoint to use jsonify.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43816/#review120272 --- Ship it! Ship It! - Klaus Ma On Feb. 22, 2016, 3:11 p.m

Re: Review Request 43822: Updated `/slaves` master endpoint to use jsonify.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43822/#review120291 --- Ship it! Ship It! - Klaus Ma On Feb. 22, 2016, 3:12 p.m

Re: Review Request 43823: Updated `/tasks` master endpoint to use jsonify.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43823/#review120292 --- Ship it! Ship It! - Klaus Ma On Feb. 22, 2016, 3:13 p.m

Re: Review Request 43817: Removed no-longer-used model functions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43817/#review120293 --- Ship it! Ship It! - Klaus Ma On Feb. 22, 2016, 3:13 p.m

Re: Review Request 43848: Used `size_t` to track number of frameworks per role.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43848/#review120294 --- Ship it! Ship It! - Klaus Ma On Feb. 23, 2016, 3:54 a.m

Re: Review Request 42390: Fixed fetching uris when slave is running inside a container.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42390/#review120296 --- Ship it! Ship It! - Klaus Ma On Jan. 20, 2016, 10:12 p.m

Re: Review Request 43868: Add a getter for master::Flags.

o, please clear it up. Can you also show which part of code will use this function? - Klaus Ma On Feb. 23, 2016, 1:23 p.m., Jay Guo wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 43864: Fix typo of roles doc.

/http.cpp 939fab21a2240de7214ef809a194ffb3837a9f1b Diff: https://reviews.apache.org/r/43864/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 43864: Fix typo of roles doc.

.org/r/43864/#review120347 --- On Feb. 24, 2016, 11:34 a.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To reply, visit: &

Re: Review Request 43879: Added allocator metrics for number of allocations made.

tps://reviews.apache.org/r/43879/#comment181976> Why a anonymous namespace? `static` should be fine. src/tests/hierarchical_allocator_tests.cpp (line 2395) <https://reviews.apache.org/r/43879/#comment181975> What are you going to do for this comments? or just a notes? - Klaus

Re: Review Request 43880: Added allocator metrics for total and allocated scalar resources.

eviews.apache.org/r/43880/#comment181980> Can we move the comments above the code and move `getMetric()` into one line? - Klaus Ma On Feb. 24, 2016, 6:58 p.m., Benjamin Bannier wrote: > > --- > This is an automatically generated e

Re: Review Request 43881: Added allocator metric for the number of allocations to a framework.

;https://reviews.apache.org/r/43881/#comment182182> ditto - Klaus Ma On Feb. 25, 2016, 7:37 p.m., Benjamin Bannier wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 43882: Added allocation metrics for allocation time.

tps://reviews.apache.org/r/43882/#comment182184> It seems a general functional class, can we move it into `process/metrics`? - Klaus Ma On Feb. 25, 2016, 7:37 p.m., Benjamin Bannier wrote: > > --- > This is an automatica

Re: Review Request 43718: Added fs::supported() function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43718/#review121140 --- Ship it! Ship It! - Klaus Ma On Feb. 28, 2016, 9:08 p.m

Re: Review Request 43718: Added fs::supported() function.

> On Feb. 28, 2016, 9:16 p.m., Guangya Liu wrote: > > src/linux/fs.cpp, line 60 > > > > > > I think that should `continue` here, if one line failed, other lines > > should still be tried to see if the fs is supportte

Re: Review Request 44063: Waited for status update to happen before proceeding in test.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44063/#review121143 --- Ship it! Ship It! - Klaus Ma On Feb. 28, 2016, 6:27 p.m

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

g/r/44047/#comment182798> s/slave/agent/ src/master/http.cpp (line 1047) <https://reviews.apache.org/r/44047/#comment182799> `` for /unreserve and /destroy-volumes. - Klaus Ma On Feb. 28, 2016, 8:17 a.m., Neil

Re: Review Request 43883: Added allocator metrics for number of offer filters per framework.

tps://reviews.apache.org/r/43883/#comment182801> Should we remove this `Gauge` when remove framework? - Klaus Ma On Feb. 27, 2016, 1:01 a.m., Benjamin Bannier wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 43884: Added allocator metrics for used quotas.

tps://reviews.apache.org/r/43884/#comment182802> Remove it when removeQuota. src/master/allocator/mesos/hierarchical.cpp (line 1108) <https://reviews.apache.org/r/43884/#comment182803> This's not necessary; all resources in quotaSorter are `nonRevocable()`. - Klaus Ma On

Re: Review Request 43881: Added allocator metric for the number of allocations to a framework.

> On Feb. 26, 2016, 12:30 a.m., Klaus Ma wrote: > > src/master/allocator/mesos/hierarchical.hpp, line 376 > > <https://reviews.apache.org/r/43881/diff/7/?file=1270846#file1270846line376> > > > > We also need to remove counter in `removeFramework`; or

Re: Review Request 44110: Updated flag examples to refer to /role instead of stats.json.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44110/#review121151 --- Ship it! Ship It! - Klaus Ma On Feb. 27, 2016, 8:55 a.m

Re: Review Request 43561: Improve Ranges parsing to handle single values.

/resources_tests.cpp a545100522bf4b1f03e50656d461b3cda6b41e11 src/tests/values_tests.cpp 929861549e3155c33966896f817f9bf9e6d14354 Diff: https://reviews.apache.org/r/43561/diff/ Testing --- make make check GTEST_FILTER=~"*" ./src/mesos-test Thanks, Klaus Ma

Review Request 44236: Update slave's version after re-registering.

ces": {}, "reserved_resources_full": {}, "resources": { "cpus": 8.0, "disk": 470832.0, "mem": 15360.0, "ports": "[31000-32000]" }, "unreserved_resources": { "cpus": 8.0, "disk": 470832.0, "mem": 15360.0, "ports": "[31000-32000]" }, "used_resources": { "cpus": 0.0, "disk": 0.0, "mem": 0.0 }, "used_resources_full": [], "version": "0.27.0" } ] } ``` Thanks, Klaus Ma

Re: Review Request 44236: Update slave's version after re-registering.

01:5051", "registered_time": 1456874482.32198, "reregistered_time": 1456874497.25686, "reserved_resources": {}, "reserved_resources_full": {}, "resources": { "cpus": 8.0, "disk": 470832.0, "mem": 15360.0, "ports": "[31000-32000]" }, "unreserved_resources": { "cpus": 8.0, "disk": 470832.0, "mem": 15360.0, "ports": "[31000-32000]" }, "used_resources": { "cpus": 0.0, "disk": 0.0, "mem": 0.0 }, "used_resources_full": [], "version": "0.27.0" } ] } ``` Thanks, Klaus Ma

Re: Review Request 44254: Add curl_easy_cleanup when fopen is failed.

159) <https://reviews.apache.org/r/44254/#comment183322> also close fd (`os::close(fd.get())`) - Klaus Ma On March 2, 2016, 11:45 a.m., Andy Pang wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 44254: Add curl_easy_cleanup when fopen is failed.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44254/#review121595 --- Ship it! Ship It! - Klaus Ma On March 2, 2016, 2:21 p.m

Re: Review Request 44258: Fixed http endpoint trigger two inverse offer calls.

nd remove the // unavailability. master->machines[id].info.set_mode(MachineInfo::UP); master->updateUnavailability(id, None()); } ``` - Klaus Ma On March 2, 2016, 3:45 p.m., Guangya Liu wrote: > > --

Review Request 44313: Described "/maintanence/schedule" GET/POST in the two paragraph.

/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 44258: Fixed http endpoint trigger two inverse offer calls.

> On March 2, 2016, 10:43 p.m., Klaus Ma wrote: > > src/master/http.cpp, lines 1999-2000 > > <https://reviews.apache.org/r/44258/diff/1/?file=1276346#file1276346line1999> > > > > I think we can just remove `master->updateUnavailability(id, > > upda

Re: Review Request 39939: Make docker_socket option support different protocols.

> On Feb. 22, 2016, 12:41 p.m., Klaus Ma wrote: > > src/slave/flags.cpp, line 437 > > <https://reviews.apache.org/r/39939/diff/1/?file=1115726#file1115726line437> > > > > I think we need to handle feature interaction with > > `docker_mesos_image

Re: Review Request 39939: Make docker_socket option support different protocols.

gt; Two comments here: 1. Error message; I'd like to say: "Failed to create docker container; only unix domain socket is supported for `docker_socket` when `docker_mesos_image` is enabled" 2. Can you also highlight this limitation/restriction to document/help - Kl

Re: Review Request 39939: Make docker_socket option support different protocols.

> On March 4, 2016, 11:02 a.m., Klaus Ma wrote: > > src/slave/containerizer/docker.cpp, lines 136-138 > > <https://reviews.apache.org/r/39939/diff/3/?file=1280373#file1280373line136> > > > > Two comments here: > > > > 1. Error messa

Re: Review Request 39939: Make docker_socket option support different protocols.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39939/#review122012 --- Ship it! Ship It! - Klaus Ma On March 4, 2016, 11:54 a.m

Re: Review Request 44366: Added GPUs as an explicit resource.

idia_gpus` must be set when specifying `gpu` resources." - Klaus Ma On March 4, 2016, 9:11 a.m., Kevin Klues wrote: > > --- > This is an automatically generated e-mail. To reply, visit:

Review Request 44396: Updated unavailable in batch to avoid several allocate(slaveId) call in "/maintenance/schedule".

://reviews.apache.org/r/44396/diff/ Testing --- make && make check at Mac OS Thanks, Klaus Ma

Re: Review Request 44366: Added GPUs as an explicit resource.

> On March 4, 2016, 8:27 p.m., Klaus Ma wrote: > > src/slave/containerizer/containerizer.cpp, line 115 > > <https://reviews.apache.org/r/44366/diff/1/?file=1280284#file1280284line115> > > > > I'd like to say "`--navidia_gpus` must be set when s

Re: Review Request 44360: Added a script to install the Nvidia GDK on a host.

tps://reviews.apache.org/r/44360/#comment184127> Are we going to provide documents about those vars? e.g. which version are we going to support? - Klaus Ma On March 4, 2016, 9:21 a.m., Kevin Klues wrote: > > --- > This is an automati

Re: Review Request 44408: Remove setting up of ACLs (unneeded) in PersistentVolumeTests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44408/#review122218 --- Ship it! Ship It! - Klaus Ma On March 5, 2016, 12:17 p.m

Re: Review Request 44414: Added documentation about container image support.

md (lines 67 - 77) <https://reviews.apache.org/r/44414/#comment184131> +1 - Klaus Ma On March 5, 2016, 10:20 a.m., Jie Yu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: >

Re: Review Request 43884: Added allocator metrics for used quotas.

tps://reviews.apache.org/r/43884/#comment184146> Add comments on keys, e.g. role : . - Klaus Ma On March 5, 2016, 12:01 a.m., Benjamin Bannier wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 37168: Add an example framework using dynamic reservation.

s.cpp 6ddac17bb2ac0330bcc09eaab975ae70e84a7695 Diff: https://reviews.apache.org/r/37168/diff/ Testing --- make make check Thanks, Klaus Ma

Review Request 44433: Added empty line for list in maintenance doc.

--- Added empty line for list in maintenance doc. Diffs - docs/maintenance.md 365c920719dbd0c5e61efe1975547a2848647bce Diff: https://reviews.apache.org/r/44433/diff/ Testing --- Document update. Thanks, Klaus Ma

Re: Review Request 37168: Add an example framework using dynamic reservation.

ws.apache.org/r/37168/#review120212 --- On March 7, 2016, 5:20 p.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews

Re: Review Request 44570: Added an owner field to TaskInfo and ExecutorInfo.

org/r/44570/#comment185003> Also added in v1 APIs? src/master/validation.cpp (lines 381 - 382) <https://reviews.apache.org/r/44570/#comment185002> move `{` into one line. - Klaus Ma On March 9, 2016, 11:48 p.m., Jan Sc

Re: Review Request 40375: Support distinguishing revocable resources in the Resource protobuf.

/resources.cpp 296c4c2392384a2fcc4f2c1843980ff97e71114d Diff: https://reviews.apache.org/r/40375/diff/ Testing --- make && make check Thanks, Klaus Ma

Re: Review Request 40379: MESOS-3930: Set resource type as USAGE_SLACK for Oversubscription.

840534ff0687e82ed063c386e36bbabada230697 Diff: https://reviews.apache.org/r/40379/diff/ Testing --- make (make check is on-going) Thanks, Klaus Ma

Re: Review Request 41850: Add map to trace evictable executors.

can be evicted. Diffs (updated) - src/slave/slave.hpp 7520cc356e2b1b7f5fff15f33071a46a7b05e762 src/slave/slave.cpp 840534ff0687e82ed063c386e36bbabada230697 Diff: https://reviews.apache.org/r/41850/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41855: Trace pending executors.

from pendingExecutor list. Diffs (updated) - src/slave/slave.hpp 7520cc356e2b1b7f5fff15f33071a46a7b05e762 src/slave/slave.cpp 840534ff0687e82ed063c386e36bbabada230697 Diff: https://reviews.apache.org/r/41855/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41856: Added helper functions to get allocation slack.

840534ff0687e82ed063c386e36bbabada230697 src/v1/resources.cpp 296c4c2392384a2fcc4f2c1843980ff97e71114d Diff: https://reviews.apache.org/r/41856/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41857: Got evictable executors.

7520cc356e2b1b7f5fff15f33071a46a7b05e762 src/slave/slave.cpp 840534ff0687e82ed063c386e36bbabada230697 Diff: https://reviews.apache.org/r/41857/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41858: Added helper functions to kill executor.

7520cc356e2b1b7f5fff15f33071a46a7b05e762 src/slave/slave.cpp 840534ff0687e82ed063c386e36bbabada230697 Diff: https://reviews.apache.org/r/41858/diff/ Testing --- Thanks, Klaus Ma

Re: Review Request 41859: Launch tasks after evicting executors.

/ Testing --- Thanks, Klaus Ma

Re: Review Request 42127: Did not rescind offer if only ALLOCATION_SLACK resources incuded.

) - src/master/master.cpp 249e82ffcef35aa8df3c5b9faef5b9b25d68facc src/tests/oversubscription_tests.cpp e528476cd83b0e3f7ae8cea7d86dfabc1f66484e Diff: https://reviews.apache.org/r/42127/diff/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 42590: Renamed reserved() to reservations().

tps://reviews.apache.org/r/42590/#comment185472> I think we need to update document about the `Option role` - Klaus Ma On March 12, 2016, 10:47 a.m., Guangya Liu wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 40375: Support distinguishing revocable resources in the Resource protobuf.

src/tests/resources_tests.cpp 6b004d64bb25112b19fc5d98b5bca874c5329e8c src/v1/resources.cpp c6f125ec317e2da537a6456f5cff2da0a48701d8 Diff: https://reviews.apache.org/r/40375/diff/ Testing --- make && make check Thanks, Klaus Ma

Re: Review Request 44360: Added a script to install the Nvidia GDK on a host.

> On March 6, 2016, 8:51 a.m., Klaus Ma wrote: > > support/install-nvidia-gdk.sh, lines 3-13 > > <https://reviews.apache.org/r/44360/diff/2/?file=1280297#file1280297line3> > > > > Are we going to provide documents about those vars? e.g. which version > &

Re: Review Request 40375: Support distinguishing revocable resources in the Resource protobuf.

6b004d64bb25112b19fc5d98b5bca874c5329e8c src/v1/resources.cpp c6f125ec317e2da537a6456f5cff2da0a48701d8 Diff: https://reviews.apache.org/r/40375/diff/ Testing --- make && make check Thanks, Klaus Ma

Re: Review Request 40375: Support distinguishing revocable resources in the Resource protobuf.

/resources.cpp c6f125ec317e2da537a6456f5cff2da0a48701d8 Diff: https://reviews.apache.org/r/40375/diff/ Testing --- make && make check Thanks, Klaus Ma

Re: Review Request 42547: Added helper function to get non usage slack resources.

tps://reviews.apache.org/r/42547/#comment186110> should be `nonThrottleable`? - Klaus Ma On March 15, 2016, 10:32 p.m., Guangya Liu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 41333: Added helper functions to filter usage slack resources.

tps://reviews.apache.org/r/41333/#comment186112> `isThrottleable`? - Klaus Ma On March 15, 2016, 10:33 p.m., Guangya Liu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 41334: Added helper functions to filter allocation slack resources.

tps://reviews.apache.org/r/41334/#comment186113> I think we should also update those helper fuction according to protobuf change, using `throttleable` instead of ALLOCATION_SLACK/USAGE_SLACK, e.g. `throttleable()`, `nonThrottleable()` and `revocable()`. - Klaus Ma On March 15, 2016, 10:33 p.m., Guang

Re: Review Request 37168: Add an example framework using dynamic reservation.

make check Thanks, Klaus Ma

Re: Review Request 37168: Add an example framework using dynamic reservation.

----- On March 16, 2016, 2:27 p.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37168/ >

Re: Review Request 37168: Add an example framework using dynamic reservation.

Thanks, Klaus Ma

Re: Review Request 37168: Add an example framework using dynamic reservation.

make check Thanks, Klaus Ma

Re: Review Request 44836: Update TaskId in long_lived_framework to use fixed length taskid.

tps://reviews.apache.org/r/44836/#comment186878> I think we need check `format()`'s result; if `tasksLaunched` is bigger than `99`, what's the behaviour? - Klaus Ma On March 16, 2016, 10:27 a.m., Jay Guo wrote: > > --

Re: Review Request 37168: Add an example framework using dynamic reservation.

/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 37168: Add an example framework using dynamic reservation.

-------- On March 16, 2016, 2:32 p.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37168/ > --

Re: Review Request 37168: Add an example framework using dynamic reservation.

VED state the > > first time? It seems to me we should have something here similar to what's > > in the switch for `State::UNRESERVING`, i.e., check the offer and see if > > the reserved resources are contained in it? The framework test passes for > > me thou

Review Request 45081: Excluded reserved resources when got nonRevocable resources in stage 1.

l.cpp 70291075c00a9a557529c2562dedcfc6c6c3ec32 Diff: https://reviews.apache.org/r/45081/diff/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 45081: Excluded reserved resources when got nonRevocable resources in stage 1.

ry to call `nonRevocable()` for reserved resources. Diffs - src/master/allocator/mesos/hierarchical.cpp 70291075c00a9a557529c2562dedcfc6c6c3ec32 Diff: https://reviews.apache.org/r/45081/diff/ Testing --- make make check Thanks, Klaus Ma

Re: Review Request 37168: Add an example framework using dynamic reservation.

make check Thanks, Klaus Ma

Re: Review Request 43561: Improve Ranges parsing to handle single values.

ot; ./src/mesos-test Thanks, Klaus Ma

Re: Review Request 43561: Improve Ranges parsing to handle single values.

GTEST_FILTER=~"*" ./src/mesos-test Thanks, Klaus Ma

Re: Review Request 43561: Improve Ranges parsing to handle single values.

---------- On March 22, 2016, 9:22 p.m., Klaus Ma wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43561/ >

Re: Review Request 43561: Improve Ranges parsing to handle single values.

-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43561/#review124545 --- On March 22, 2016, 9:22 p.m., Klaus Ma wrote: > > -

Re: Review Request 45267: Fixed a memory leak in process::subprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45267/#review125200 --- Ship it! Ship It! - Klaus Ma On March 24, 2016, 11:41 a.m

Re: Review Request 45267: Fixed a memory leak in process::subprocess.

tps://reviews.apache.org/r/45267/#comment187990> Sugguest to add a NOTE that, the envp is environment->size() + 1, so the last NULL ptr is not handled (delete). - Klaus Ma On March 24, 2016, 11:41 a.m., Ben Mahler wrote: > > -

  1   2   3   4   5   6   7   >