reviews  

Messages by Thread

• • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark David McLaughlin
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Jordan Ly
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Jordan Ly
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Aurora ReviewBot
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Santhosh Kumar Shanmugham
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark David McLaughlin
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Jordan Ly
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Aurora ReviewBot
  • Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark Stephan Erb
• Review Request 66186: Upgrade to psutil with optimized Process.children() Stephan Erb
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Stephan Erb
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Aurora ReviewBot
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Stephan Erb
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Aurora ReviewBot
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Stephan Erb
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Reza Motamedi
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Stephan Erb
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Renan DelValle
  • Re: Review Request 66186: Upgrade to psutil with optimized Process.children() Santhosh Kumar Shanmugham
• Review Request 66154: Adding support for using custom executors via the Aurora DSL Renan DelValle
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Aurora ReviewBot
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL David McLaughlin
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL David McLaughlin
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Renan DelValle
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Reza Motamedi
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Renan DelValle
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Santhosh Kumar Shanmugham
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Renan DelValle
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Santhosh Kumar Shanmugham
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Aurora ReviewBot
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Aurora ReviewBot
  • Re: Review Request 66154: Adding support for using custom executors via the Aurora DSL Santhosh Kumar Shanmugham
• Review Request 66152: Custom executor support for Aurora DSL Renan DelValle
• Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Aurora ReviewBot
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Aurora ReviewBot
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Reza Motamedi
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Aurora ReviewBot
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Reza Motamedi
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Reza Motamedi
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Santhosh Kumar Shanmugham
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Santhosh Kumar Shanmugham
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
  • Re: Review Request 66139: Speedup regular Thermos observer checkpoint refresh Stephan Erb
• Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Santhosh Kumar Shanmugham
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Kai Huang
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Santhosh Kumar Shanmugham
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Santhosh Kumar Shanmugham
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Franck Cuny via Review Board
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Stephan Erb
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Stephan Erb
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Santhosh Kumar Shanmugham
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Santhosh Kumar Shanmugham
  • Re: Review Request 66103: Introduce mesos disk collector Reza Motamedi
  • Re: Review Request 66103: Introduce mesos disk collector Aurora ReviewBot
  • Re: Review Request 66103: Introduce mesos disk collector Stephan Erb
• Review Request 66136: Switch Thermos to lazy log formatting Stephan Erb
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Aurora ReviewBot
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Stephan Erb
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Aurora ReviewBot
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Jordan Ly
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Santhosh Kumar Shanmugham
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Jordan Ly
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Stephan Erb
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Aurora ReviewBot
  • Re: Review Request 66136: Switch Thermos to lazy log formatting Reza Motamedi
• Review Request 66074: Refactor ClusterState to more appropriate package, move binding to StateModule Jordan Ly
  • Re: Review Request 66074: Refactor ClusterState to more appropriate package, move binding to StateModule Aurora ReviewBot
  • Re: Review Request 66074: Refactor ClusterState to more appropriate package, move binding to StateModule Stephan Erb
• Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Jordan Ly
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Aurora ReviewBot
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on David McLaughlin
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Jordan Ly
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Santhosh Kumar Shanmugham
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Jordan Ly
  • Re: Review Request 65941: Avoid scheduling on the same host the ancestor of a task recently failed on Stephan Erb
• Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Jordan Ly
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Aurora ReviewBot
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Santhosh Kumar Shanmugham
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Renan DelValle
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Jordan Ly
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Aurora ReviewBot
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Renan DelValle
  • Re: Review Request 65896: Persist scheduler/observer logs to /var/log/aurora/[FILE].log Santhosh Kumar Shanmugham
• Review Request 65873: Upgrade RBT to 0.7.11 Renan DelValle
  • Re: Review Request 65873: Upgrade RBT to 0.7.11 Aurora ReviewBot
  • Re: Review Request 65873: Upgrade RBT to 0.7.11 Jordan Ly
  • Re: Review Request 65873: Upgrade RBT to 0.7.11 Renan DelValle
  • Re: Review Request 65873: Upgrade RBT to 0.7.11 Stephan Erb
• Review Request 65824: Second try at fixing cron id collision bug by avoiding state in Quartz jobs Jordan Ly
  • Re: Review Request 65824: Second try at fixing cron id collision bug by avoiding state in Quartz jobs Jordan Ly
  • Re: Review Request 65824: Second try at fixing cron id collision bug by avoiding state in Quartz jobs Aurora ReviewBot
  • Re: Review Request 65824: Second try at fixing cron id collision bug by avoiding state in Quartz jobs Renan DelValle
• Review Request 65810: Revert "Fix cron id collision bug by avoiding state in Quartz jobs" Jordan Ly
  • Re: Review Request 65810: Revert "Fix cron id collision bug by avoiding state in Quartz jobs" Aurora ReviewBot
  • Re: Review Request 65810: Revert "Fix cron id collision bug by avoiding state in Quartz jobs" David McLaughlin
• Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Jordan Ly
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Aurora ReviewBot
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Jordan Ly
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Renan DelValle
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Stephan Erb
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Jordan Ly
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Stephan Erb
  • Re: Review Request 65769: Remove unused module in RecoveryTool, move TaskTestUtil to test folder Stephan Erb
• Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Jordan Ly
  • Re: Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Renan DelValle
  • Re: Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Aurora ReviewBot
  • Re: Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Renan DelValle
  • Re: Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Jordan Ly
  • Re: Review Request 65761: Use StandardCharset instead of Charset.forName in ApiModule Aurora ReviewBot
• Review Request 65735: Add GPUs to the resources. Franck Cuny via Review Board
  • Re: Review Request 65735: Add GPUs to the resources. David McLaughlin
  • Re: Review Request 65735: Add GPUs to the resources. Aurora ReviewBot
  • Re: Review Request 65735: Add GPUs to the resources. Franck Cuny via Review Board
  • Re: Review Request 65735: Add GPUs to the resources. Franck Cuny via Review Board
  • Re: Review Request 65735: Add GPUs to the resources. Aurora ReviewBot
  • Re: Review Request 65735: Add GPUs to the resources. Franck Cuny via Review Board
  • Re: Review Request 65735: Add GPUs to the resources. Franck Cuny via Review Board
  • Re: Review Request 65735: Add GPUs to the resources. Aurora ReviewBot
  • Re: Review Request 65735: Add GPUs to the resources. Reza Motamedi
  • Re: Review Request 65735: Add GPUs to the resources. Santhosh Kumar Shanmugham
  • Re: Review Request 65735: Add GPUs to the resources. David McLaughlin
• Review Request 65690: Make charset parsing in HTTP headers case insensitve Jordan Ly
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Jordan Ly
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Aurora ReviewBot
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Renan DelValle
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Santhosh Kumar Shanmugham
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Jordan Ly
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Jordan Ly
  • Re: Review Request 65690: Make charset parsing in HTTP headers case insensitve Aurora ReviewBot
• Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Jordan Ly
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Aurora ReviewBot
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Jordan Ly
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Aurora ReviewBot
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Santhosh Kumar Shanmugham
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Jordan Ly
  • Re: Review Request 65680: Fix cron id collision bug by avoiding state in Quartz jobs Aurora ReviewBot
• Review Request 65650: Add GPG key for [email protected] Jordan Ly
  • Re: Review Request 65650: Add GPG key for [email protected] Aurora ReviewBot
  • Re: Review Request 65650: Add GPG key for [email protected] Renan DelValle
  • Re: Review Request 65650: Add GPG key for [email protected] Stephan Erb
• Review Request 65648: Do not reschedule a PARTITIONED task if it was in KILLING state David McLaughlin
  • Re: Review Request 65648: Do not reschedule a PARTITIONED task if it was in KILLING state Aurora ReviewBot
  • Re: Review Request 65648: Do not reschedule a PARTITIONED task if it was in KILLING state Jordan Ly
• Review Request 65649: Adding support for a Thrift JSON request which defines UTF-8 as the charset for the Content-Type in the Request Headers Renan DelValle
  • Re: Review Request 65649: Adding support for a Thrift JSON request which defines UTF-8 as the charset for the Content-Type in the Request Headers David McLaughlin
  • Re: Review Request 65649: Adding support for a Thrift JSON request which defines UTF-8 as the charset for the Content-Type in the Request Headers Aurora ReviewBot
  • Re: Review Request 65649: Adding support for a Thrift JSON request which defines UTF-8 as the charset for the Content-Type in the Request Headers Jordan Ly
  • Re: Review Request 65649: Adding support for a Thrift JSON request which defines UTF-8 as the charset for the Content-Type in the Request Headers Stephan Erb
• Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Stephan Erb
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Stephan Erb
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Aurora ReviewBot
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Jordan Ly
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Renan DelValle
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Stephan Erb
  • Re: Review Request 65598: Disable pytest-fast mode as a workaround for failing health checker tests Renan DelValle
• Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Renan DelValle
  • Re: Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Aurora ReviewBot
  • Re: Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Stephan Erb
  • Re: Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Renan DelValle
  • Re: Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Aurora ReviewBot
  • Re: Review Request 65565: Vagrant image upgrade to Ubuntu Xenial Jordan Ly
• Review Request 65537: Different approach to overlapping tables David McLaughlin
  • Re: Review Request 65537: Use overflow to prevent overlapping config summary tables David McLaughlin
  • Re: Review Request 65537: Use overflow to prevent overlapping config summary tables Jordan Ly
  • Re: Review Request 65537: Use overflow to prevent overlapping config summary tables Aurora ReviewBot
• Review Request 65501: Show cron job preview when no active tasks. David McLaughlin
  • Re: Review Request 65501: Show cron job preview when no active tasks. Aurora ReviewBot
  • Re: Review Request 65501: Show cron job preview when no active tasks. Santhosh Kumar Shanmugham
  • Re: Review Request 65501: Show cron job preview when no active tasks. Santhosh Kumar Shanmugham
  • Re: Review Request 65501: Show cron job preview when no active tasks. Reza Motamedi
  • Re: Review Request 65501: Show cron job preview when no active tasks. Aurora ReviewBot
• Review Request 65494: Add task page to Scheduler UI David McLaughlin
  • Re: Review Request 65494: Add task page to Scheduler UI Aurora ReviewBot
  • Re: Review Request 65494: Add task page to Scheduler UI Kai Huang

• Earlier messages
• Later messages