mesos-reviews mailing list archives: December 2016

Site index · List index
Message list« Previous · 1 · 2 · 3 · 4 · 5 · 6 · 7 · 8 · 9 · 10 · 11 · 12 · Next »Thread · Author · Date
haosdent huang Review Request 54832: Removed extra space in method definitions in src [2/2]. Sat, 17 Dec, 09:38
Mesos ReviewBot   Re: Review Request 54832: Removed extra space in method definitions in src [2/2]. Sat, 17 Dec, 11:02
Joseph Wu   Re: Review Request 54832: Removed extra space in method definitions in src [2/2]. Mon, 19 Dec, 21:52
Re: Review Request 53291: Fixed incorrect `/monitor/statistics` URL in WebUI.
haosdent huang   Re: Review Request 53291: Fixed incorrect `/monitor/statistics` URL in WebUI. Sat, 17 Dec, 09:40
haosdent huang Review Request 54833: Fixed the metrics display bug in agent.html. Sat, 17 Dec, 09:41
haosdent huang   Re: Review Request 54833: Fixed the metrics display bug in agent.html. Sat, 17 Dec, 09:48
Joseph Wu     Re: Review Request 54833: Fixed the metrics display bug in agent.html. Mon, 19 Dec, 21:49
haosdent huang Review Request 54834: Fixed a typo in webui. Sat, 17 Dec, 09:41
Mesos ReviewBot   Re: Review Request 54834: Fixed a typo in webui. Sat, 17 Dec, 12:52
Benjamin Hindman Review Request 54838: Used process::io::BUFFERED_READ_SIZE instead of constant. Sat, 17 Dec, 21:35
Kevin Klues   Re: Review Request 54838: Used process::io::BUFFERED_READ_SIZE instead of constant. Sat, 17 Dec, 21:39
Review Request 54839: Updated Mesos process::loop uses with process::ControlFlow.
Benjamin Hindman   Review Request 54839: Updated Mesos process::loop uses with process::ControlFlow. Sat, 17 Dec, 21:37
Kevin Klues   Re: Review Request 54839: Updated Mesos process::loop uses with process::ControlFlow. Sat, 17 Dec, 21:53
Benjamin Hindman     Re: Review Request 54839: Updated Mesos process::loop uses with process::ControlFlow. Sun, 18 Dec, 01:29
Review Request 54840: Used process::loop to avoid stack overflow due to recursion.
Benjamin Hindman   Review Request 54840: Used process::loop to avoid stack overflow due to recursion. Sat, 17 Dec, 21:38
Kevin Klues   Re: Review Request 54840: Used process::loop to avoid stack overflow due to recursion. Sat, 17 Dec, 21:45
Benjamin Hindman   Re: Review Request 54840: Used process::loop to avoid stack overflow due to recursion. Sat, 17 Dec, 21:50
Jie Yu     Re: Review Request 54840: Used process::loop to avoid stack overflow due to recursion. Mon, 19 Dec, 17:06
Jie Yu       Re: Review Request 54840: Used process::loop to avoid stack overflow due to recursion. Mon, 19 Dec, 18:27
Benjamin Hindman Review Request 54841: Used `loop` in implementation of io::read and io::write. Sat, 17 Dec, 21:40
Benjamin Hindman   Re: Review Request 54841: Used `loop` in implementation of io::read and io::write. Sun, 18 Dec, 01:04
Mesos ReviewBot     Re: Review Request 54841: Used `loop` in implementation of io::read and io::write. Mon, 19 Dec, 14:39
Jie Yu     Re: Review Request 54841: Used `loop` in implementation of io::read and io::write. Wed, 21 Dec, 19:30
Benjamin Mahler Review Request 54842: Updated Resources to handle Resource.AllocationInfo. Sat, 17 Dec, 22:50
Guangya Liu   Re: Review Request 54842: Updated Resources to handle Resource.AllocationInfo. Sun, 18 Dec, 00:56
Benjamin Mahler Review Request 54836: Added helpers to allocate / unallocate Resources. Sat, 17 Dec, 22:50
Mesos ReviewBot   Re: Review Request 54836: Added helpers to allocate / unallocate Resources. Sat, 17 Dec, 23:49
Guangya Liu   Re: Review Request 54836: Added helpers to allocate / unallocate Resources. Sun, 18 Dec, 01:10
Benjamin Mahler     Re: Review Request 54836: Added helpers to allocate / unallocate Resources. Sun, 18 Dec, 16:35
Guangya Liu     Re: Review Request 54836: Added helpers to allocate / unallocate Resources. Mon, 19 Dec, 01:37
Benjamin Mahler   Re: Review Request 54836: Added helpers to allocate / unallocate Resources. Sun, 18 Dec, 16:36
Benjamin Mahler Review Request 54837: Added a CHECK when adding a framework in the allocator. Sat, 17 Dec, 22:50
Guangya Liu   Re: Review Request 54837: Added a CHECK when adding a framework in the allocator. Sun, 18 Dec, 01:12
Mesos ReviewBot   Re: Review Request 54837: Added a CHECK when adding a framework in the allocator. Sun, 18 Dec, 01:21
haosdent huang Review Request 54846: Removed `docker exec` when perform health checks in docker executor. Sun, 18 Dec, 17:26
haosdent huang   Re: Review Request 54846: Removed `docker exec` when perform health checks in docker executor. Sun, 18 Dec, 17:30
Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options.
Zhitao Li   Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Sun, 18 Dec, 17:37
haosdent huang   Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Sun, 18 Dec, 17:50
haosdent huang   Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Mon, 19 Dec, 02:41
Mesos ReviewBot   Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Tue, 20 Dec, 01:19
Zhitao Li   Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Wed, 21 Dec, 18:52
haosdent huang     Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Thu, 22 Dec, 04:21
Zhitao Li       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Thu, 22 Dec, 17:21
haosdent huang       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Fri, 23 Dec, 03:16
haosdent huang       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Fri, 23 Dec, 03:19
haosdent huang       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Fri, 23 Dec, 04:40
Zhitao Li     Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Thu, 22 Dec, 17:17
haosdent huang       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Fri, 23 Dec, 04:38
Mesos ReviewBot       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Tue, 27 Dec, 07:22
Zhitao Li       Re: Review Request 54821: Refactored Docker::run() to make it only aware of docker cli options. Sat, 31 Dec, 01:27
Re: Review Request 54035: Extended test coverage of posix/rlimits isolator.
Benjamin Bannier   Re: Review Request 54035: Extended test coverage of posix/rlimits isolator. Mon, 19 Dec, 11:07
Jie Yu     Re: Review Request 54035: Extended test coverage of posix/rlimits isolator. Tue, 20 Dec, 19:29
Benjamin Bannier Review Request 54859: Shortened socket paths used in test. Mon, 19 Dec, 11:16
Joseph Wu   Re: Review Request 54859: Shortened socket paths used in test. Mon, 19 Dec, 20:03
Re: Review Request 53713: Reversed recoursive path creation.
Joseph Wu   Re: Review Request 53713: Reversed recoursive path creation. Mon, 19 Dec, 18:34
Re: Review Request 53712: Added `system` environement variables in ` execvpe.cpp`.
Joseph Wu   Re: Review Request 53712: Added `system` environement variables in ` execvpe.cpp`. Mon, 19 Dec, 19:07
Daniel Pravat     Re: Review Request 53712: Added `system` environement variables in ` execvpe.cpp`. Mon, 19 Dec, 23:33
Jie Yu Review Request 54873: WIP: Added asynchronous libcurl support to Mesos. Mon, 19 Dec, 20:44
Mesos ReviewBot   Re: Review Request 54873: WIP: Added asynchronous libcurl support to Mesos. Tue, 20 Dec, 05:16
Jie Yu   Re: Review Request 54873: Added asynchronous libcurl support to Mesos. Sat, 24 Dec, 23:02
Jie Yu     Re: Review Request 54873: Added asynchronous libcurl support to Mesos. Sat, 24 Dec, 23:04
Jie Yu       Re: Review Request 54873: Added asynchronous libcurl support to Mesos. Mon, 26 Dec, 01:41
Avinash sridharan         Re: Review Request 54873: Added asynchronous libcurl support to Mesos. Mon, 26 Dec, 08:59
Avinash sridharan         Re: Review Request 54873: Added asynchronous libcurl support to Mesos. Mon, 26 Dec, 09:12
Re: Review Request 53237: Improved slave recovery tests to check reconciliation, metrics.
Neil Conway   Re: Review Request 53237: Improved slave recovery tests to check reconciliation, metrics. Mon, 19 Dec, 21:05
Re: Review Request 53237: Improved slave recovery tests.
Neil Conway   Re: Review Request 53237: Improved slave recovery tests. Mon, 19 Dec, 21:15
Vinod Kone     Re: Review Request 53237: Improved slave recovery tests. Wed, 21 Dec, 16:57
Neil Conway       Re: Review Request 53237: Improved slave recovery tests. Wed, 21 Dec, 17:42
Neil Conway     Re: Review Request 53237: Improved slave recovery tests. Wed, 21 Dec, 17:42
Re: Review Request 53239: Changed master to make use of "retired" agent IDs.
Neil Conway   Re: Review Request 53239: Changed master to make use of "retired" agent IDs. Mon, 19 Dec, 21:15
Andrew Schwartzmeyer Review Request 54875: CMake: Fixed typo `-DHAS_AUTHENTICATION=0` instructions. Mon, 19 Dec, 22:12
Joseph Wu   Re: Review Request 54875: CMake: Fixed typo `-DHAS_AUTHENTICATION=0` instructions. Mon, 19 Dec, 22:21
Alex Clemmer   Re: Review Request 54875: CMake: Fixed typo `-DHAS_AUTHENTICATION=0` instructions. Mon, 19 Dec, 22:51
Mesos ReviewBot   Re: Review Request 54875: CMake: Fixed typo `-DHAS_AUTHENTICATION=0` instructions. Tue, 20 Dec, 11:38
Re: Review Request 52190: Removed deprecated compiler warnings.
Daniel Pravat   Re: Review Request 52190: Removed deprecated compiler warnings. Mon, 19 Dec, 23:09
Andrew Schwartzmeyer Review Request 54877: Windows: Stout: Removed dependency on Shell API. Mon, 19 Dec, 23:20
Daniel Pravat   Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 00:29
Andrew Schwartzmeyer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 00:39
Alex Clemmer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 03:44
Andrew Schwartzmeyer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 18:50
Alex Clemmer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 19:54
Andrew Schwartzmeyer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Tue, 20 Dec, 21:28
Andrew Schwartzmeyer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Thu, 22 Dec, 18:32
Alex Clemmer     Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Sat, 24 Dec, 20:04
Mesos ReviewBot   Re: Review Request 54877: Windows: Stout: Removed dependency on Shell API. Wed, 21 Dec, 12:39
James Peach Review Request 54878: Add some simple ldd() tests. Mon, 19 Dec, 23:22
Re: Review Request 53707: Added a Windows isolator. Removed `#ifdef`-ed block from Posix.
Joseph Wu   Re: Review Request 53707: Added a Windows isolator. Removed `#ifdef`-ed block from Posix. Mon, 19 Dec, 23:34
Re: Review Request 53717: Added `Winsock` initialization in `docker-mesos-executor`.
Joseph Wu   Re: Review Request 53717: Added `Winsock` initialization in `docker-mesos-executor`. Tue, 20 Dec, 00:34
Avinash sridharan Review Request 54880: Added unit-test for dynamic addition/deletion of CNI config. Tue, 20 Dec, 03:37
Mesos ReviewBot   Re: Review Request 54880: Added unit-test for dynamic addition/deletion of CNI config. Tue, 20 Dec, 19:47
Avinash sridharan   Re: Review Request 54880: Added unit-test for dynamic addition/deletion of CNI config. Tue, 20 Dec, 22:36
Mesos ReviewBot     Re: Review Request 54880: Added unit-test for dynamic addition/deletion of CNI config. Wed, 21 Dec, 16:28
Jie Yu     Re: Review Request 54880: Added unit-test for dynamic addition/deletion of CNI config. Thu, 22 Dec, 01:50
Review Request 54889: Handled all possible offers in test.
Benjamin Bannier   Review Request 54889: Handled all possible offers in test. Tue, 20 Dec, 11:48
Neil Conway   Re: Review Request 54889: Handled all possible offers in test. Tue, 20 Dec, 16:41
Benjamin Bannier     Re: Review Request 54889: Handled all possible offers in test. Tue, 20 Dec, 22:15
Benjamin Bannier     Re: Review Request 54889: Handled all possible offers in test. Tue, 20 Dec, 22:15
Neil Conway     Re: Review Request 54889: Handled all possible offers in test. Wed, 21 Dec, 15:39
Benjamin Bannier     Re: Review Request 54889: Handled all possible offers in test. Wed, 21 Dec, 15:50
Neil Conway     Re: Review Request 54889: Handled all possible offers in test. Wed, 21 Dec, 16:41
Benjamin Bannier   Re: Review Request 54889: Handled all possible offers in test. Tue, 20 Dec, 22:15
Mesos ReviewBot     Re: Review Request 54889: Handled all possible offers in test. Wed, 21 Dec, 14:10
Neil Conway     Re: Review Request 54889: Handled all possible offers in test. Wed, 21 Dec, 16:41
Alexander Rukletsov     Re: Review Request 54889: Handled all possible offers in test. Thu, 22 Dec, 15:16
Re: Review Request 51028: Fix tests with rapidly triggered allocations.
Jacob Janco   Re: Review Request 51028: Fix tests with rapidly triggered allocations. Tue, 20 Dec, 12:02
Jacob Janco   Re: Review Request 51028: Fix tests with rapidly triggered allocations. Tue, 20 Dec, 12:02
Jacob Janco   Re: Review Request 51028: Fix tests with rapidly triggered allocations. Wed, 21 Dec, 05:59
Benjamin Bannier Review Request 54896: Fixed copy-template-and-create-symlink make target. Tue, 20 Dec, 13:53
James Peach   Re: Review Request 54896: Fixed copy-template-and-create-symlink make target. Tue, 20 Dec, 20:08
Benjamin Bannier     Re: Review Request 54896: Fixed copy-template-and-create-symlink make target. Wed, 21 Dec, 11:20
Benjamin Bannier   Re: Review Request 54896: Fixed copy-template-and-create-symlink make target. Wed, 21 Dec, 11:20
James Peach     Re: Review Request 54896: Fixed copy-template-and-create-symlink make target. Wed, 21 Dec, 17:49
Till Toenshoff     Re: Review Request 54896: Fixed copy-template-and-create-symlink make target. Thu, 22 Dec, 10:49
Jay Guo Review Request 54898: Added a CHECK in updateFrameworkInfo. Tue, 20 Dec, 15:06
Mesos ReviewBot   Re: Review Request 54898: Added a CHECK in updateFrameworkInfo. Wed, 21 Dec, 11:05
Neil Conway Review Request 54901: Updated description of "--agent_reregister_timeout". Tue, 20 Dec, 16:09
haosdent huang   Re: Review Request 54901: Updated description of "--agent_reregister_timeout". Tue, 20 Dec, 16:16
Vinod Kone   Re: Review Request 54901: Updated description of "--agent_reregister_timeout". Tue, 20 Dec, 16:25
Alex Clemmer Review Request 54909: Added member to agent to avoid spurious re-registrations. Tue, 20 Dec, 17:54
Alex Clemmer   Re: Review Request 54909: Added member to agent to avoid spurious re-registrations. Tue, 20 Dec, 23:18
Alex Clemmer     Re: Review Request 54909: Fixed spurious registration bug in framework and agent. Wed, 21 Dec, 03:09
Joseph Wu       Re: Review Request 54909: Fixed spurious registration bug in framework and agent. Thu, 22 Dec, 11:49
Alex Clemmer Review Request 54910: Fixed more tests to pass when `HAS_AUTHENTICATION` is undefined. Tue, 20 Dec, 18:02
Alex Clemmer   Re: Review Request 54910: Fixed more tests to pass when `HAS_AUTHENTICATION` is undefined. Tue, 20 Dec, 20:37
Alex Clemmer     Re: Review Request 54910: Fixed more tests to pass when `HAS_AUTHENTICATION` is undefined. Tue, 20 Dec, 22:09
Joseph Wu       Re: Review Request 54910: Fixed more tests to pass when `HAS_AUTHENTICATION` is undefined. Thu, 22 Dec, 11:49
Re: Review Request 53718: Defined and used Windows string `docker-mesos-executor.exe`.
Joseph Wu   Re: Review Request 53718: Defined and used Windows string `docker-mesos-executor.exe`. Tue, 20 Dec, 21:20
Re: Review Request 53719: Created default mount point on Windows.
Joseph Wu   Re: Review Request 53719: Created default mount point on Windows. Tue, 20 Dec, 21:29
Anand Mazumdar Review Request 54913: Added some missing log statements for scheduler calls. Tue, 20 Dec, 21:40
Joris Van Remoortere   Re: Review Request 54913: Added some missing log statements for scheduler calls. Tue, 20 Dec, 21:59
Anand Mazumdar   Re: Review Request 54913: Added some missing log statements for scheduler calls. Tue, 20 Dec, 23:09
Re: Review Request 53715: Define docker `--entrypoint` for `Windows`.
Joseph Wu   Re: Review Request 53715: Define docker `--entrypoint` for `Windows`. Tue, 20 Dec, 21:53
Re: Review Request 53716: Used `Shell::entrypoint`.
Joseph Wu   Re: Review Request 53716: Used `Shell::entrypoint`. Tue, 20 Dec, 22:06
Message list« Previous · 1 · 2 · 3 · 4 · 5 · 6 · 7 · 8 · 9 · 10 · 11 · 12 · Next »Thread · Author · Date
Box list
Apr 202038
Mar 2020342
Feb 2020191
Jan 2020303
Dec 2019159
Nov 2019239
Oct 2019162
Sep 2019312
Aug 2019540
Jul 2019707
Jun 2019621
May 2019472
Apr 2019702
Mar 2019955
Feb 2019758
Jan 2019749
Dec 2018455
Nov 2018598
Oct 2018832
Sep 2018704
Aug 20181218
Jul 20181040
Jun 20181026
May 20181520
Apr 20181445
Mar 20181350
Feb 20181396
Jan 20181470
Dec 20171464
Nov 20171769
Oct 20171445
Sep 20171054
Aug 20171341
Jul 20171295
Jun 20171475
May 20171547
Apr 20171266
Mar 20172102
Feb 20171631
Jan 20171655
Dec 20161774
Nov 20162300
Oct 20162196
Sep 20162748
Aug 20162311
Jul 20163001
Jun 20163216
May 20162928
Apr 20163332
Mar 20164025
Feb 20162997
Jan 20163700
Dec 20152431
Nov 20152744
Oct 20152284
Sep 20152621
Aug 20152300
Jul 20152051
Jun 20152449
May 20151444
Apr 2015338