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


Ship it!




Minor changes in the comments - fix it and then ship it.

We can probably add more details in the logs - but that can be done later as 
and when needed.

- Sumit Mohanty


On Aug. 30, 2016, 12:27 p.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51522/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2016, 12:27 p.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, 
> and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-18273
>     https://issues.apache.org/jira/browse/AMBARI-18273
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Add ambari server python side logging..
> 
> 
> Diffs
> -----
> 
>   ambari-server/conf/unix/ambari.properties b8e41b7 
>   ambari-server/sbin/ambari-server 8064c45 
>   ambari-server/src/main/python/ambari-server.py f929042 
>   ambari-server/src/main/python/ambari_server/checkDatabase.py 831efde 
>   ambari-server/src/main/python/ambari_server/dbCleanup.py 7a5486c 
>   ambari-server/src/main/python/ambari_server/hostUpdate.py d5ab22a 
>   ambari-server/src/main/python/ambari_server/serverSetup.py d80c25a 
>   ambari-server/src/main/python/ambari_server/serverUpgrade.py d3cb3ba 
>   ambari-server/src/main/python/ambari_server/setupActions.py 182c11e 
>   ambari-server/src/main/python/ambari_server/setupMpacks.py f854d76 
>   ambari-server/src/main/python/ambari_server/setupSecurity.py 69579cd 
>   ambari-server/src/main/python/ambari_server/setupSso.py 2a5e44c 
>   ambari-server/src/main/python/ambari_server_main.py 17945de 
>   ambari-server/src/test/python/TestAmbariServer.py a45a4bd 
> 
> Diff: https://reviews.apache.org/r/51522/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>

Reply via email to