ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Fernandez" <afernan...@hortonworks.com>
Subject Re: Review Request 25553: Views : Extract System view on ambari-server setup
Date Thu, 11 Sep 2014 21:36:22 GMT

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



ambari-server/src/main/python/ambari-server.py
<https://reviews.apache.org/r/25553/#comment92531>

    If this class is moved, things are going to break.  Hmmnnn, is there a good way to handle
this?


- Alejandro Fernandez


On Sept. 11, 2014, 8:49 p.m., Tom Beerbower wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25553/
> -----------------------------------------------------------
> 
> (Updated Sept. 11, 2014, 8:49 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Nate Cole.
> 
> 
> Bugs: ambari-7227
>     https://issues.apache.org/jira/browse/ambari-7227
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Views marked as system views should be extracted as part of the 'ambari-server setup'
command.  This means that the system views (along with any other views that are already extracted)
can be loaded immediately.  Views not already extracted on startup will be loaded in the backgroud.
> 
> I think this should occur near the end of setup (i.e. after the database step -- the
final step before completing setup).
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/view/ViewArchiveUtility.java PRE-CREATION

>   ambari-server/src/main/java/org/apache/ambari/server/view/ViewExtractor.java PRE-CREATION

>   ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java 509e474

>   ambari-server/src/main/python/ambari-server.py 35ba17b 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AmbariPrivilegeResourceProviderTest.java
f6a49cf 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ViewPrivilegeResourceProviderTest.java
eb4ef9c 
>   ambari-server/src/test/java/org/apache/ambari/server/view/ViewExtractorTest.java PRE-CREATION

>   ambari-server/src/test/java/org/apache/ambari/server/view/ViewRegistryTest.java 38c2f9b

>   ambari-server/src/test/python/TestAmbariServer.py 53f0ba1 
> 
> Diff: https://reviews.apache.org/r/25553/diff/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> Unit tests added.  All existing tests pass.
> 
> Results :
> 
> Tests run: 1979, Failures: 0, Errors: 0, Skipped: 16
> 
> …
> 
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 40:18.051s
> [INFO] Finished at: Thu Sep 11 14:58:09 EDT 2014
> [INFO] Final Memory: 28M/201M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message