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


Ship it!




Ship It!

- Sebastian Toader


On Feb. 10, 2017, 3:21 p.m., Attila Doroszlai wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56545/
> -----------------------------------------------------------
> 
> (Updated Feb. 10, 2017, 3:21 p.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Sandor Magyari, and Sebastian 
> Toader.
> 
> 
> Bugs: AMBARI-19959
>     https://issues.apache.org/jira/browse/AMBARI-19959
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Added `--skip-view-extraction` flag, defaults to false.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/python/ambari-server.py 
> 5d4f5d7b66f1e34061a3b5d1eddfe3cf931bd23f 
>   ambari-server/src/main/python/ambari_server/serverSetup.py 
> 8a06034933b44cc9070b2d031dddad3505280c98 
>   ambari-server/src/test/python/TestAmbariServer.py 
> 7dae7c0af5211d180294a867743445aba2f105ac 
> 
> Diff: https://reviews.apache.org/r/56545/diff/
> 
> 
> Testing
> -------
> 
> Manual tested:
> 
> ```
> # ambari-server setup --silent
> [...]
> Creating tables...
> done.
> Extracting system views...
> ...ambari-admin-3.0.0.0.255.jar
> ......
> Adjusting ambari-server permissions and ownership...
> Ambari Server 'setup' completed successfully.
> 
> # ambari-server setup --silent --skip-view-extraction
> [...]
> Creating tables...
> done.
> Adjusting ambari-server permissions and ownership...
> Ambari Server 'setup' completed successfully.
> ```
> 
> 
> Unit tests:
> 
> ```
> $ mvn -am -pl ambari-server -DskipSurefireTests -DfailIfNoTests=false clean 
> test
> ...
> Total run:1183
> Total errors:0
> Total failures:0
> OK
> ...
> [INFO] Ambari Server ..................................... SUCCESS
> ```
> 
> 
> Thanks,
> 
> Attila Doroszlai
> 
>

Reply via email to