> On May 16, 2016, 9:47 p.m., Sid Wagle wrote:
> > What about RCA tables not a part of Ambari DB ?
> 
> Ajit Kumar wrote:
>     Only postgres-embedded-create.sql has ambarirca db. Users who are facing 
> this issue have stand alone DB server and don't use postgres-embedded. 
> Ideally we should remove ambarirca db from postgres-embedded-create.sql as 
> well.

We should it as a part of this patch IMO. The rca tables are no longer used.
Prior to 1.5.1 we had separate RCA database on the same database server. We 
should in all likelyhood cleaning that up as well, no? ambari.properties has 
the hdbc URL for it.


- Sid


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


On May 16, 2016, 9:28 p.m., Ajit Kumar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43126/
> -----------------------------------------------------------
> 
> (Updated May 16, 2016, 9:28 p.m.)
> 
> 
> Review request for Ambari, Jayush Luniya, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-16692
>     https://issues.apache.org/jira/browse/AMBARI-16692
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> RCA Database tables is not used anymore. This older, un-necessary data causes 
> many Ambari DB queries to return more slowly than necessary. If the database 
> gets large enough, it can have significant impact on the way ambari-server 
> functions.
> The Ambari Server upgrade process should detect the presence of these older, 
> deprecated tables, and automate the process of cleaning up this data.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java
>  3547ad3 
>   
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java
>  f36e640 
> 
> Diff: https://reviews.apache.org/r/43126/diff/
> 
> 
> Testing
> -------
> 
> Unit test
> 
> 
> Thanks,
> 
> Ajit Kumar
> 
>

Reply via email to