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

Ship it!


Ship It!

- Tom Beerbower


On Feb. 26, 2014, 3:24 p.m., Artem Baranchuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18517/
> -----------------------------------------------------------
> 
> (Updated Feb. 26, 2014, 3:24 p.m.)
> 
> 
> Review request for Ambari and Tom Beerbower.
> 
> 
> Bugs: AMBARI-4840
>     https://issues.apache.org/jira/browse/AMBARI-4840
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Metrics sink for both Hadoop1 and Hadoop2 versions
> 
> 
> Diffs
> -----
> 
>   contrib/ambari-scom/metrics-sink/pom.xml c005c2e 
>   
> contrib/ambari-scom/metrics-sink/src/Test/java/org/apache/hadoop/metrics2/sink/SqlServerSinkHadoop1Test.java
>  PRE-CREATION 
>   
> contrib/ambari-scom/metrics-sink/src/Test/java/org/apache/hadoop/metrics2/sink/SqlServerSinkHadoop2Test.java
>  PRE-CREATION 
>   
> contrib/ambari-scom/metrics-sink/src/Test/java/org/apache/hadoop/metrics2/sink/SqlServerSinkTest.java
>  4d4f6dd 
>   
> contrib/ambari-scom/metrics-sink/src/main/java/org/apache/hadoop/metrics2/sink/SqlServerSink.java
>  034876a 
>   
> contrib/ambari-scom/metrics-sink/src/main/java/org/apache/hadoop/metrics2/sink/SqlServerSinkHadoop1.java
>  PRE-CREATION 
>   
> contrib/ambari-scom/metrics-sink/src/main/java/org/apache/hadoop/metrics2/sink/SqlServerSinkHadoop2.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/18517/diff/
> 
> 
> Testing
> -------
> 
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> Running org.apache.hadoop.metrics2.sink.SqlServerSinkHadoop1Test
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Entering init
> 14/02/26 17:22:49 INFO sink.SqlServerSink: Exit init, cluster name = localhost
> Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.809 sec
> 
> Results :
> 
> Tests run: 7, Failures: 0, Errors: 0, Skipped: 0
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> Running org.apache.hadoop.metrics2.sink.SqlServerSinkHadoop2Test
> log4j:WARN No appenders could be found for logger 
> (org.apache.hadoop.metrics2.sink.SqlServerSink).
> log4j:WARN Please initialize the log4j system properly.
> log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
> info.
> Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.921 sec
> 
> Results :
> 
> Tests run: 7, Failures: 0, Errors: 0, Skipped: 0
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Artem Baranchuk
> 
>

Reply via email to