ambari-dev mailing list archives

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


> On Sept. 11, 2014, 9:36 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/python/ambari-server.py, line 192
> > <https://reviews.apache.org/r/25553/diff/1/?file=686511#file686511line192>
> >
> >     If this class is moved, things are going to break.  Hmmnnn, is there a good
way to handle this?

Thanks for reviewing.  

True, but the same also applies to every other java command line that we run from ambari-server.py
including "org.apache.ambari.server.controller.AmbariServer".  I think that it's okay to say
that the script has to be in sync with the classes that it's shipped with.  If the class package
changes then so should the script for that release.


- Tom


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


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