[GitHub] phoenix pull request #338: PHOENIX-4870 LoggingPhoenixConnection should log ...

[karanmehta93]

Github user karanmehta93 commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/338#discussion_r214199456
  
    --- Diff: 
phoenix-core/src/it/java/org/apache/phoenix/monitoring/PhoenixLoggingMetricsIT.java
 ---
    @@ -154,6 +154,35 @@ private void verifyQueryLevelMetricsLogging(String 
query) throws SQLException {
             assertTrue(logRequestReadMetricsFuncCallCount == 1);
         }
     
    +    @Test
    +    public void testPhoenixMetricsLoggedOnAutoCommit() throws Exception {
    --- End diff --
    
    nit: `testPhoenixMetricsLoggedOnAutoCommit` to 
`testPhoenixMetricsLoggedOnAutoCommitTrue`


---