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

Review request for Ambari, Sumit Mohanty and Swapan Shridhar.


Bugs: AMBARI-18866
    https://issues.apache.org/jira/browse/AMBARI-18866


Repository: ambari


Description
-------

Applied logic to rolling over during creation of recommendations artifacts 
greater than a certain set limit.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorHelper.java
 e175c69 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ComponentLayoutRecommendationCommand.java
 87725dc 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ComponentLayoutValidationCommand.java
 310046f 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationDependenciesRecommendationCommand.java
 dd2a7d2 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationRecommendationCommand.java
 e6279c4 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationValidationCommand.java
 60132a1 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java
 6f612af 
  
ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java
 6cc4120 
  
ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationRecommendationCommandTest.java
 172fc6c 
  
ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java
 25d634b 

Diff: https://reviews.apache.org/r/53684/diff/


Testing
-------

1. Set the recommendations.artifacts.rollovermax = 5
Expected : Calls to stack advisor > 5 roll over starting from 0 again. Hence at 
any point of time there will be upto 5 folders only in 
/var/run/ambari-server/stack-recommendations
2. Do not set this value
Expected : Defaults to 100
3. Set the recommendations.artifacts.rollovermax = 0
Expected : Defaults to 100

On ambari-server restart, the ids are reset as before.


Thanks,

Madhuvanthi Radhakrishnan

Reply via email to