[ 
https://issues.apache.org/jira/browse/PHOENIX-4835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16572111#comment-16572111
 ] 

Ankit Singhal commented on PHOENIX-4835:
----------------------------------------

bq. We should move it to {{close()}} method instead.
Not sure about the current definition of mutation metrics but are the mutation 
metrics relevant if the corresponding mutations are never committed? 

> LoggingPhoenixConnection should log metrics upon connection close
> -----------------------------------------------------------------
>
>                 Key: PHOENIX-4835
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-4835
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Karan Mehta
>            Priority: Major
>         Attachments: PHOENIX-4835.4.x-HBase-1.4.001.patch
>
>
> {{LoggingPhoenixConnection}} currently logs metrics upon {{commit()}}, which 
> may miss the logging of metrics sometimes if commit is never called. We 
> should move it to {{close()}} method instead.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to