----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37135/#review94402 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On Aug. 5, 2015, 2:56 p.m., Robert Nettleton wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37135/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2015, 2:56 p.m.) > > > Review request for Ambari, Mahadev Konar, Robert Levas, and Sumit Mohanty. > > > Bugs: AMBARI-12652 > https://issues.apache.org/jira/browse/AMBARI-12652 > > > Repository: ambari > > > Description > ------- > > 1. This patch resolves AMBARI-12652. > 2. A repeated log message added by a recent checkin is not necessary for > normal operation or debugging, and so makes debugging a little more > difficult. > > This patch addresses this problem by moving this log message to the DEBUG > level. This log is useful when debugging large cluster deployments and > concurrency issues, but in general is not necessary for normal operations. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java > 9e18fec > > Diff: https://reviews.apache.org/r/37135/diff/ > > > Testing > ------- > > 1. Ran ambari-server's "mvn clean test" suite, which passed. > 2. Deployed a 3-node HDFS HA cluster, using a Blueprint, with this patch > applied. Verified that the cluster deployed successfully, and that the log > message no longer appears by default during a Blueprint deployment. > > > Thanks, > > Robert Nettleton > >
