ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Levas" <rle...@hortonworks.com>
Subject Re: Review Request 37135: Move repeated log statement to DEBUG level
Date Thu, 06 Aug 2015 14:41:01 GMT

-----------------------------------------------------------
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
> 
>


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