You are viewing a plain text version of this content. The canonical link for it is here.
- Review Request 27467: Retry Driver.killTask periodically until the task changes states. - posted by Bill Farner <wf...@apache.org> on 2014/11/01 18:48:45 UTC, 12 replies.
- Review Request 27492: AURORA-617: Switch pants 3rdparty to use python_requirements - posted by Dan Norris <pr...@gmail.com> on 2014/11/02 23:18:05 UTC, 15 replies.
- Re: Review Request 27317: Adding resource consumption calculation for cron jobs. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/03 19:00:24 UTC, 4 replies.
- Review Request 27538: Optionally ignore unknown transaction operations. - posted by Bill Farner <wf...@apache.org> on 2014/11/03 21:30:52 UTC, 3 replies.
- Review Request 27541: Revert "Adding resource consumption calculation for cron jobs." - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/03 22:59:18 UTC, 2 replies.
- Review Request 27542: Add test coverage for WriteAheadStorage. - posted by Bill Farner <wf...@apache.org> on 2014/11/03 23:09:37 UTC, 13 replies.
- Review Request 27545: Replace twitter.commons.io.FileUtils dependency w/ guava's Files - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/04 00:09:03 UTC, 4 replies.
- Review Request 27586: Increase robustness when reading PEX-INFO - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/04 20:39:39 UTC, 12 replies.
- Review Request 27591: Remove --bind-var for cron deschedule. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/04 22:39:33 UTC, 7 replies.
- Review Request 27598: Fixing the log replay for the job update history pruner. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/05 00:10:53 UTC, 17 replies.
- Review Request 27601: Adding resource consumption calculation for cron jobs. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/05 02:06:30 UTC, 9 replies.
- Review Request 27628: Convert most uses of Mock to create_autospec, remove some uses of mocking altogether. - posted by Bill Farner <wf...@apache.org> on 2014/11/05 18:58:46 UTC, 20 replies.
- Review Request 27630: Mark inotify disk collector test as flaky. - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/05 19:04:59 UTC, 5 replies.
- Review Request 27650: Remove duplicate call to log handling code. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/06 01:02:46 UTC, 5 replies.
- Review Request 27657: Print out the job url after scheduling a cron job. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/06 02:33:54 UTC, 12 replies.
- Review Request 27698: Move zookeeper connection off the main thread to prevent client deadlocks. - posted by David McLaughlin <da...@dmclaughlin.com> on 2014/11/06 21:45:32 UTC, 17 replies.
- Review Request 27705: Adding instrumentation into the scheduling pipeline. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/06 23:23:53 UTC, 30 replies.
- Review Request 27710: Remove stracktrace redirection. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/07 00:40:41 UTC, 11 replies.
- Review Request 27712: Fixing beta-update OK status messaging. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/07 01:30:37 UTC, 6 replies.
- Review Request 27716: Remove SchedulerActive, replace with explicit lifecycle. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/07 03:03:30 UTC, 12 replies.
- Review Request 27747: Splitting cron and non-cron job creation in client v2. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/07 22:59:06 UTC, 9 replies.
- Review Request 27746: Simplify management of the driver lifecycle using AbstractidleService. - posted by Bill Farner <wf...@apache.org> on 2014/11/07 23:03:24 UTC, 11 replies.
- Review Request 27757: Replace bindStartupAction with Service registration - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/08 01:06:36 UTC, 17 replies.
- Review Request 27762: Use async eventbus - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/08 01:39:13 UTC, 6 replies.
- Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage. - posted by Bill Farner <wf...@apache.org> on 2014/11/08 05:24:15 UTC, 0 replies.
- Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage. - posted by Aurora ReviewBot <wf...@apache.org> on 2014/11/08 05:43:11 UTC, 6 replies.
- Review Request 27772: Fix fails pass on SchedulerDriverServiceTest case. - posted by Bill Farner <wf...@apache.org> on 2014/11/08 06:46:16 UTC, 0 replies.
- Re: Review Request 27772: Fix false passing test case in SchedulerDriverServiceTest. - posted by Bill Farner <wf...@apache.org> on 2014/11/08 06:53:00 UTC, 2 replies.
- Review Request 27778: Fix false positive client test. - posted by Bill Farner <wf...@apache.org> on 2014/11/08 20:06:20 UTC, 1 replies.
- Review Request 27779: Create the PyCharm project configuration for easier IDE setup. - posted by Bill Farner <wf...@apache.org> on 2014/11/08 20:27:48 UTC, 3 replies.
- Review Request 27828: Move read-modify-write logic out of AttributeStore.Mutable for simpler layering. - posted by Bill Farner <wf...@apache.org> on 2014/11/10 21:50:37 UTC, 5 replies.
- Review Request 27830: Undo 0.6.0 .auroraversion and CHANGELOG changes to prepare for 0.6.0-RC1 - posted by Bill Farner <wf...@apache.org> on 2014/11/10 22:11:51 UTC, 1 replies.
- Review Request 27848: Add friendly error message to the client when lock is held. - posted by David McLaughlin <da...@dmclaughlin.com> on 2014/11/11 01:28:07 UTC, 20 replies.
- Review Request 27852: Ensure run verb returns an exit code. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/11 03:50:03 UTC, 7 replies.
- Review Request 27880: Fix missing paren in review bot. - posted by Bill Farner <wf...@apache.org> on 2014/11/11 19:31:04 UTC, 4 replies.
- Review Request 27889: Send an event for host attributes changing rather than maintenance mode changing. - posted by Bill Farner <wf...@apache.org> on 2014/11/11 23:42:56 UTC, 6 replies.
- Review Request 27892: Fixes for the make pycharm script: - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/12 00:11:13 UTC, 8 replies.
- Review Request 27894: Bump commons version. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/12 01:26:05 UTC, 5 replies.
- Review Request 27902: Store host attributes alongside offers to reduce number of lookups. - posted by Bill Farner <wf...@apache.org> on 2014/11/12 19:58:48 UTC, 10 replies.
- Review Request 27934: Remove non-fixed tickets from CHANGELOG. - posted by Bill Farner <wf...@apache.org> on 2014/11/12 23:35:02 UTC, 3 replies.
- Review Request 27935: Fix coverage epsilon. - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/12 23:50:22 UTC, 4 replies.
- Review Request 27941: Fix review bot to use correct latest diff time. - posted by Bill Farner <wf...@apache.org> on 2014/11/13 00:54:19 UTC, 2 replies.
- Review Request 27942: Don't fail builds on exceeded coverage. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/13 01:01:32 UTC, 4 replies.
- Review Request 27947: Setting the max thread limit on AsyncEventBus. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/13 02:12:37 UTC, 14 replies.
- Review Request 27949: Don't fail the build when exceeding coverage thresholds. - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/13 02:20:22 UTC, 1 replies.
- Review Request 27954: Fix bad review text in review bot. - posted by Bill Farner <wf...@apache.org> on 2014/11/13 03:03:49 UTC, 5 replies.
- Review Request 27996: Upgrade to gradle 2.2 - posted by Bill Farner <wf...@apache.org> on 2014/11/13 18:05:31 UTC, 4 replies.
- Review Request 28003: Remove 0.6.0 CHANGELOG entries in prep for 0.6.0-RC2. - posted by Bill Farner <wf...@apache.org> on 2014/11/13 22:15:32 UTC, 0 replies.
- Review Request 28005: Reset CHANGELOG and .auroraversion in prep for 0.6.0-RC2. - posted by Bill Farner <wf...@apache.org> on 2014/11/13 22:19:16 UTC, 1 replies.
- Re: Review Request 26899: Require StateManager callers to open their own transactions. - posted by Bill Farner <wf...@apache.org> on 2014/11/13 22:47:24 UTC, 2 replies.
- Review Request 28019: Show update page URL in the output for beta-update start. - posted by David McLaughlin <da...@dmclaughlin.com> on 2014/11/14 01:19:21 UTC, 7 replies.
- Review Request 28021: Remove dependency on commons-lang. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/14 02:07:02 UTC, 6 replies.
- Review Request 28026: Add more test coverage to SchedulerThriftInterface. - posted by Bill Farner <wf...@apache.org> on 2014/11/14 02:30:07 UTC, 11 replies.
- Review Request 28048: Fix task_util dependency and add it to all target. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/14 20:00:16 UTC, 2 replies.
- Re: Review Request 27852: Ensure all verbs return an exit code. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/14 23:16:32 UTC, 10 replies.
- Review Request 28066: Fixing flaky test. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/15 02:06:24 UTC, 4 replies.
- Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead. - posted by Bill Farner <wf...@apache.org> on 2014/11/15 19:58:04 UTC, 12 replies.
- Review Request 28103: Simplify Preemptor code, encapsulate parameters used there and in SchedulingFilter. - posted by Bill Farner <wf...@apache.org> on 2014/11/16 02:22:46 UTC, 9 replies.
- Review Request 28140: Drop argparse dependency; it's part of python 2.7 - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/18 01:29:46 UTC, 3 replies.
- Review Request 28170: AURORA-933 - python missing license headers - posted by Jake Farrell <jf...@apache.org> on 2014/11/18 16:46:49 UTC, 3 replies.
- Review Request 28193: Prevent Aurora from creating zero sized Executor tasks. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/19 00:33:04 UTC, 29 replies.
- Review Request 28198: Add an example on how to build with Docker. - posted by Tony Dong <to...@gmail.com> on 2014/11/19 01:33:58 UTC, 22 replies.
- Review Request 28237: Add default pip timeout to pants bootstrapping. - posted by Joshua Cohen <jc...@twopensource.com> on 2014/11/19 18:36:32 UTC, 4 replies.
- Review Request 28250: Fix test_version to use explicit injection points. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/19 21:44:47 UTC, 11 replies.
- Review Request 28272: Avoid creating garbage copies of Snapshot#tasks and AssignedTask#task. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/20 02:08:10 UTC, 10 replies.
- Review Request 28302: Fail builds if not compiled with Gradle 2.2 - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/21 00:05:55 UTC, 12 replies.
- Review Request 28306: Return an iterable of frame chunks. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/21 01:54:52 UTC, 19 replies.
- Review Request 28310: Extract a cluster state abstraction from PreemptorImpl. - posted by Bill Farner <wf...@apache.org> on 2014/11/21 03:14:40 UTC, 20 replies.
- Re: Review Request 27182: Add a test for the thermos resource module - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/21 20:26:10 UTC, 0 replies.
- Review Request 28345: Move thermos_runner out of the apache.aurora.executor package. - posted by Brian Wickman <wi...@apache.org> on 2014/11/21 20:52:18 UTC, 10 replies.
- Review Request 28350: Add cron replace command. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/22 00:14:10 UTC, 16 replies.
- Review Request 28360: Make PreemptorImpl package private. - posted by Bill Farner <wf...@apache.org> on 2014/11/22 02:34:23 UTC, 4 replies.
- Review Request 28361: Extract thrift into an API subproject. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/22 02:35:27 UTC, 21 replies.
- Review Request 28398: Adding cron check into aurora beta-update start. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/24 20:19:56 UTC, 6 replies.
- Review Request 28409: Bump the min coverage values. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/24 22:11:20 UTC, 4 replies.
- Review Request 28411: Adding quota check into replaceCronTemplate rpc. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/24 22:27:29 UTC, 6 replies.
- Review Request 28445: Improving messages in CronJobManager. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/25 19:58:43 UTC, 2 replies.
- Review Request 28447: Disambiguate tag push from branch push in release script. - posted by Bill Farner <wf...@apache.org> on 2014/11/25 21:36:05 UTC, 5 replies.
- Review Request 28449: Fix some URLs and text in release emails to include "-incubating" where appropriate. - posted by Bill Farner <wf...@apache.org> on 2014/11/25 22:00:02 UTC, 3 replies.
- Review Request 28450: Fixing admin query command - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/25 22:16:50 UTC, 3 replies.
- Review Request 28451: Modify TRequestsTransport to raise an exception 4xx or 5xx responses. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/25 22:22:32 UTC, 15 replies.
- Review Request 28452: In review bot, tee test output rather than piping to a file. - posted by Bill Farner <wf...@apache.org> on 2014/11/25 22:34:57 UTC, 1 replies.
- Review Request 28460: Fix resolutionStrategy to be global. - posted by Kevin Sweeney <ke...@apache.org> on 2014/11/26 02:28:30 UTC, 2 replies.
- Review Request 28474: Added manual perf tests for the scheduling pipeline. - posted by Maxim Khutornenko <ma...@apache.org> on 2014/11/26 19:15:38 UTC, 2 replies.
- Review Request 28483: Improve unbounded ref exception. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/26 23:12:02 UTC, 6 replies.
- Review Request 28486: Move sandbox initialization into try block. - posted by Zameer Manji <zm...@twopensource.com> on 2014/11/27 00:02:07 UTC, 3 replies.