d-c-manning commented on code in PR #2061:
URL: https://github.com/apache/phoenix/pull/2061#discussion_r1933335940


##########
phoenix-core/src/it/java/org/apache/phoenix/monitoring/PhoenixLoggingMetricsIT.java:
##########
@@ -156,9 +156,9 @@ public void testPhoenixMetricsLoggedOnClose() throws 
Exception {
         // Autocommit is turned off by default
         // Hence mutation metrics are not expected during connection close

Review Comment:
   this comment is still here though (are mutation metrics now expected during 
connection close?



##########
phoenix-core/src/it/java/org/apache/phoenix/monitoring/PhoenixLoggingMetricsIT.java:
##########
@@ -156,9 +156,9 @@ public void testPhoenixMetricsLoggedOnClose() throws 
Exception {
         // Autocommit is turned off by default
         // Hence mutation metrics are not expected during connection close

Review Comment:
   this line 157 comment is still here though (are mutation metrics now 
expected during connection close?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to