sanjeet006py commented on code in PR #2061:
URL: https://github.com/apache/phoenix/pull/2061#discussion_r1934289075
##########
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:
> Should the test confirm that those are the only metrics present?
Sure, I can add this assertion also. Thanks
--
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
