reviews  

Messages by Date

• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62700: Convert Webhook to AbstractIdleService, use async HTTP client Aurora ReviewBot
• 2017/09/29 Re: Review Request 62700: Convert Webhook to AbstractIdleService, use async HTTP client David McLaughlin
• 2017/09/29 Re: Review Request 62700: Convert Webhook to AbstractIdleService, use async HTTP client Aurora ReviewBot
• 2017/09/29 Review Request 62700: Convert Webhook to AbstractIdleService, use async HTTP client Jordan Ly
• 2017/09/29 Re: Review Request 62601: Remove the rewriteConfigs thrift method Aurora ReviewBot
• 2017/09/29 Re: Review Request 62692: Move job environment validation to the scheduler. Bill Farner
• 2017/09/29 Re: Review Request 62601: Remove the rewriteConfigs thrift method Bill Farner
• 2017/09/29 Re: Review Request 62601: Remove the rewriteConfigs thrift method Reza Motamedi
• 2017/09/29 Re: Review Request 62601: Remove the rewriteConfigs thrift method Jordan Ly
• 2017/09/29 Re: Review Request 62692: Move job environment validation to the scheduler. Mauricio Garavaglia
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system David McLaughlin
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Santhosh Kumar Shanmugham
• 2017/09/29 Re: Review Request 62692: Move job environment validation to the scheduler. Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Jordan Ly
• 2017/09/29 Re: Review Request 62692: Move job environment validation to the scheduler. Aurora ReviewBot
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Santhosh Kumar Shanmugham
• 2017/09/29 Review Request 62692: Move job environment validation to the scheduler. Mauricio Garavaglia
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/29 Re: Review Request 62623: Use a simpler command line argument system Reza Motamedi
• 2017/09/28 Re: Review Request 62623: Use a simpler command line argument system Santhosh Kumar Shanmugham
• 2017/09/28 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Reza Motamedi
• 2017/09/28 Re: Review Request 62652: Remove legacy commons ZK code Stephan Erb
• 2017/09/28 Re: Review Request 62652: Remove legacy commons ZK code Renan DelValle
• 2017/09/27 Re: Review Request 62623: Use a simpler command line argument system Aurora ReviewBot
• 2017/09/27 Review Request 62623: Use a simpler command line argument system Bill Farner
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Joshua Cohen
• 2017/09/27 Re: Review Request 62652: Remove legacy commons ZK code Bill Farner
• 2017/09/27 Re: Review Request 62652: Remove legacy commons ZK code Aurora ReviewBot
• 2017/09/27 Re: Review Request 62652: Remove legacy commons ZK code David McLaughlin
• 2017/09/27 Review Request 62652: Remove legacy commons ZK code Bill Farner
• 2017/09/27 Re: Review Request 62397: Replica Hot Standby Bill Farner
• 2017/09/27 Re: Review Request 62601: Remove the rewriteConfigs thrift method Bill Farner
• 2017/09/27 Re: Review Request 62590: WIP: Update to Thrift 0.10.0 Bill Farner
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Bill Farner
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. Aurora ReviewBot
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. Reza Motamedi
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. Santhosh Kumar Shanmugham
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. Kai Huang
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. Kai Huang
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Jordan Ly
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Stephan Erb
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Stephan Erb
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Aurora ReviewBot
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Santhosh Kumar Shanmugham
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Kai Huang
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Aurora ReviewBot
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Santhosh Kumar Shanmugham
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Jordan Ly
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Kai Huang
• 2017/09/27 Re: Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Reza Motamedi
• 2017/09/27 Review Request 62626: Added additional stop() to prevent errors in run() to stop shutdown in SchedulerMain Jordan Ly
• 2017/09/27 Re: Review Request 62601: Remove the rewriteConfigs thrift method Stephan Erb
• 2017/09/27 Re: Review Request 62590: WIP: Update to Thrift 0.10.0 Stephan Erb
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Stephan Erb
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Jordan Ly
• 2017/09/27 Re: Review Request 62397: Replica Hot Standby Stephan Erb
• 2017/09/27 Re: Review Request 62620: Fix binding issues preventing ./gradle run from working Aurora ReviewBot
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Aurora ReviewBot
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Aurora ReviewBot
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Jordan Ly
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Aurora ReviewBot
• 2017/09/27 Re: Review Request 62620: Fix binding issues preventing ./gradle run from working Aurora ReviewBot
• 2017/09/27 Re: Review Request 62620: Fix binding issues preventing ./gradle run from working Jordan Ly
• 2017/09/27 Re: Review Request 62608: Workaround to get pants working in macOS high sierra Aurora ReviewBot
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Jordan Ly
• 2017/09/27 Re: Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Bill Farner
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Bill Farner
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Aurora ReviewBot
• 2017/09/27 Review Request 62621: Allow transitions from any state to STOPPED in CallOrderEnforcingStorage Jordan Ly
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62608: Workaround to get pants working in macOS high sierra Bill Farner
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Joshua Cohen
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Aurora ReviewBot
• 2017/09/27 Re: Review Request 62607: Replace Preact and custom testing with React + Enzyme Bill Farner
• 2017/09/27 Review Request 62620: Fix binding issues preventing ./gradle run from working Bill Farner
• 2017/09/27 Review Request 62607: Replace Preact and custom testing with React + Enzyme David McLaughlin
• 2017/09/26 Re: Review Request 62601: Remove the rewriteConfigs thrift method David McLaughlin
• 2017/09/26 Re: Review Request 62601: Remove the rewriteConfigs thrift method David McLaughlin
• 2017/09/26 Re: Review Request 62608: Workaround to get pants working in macOS high sierra John Sirois
• 2017/09/26 Re: Review Request 62608: Workaround to get pants working in macOS high sierra Aurora ReviewBot
• 2017/09/26 Re: Review Request 62608: Workaround to get pants working in macOS high sierra Bill Farner
• 2017/09/26 Re: Review Request 62608: Workaround to get pants working in macOS high sierra Aurora ReviewBot
• 2017/09/26 Review Request 62608: Workaround to get pants working in macOS high sierra Bill Farner
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Reza Motamedi
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Aurora ReviewBot
• 2017/09/26 Re: Review Request 62601: Remove the rewriteConfigs thrift method Aurora ReviewBot
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Bill Farner
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Aurora ReviewBot
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Bill Farner
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Bill Farner
• 2017/09/26 Re: Review Request 62604: Use a more efficient query for instance ID collision detection Jordan Ly
• 2017/09/26 Re: Review Request 62601: Remove the rewriteConfigs thrift method Bill Farner
• 2017/09/26 Re: Review Request 62451: Implement Role and Environment pages in Preact. Kai Huang
• 2017/09/26 Review Request 62604: Use a more efficient query for instance ID collision detection Bill Farner
• 2017/09/26 Re: Review Request 62601: Remove the rewriteConfigs thrift method Aurora ReviewBot
• 2017/09/26 Re: Review Request 62590: WIP: Update to Thrift 0.10.0 Bill Farner
• 2017/09/26 Re: Review Request 62590: WIP: Update to Thrift 0.10.0 Bill Farner
• 2017/09/26 Review Request 62601: Remove the rewriteConfigs thrift method Bill Farner
• 2017/09/26 Re: Review Request 62590: WIP: Update to Thrift 0.10.0 Aurora ReviewBot
• 2017/09/26 Review Request 62590: WIP: Update to Thrift 0.10.0 Stephan Erb
• 2017/09/26 Re: Review Request 62558: Restore scheduler benchmarks to working order David McLaughlin
• 2017/09/26 Re: Review Request 62558: Restore scheduler benchmarks to working order Stephan Erb
• 2017/09/26 Re: Review Request 62558: Restore scheduler benchmarks to working order Stephan Erb
• 2017/09/25 Re: Review Request 62558: Restore scheduler benchmarks to working order Aurora ReviewBot
• 2017/09/25 Review Request 62558: Restore scheduler benchmarks to working order Bill Farner
• 2017/09/25 Re: Review Request 62451: Implement Role and Environment pages in Preact. Reza Motamedi
• 2017/09/25 Re: Review Request 62451: Implement Role and Environment pages in Preact. Santhosh Kumar Shanmugham
• 2017/09/25 Re: Review Request 62451: Implement Role and Environment pages in Preact. Kai Huang
• 2017/09/24 Re: Review Request 62524: Build with gradle 4.2 Stephan Erb
• 2017/09/23 Review Request 62524: Build with gradle 4.2 Bill Farner
• 2017/09/22 Re: Review Request 62517: Update to gradle 4.2 Stephan Erb
• 2017/09/22 Re: Review Request 62517: Update to gradle 4.2 David McLaughlin
• 2017/09/22 Re: Review Request 62517: Update to gradle 4.2 Aurora ReviewBot
• 2017/09/22 Review Request 62517: Update to gradle 4.2 Bill Farner
• 2017/09/21 Re: Review Request 62451: Implement Role and Environment pages in Preact. Aurora ReviewBot
• 2017/09/21 Review Request 62451: Implement Role and Environment pages in Preact. David McLaughlin
• 2017/09/21 Re: Review Request 62423: Improve in-process test ZooKeeper support Stephan Erb
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Aurora ReviewBot
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Keisuke Nishimoto
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Bill Farner
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Aurora ReviewBot
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Keisuke Nishimoto
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Keisuke Nishimoto
• 2017/09/20 Re: Review Request 61249: Cron timezone should be configurable per job David McLaughlin
• 2017/09/20 Re: Review Request 62423: Improve in-process test ZooKeeper support Bill Farner
• 2017/09/20 Re: Review Request 61249: Cron timezone should be configurable per job Bill Farner
• 2017/09/19 Re: Review Request 62423: Improve in-process test ZooKeeper support Aurora ReviewBot
• 2017/09/19 Review Request 62423: Improve in-process test ZooKeeper support Keisuke Nishimoto
• 2017/09/19 Re: Review Request 62220: Add ca-certificates-java to ubuntu-trusty Dockerfile Stephan Erb
• 2017/09/18 Re: Review Request 62397: Replica Hot Standby Aurora ReviewBot
• 2017/09/17 Re: Review Request 62347: Add Houghton Mifflin Harcourt to adopters list Stephan Erb
• 2017/09/15 Re: Review Request 62347: Add Houghton Mifflin Harcourt to adopters list Aurora ReviewBot
• 2017/09/14 Review Request 62347: Add Houghton Mifflin Harcourt to adopters list Robert Allen
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact Aurora ReviewBot
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact Aurora ReviewBot
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact Aurora ReviewBot
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/13 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/12 Re: Review Request 62135: HomePage implemented in Preact Kai Huang
• 2017/09/12 Re: Review Request 61249: Cron timezone should be configurable per job Aurora ReviewBot
• 2017/09/12 Re: Review Request 61249: Cron timezone should be configurable per job Jing Chen
• 2017/09/11 Re: Review Request 62220: Add ca-certificates-java to ubuntu-trusty Dockerfile Santhosh Kumar Shanmugham
• 2017/09/11 Re: Review Request 62220: Add ca-certificates-java to ubuntu-trusty Dockerfile Renan DelValle
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact Kai Huang
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact Kai Huang
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact Kai Huang
• 2017/09/11 Re: Review Request 62135: HomePage implemented in Preact Joshua Cohen
• 2017/09/11 Review Request 62220: Add ca-certificates-java to ubuntu-trusty Dockerfile Nicolás Donatucci
• 2017/09/08 Re: Review Request 62135: HomePage implemented in Preact Aurora ReviewBot
• 2017/09/08 Review Request 62135: HomePage implemented in Preact David McLaughlin
• 2017/09/07 Re: Review Request 61249: Cron timezone should be configurable per job Jing Chen
• 2017/08/31 Re: Review Request 60942: Remove task level resource fields from thrift interface and db Nicolás Donatucci
• 2017/08/31 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Dmitry Zhuk
• 2017/08/30 Re: Review Request 60942: Remove task level resource fields from thrift interface and db Joshua Cohen
• 2017/08/30 Re: Review Request 61249: Cron timezone should be configurable per job Stephan Erb
• 2017/08/30 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Stephan Erb
• 2017/08/30 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Aurora ReviewBot
• 2017/08/30 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Aurora ReviewBot
• 2017/08/30 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Jordan Ly
• 2017/08/30 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Dmitry Zhuk
• 2017/08/29 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Aurora ReviewBot
• 2017/08/29 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Jordan Ly
• 2017/08/28 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Santhosh Kumar Shanmugham
• 2017/08/28 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Santhosh Kumar Shanmugham
• 2017/08/28 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Aurora ReviewBot
• 2017/08/28 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Jordan Ly
• 2017/08/28 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Santhosh Kumar Shanmugham
• 2017/08/25 Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers Aurora ReviewBot
• 2017/08/25 Review Request 61918: Fix concurrency issues around banned offers in HostOffers Jordan Ly
• 2017/08/24 Re: Review Request 61864: Bootstrap the build pipeline for new Preact UI. Aurora ReviewBot
• 2017/08/24 Re: Review Request 61864: Bootstrap the build pipeline for new Preact UI. David McLaughlin
• 2017/08/24 Re: Review Request 61864: Bootstrap the build pipeline for new Preact UI. David McLaughlin
• 2017/08/24 Re: Review Request 61864: Bootstrap the build pipeline for new Preact UI. Joshua Cohen

• Earlier messages
• Later messages