[
https://issues.apache.org/jira/browse/PHOENIX-5537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17034848#comment-17034848
]
Daniel Wong commented on PHOENIX-5537:
--------------------------------------
[~RichardAntal] In general the changes look okay to me but I would like to see
a unit test or integration test to go with the patch. I'll see if i can get
this to patch submitted and have our HadoopQA run our tests against this.
> Phoenix-4701 made hard coupling between phoenix.log.level and getting request
> metrics.
> --------------------------------------------------------------------------------------
>
> Key: PHOENIX-5537
> URL: https://issues.apache.org/jira/browse/PHOENIX-5537
> Project: Phoenix
> Issue Type: Bug
> Affects Versions: 4.15.0
> Reporter: Daniel Wong
> Priority: Minor
> Fix For: 4.15.1
>
> Attachments: PHOENIX-5537.master.v1.patch
>
>
> Phoenix-4701 made hard coupling between phoenix.log.level and getting request
> metrics. For users who do not want to enable system to log this causes a
> regression from earlier behavior where metrics were populated. FYI [~ankit]
> {code:java}
> public OverAllQueryMetrics(boolean isRequestMetricsEnabled, LogLevel
> connectionLogLevel) { public OverAllQueryMetrics(boolean
> isRequestMetricsEnabled, LogLevel connectionLogLevel) { queryWatch = new
> MetricsStopWatch(WALL_CLOCK_TIME_MS.isLoggingEnabled(connectionLogLevel));
> {code}
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
