-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66136/#review199421
-----------------------------------------------------------



@ReviewBot retry

- Stephan Erb


On March 19, 2018, 3:55 p.m., Stephan Erb wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66136/
> -----------------------------------------------------------
> 
> (Updated March 19, 2018, 3:55 p.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Renan DelValle.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This is the first part of a small series of Thermos observer performance 
> improvements.
> 
> As a first iteration, this switches all logging to use the logger-embedded 
> formatting
> rather than doing it eager up front. This has the advantage that we produce 
> less garbage
> if debug logging is disabled.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/thermos/common/ckpt.py 
> e79ec6ac42483bfab4ccd94390d0f2af95903a95 
>   src/main/python/apache/thermos/core/helper.py 
> 0811e846ce8e39a22d55732604876c37d2caa239 
>   src/main/python/apache/thermos/core/muxer.py 
> 47e77f7685891545b2526e71e016ffcd454f938c 
>   src/main/python/apache/thermos/core/process.py 
> 4a4678ff39c84cb87836aca19365c5b2aabc4fa4 
>   src/main/python/apache/thermos/core/runner.py 
> 1b63c083246f8ec1a43cb1686968c2b1bb4a14b5 
>   src/main/python/apache/thermos/monitoring/disk.py 
> 52c5d74fd70b5942ea3ef5101ba3f27bfc98fc21 
>   src/main/python/apache/thermos/monitoring/monitor.py 
> d77703e9027e6c23ffb67d936cf8beef0384b4a6 
>   src/main/python/apache/thermos/monitoring/process_collector_psutil.py 
> 3000e95e2930a01f57ac855960b5db7aabbfc0ca 
>   src/main/python/apache/thermos/monitoring/resource.py 
> f5e3849ca6682c6d4720698be869ca6b9f703b94 
>   src/main/python/apache/thermos/observer/http/file_browser.py 
> d0996651312a763ed68a68ae358524cfb4a94d29 
>   src/main/python/apache/thermos/observer/http/http_observer.py 
> 5bfc4f28bfbbcb654a059a3579844f5dbe082a9b 
>   src/main/python/apache/thermos/observer/http/static_assets.py 
> 83adeb3e699274483cb72910b0d25f7b30098f24 
>   src/main/python/apache/thermos/observer/observed_task.py 
> 08540e11523b0db990de57fe1ce7517047e2632b 
>   src/main/python/apache/thermos/observer/task_observer.py 
> 4bb5d239e81fe4659397f899760c0e8853e93786 
>   src/main/python/apache/thermos/runner/thermos_runner.py 
> fa4f0fb7452bf8b85f9f3fa27283f54d7c0fe4f2 
> 
> 
> Diff: https://reviews.apache.org/r/66136/diff/1/
> 
> 
> Testing
> -------
> 
> Manually verified that the observer debug logs still contain useful output.
> 
> 
> Thanks,
> 
> Stephan Erb
> 
>

Reply via email to