[ 
https://issues.apache.org/jira/browse/PHOENIX-4834?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Karan Mehta updated PHOENIX-4834:
---------------------------------
    Description: 
{{PhoenixMetricsLog}} is an interface that provides a wrapper around various 
JDBC objects with logging functionality upon close/commit. The methods take in 
{{Logger}} as an input, which is {{org.slf4j.Logger}}. A better way of doing is 
that the interface should just pass the metrics and allow the user to configure 
and use whatever logging library they want to use.

This Jira will deprecate the older methods by provide a default implementation 
for them and add the new methods.

Ideally we should have provided default implementations, but since we are on 
Java 7, we are unable to do that.

  was:
{{PhoenixMetricsLog}} is an interface that provides a wrapper around various 
JDBC objects with logging functionality upon close/commit. The methods take in 
{{Logger}} as an input, which is {{org.slf4j.Logger}}. A better way of doing is 
that the interface should just pass the metrics and allow the user to configure 
and use whatever logging library they want to use.

This Jira will deprecate the older methods by provide a default implementation 
for them and add the new methods.


> PhoenixMetricsLog interface methods should not depend on specific logger
> ------------------------------------------------------------------------
>
>                 Key: PHOENIX-4834
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-4834
>             Project: Phoenix
>          Issue Type: Improvement
>    Affects Versions: 4.15.0
>            Reporter: Karan Mehta
>            Assignee: Karan Mehta
>            Priority: Major
>             Fix For: 4.15.0
>
>         Attachments: PHOENIX-4834.001.patch
>
>
> {{PhoenixMetricsLog}} is an interface that provides a wrapper around various 
> JDBC objects with logging functionality upon close/commit. The methods take 
> in {{Logger}} as an input, which is {{org.slf4j.Logger}}. A better way of 
> doing is that the interface should just pass the metrics and allow the user 
> to configure and use whatever logging library they want to use.
> This Jira will deprecate the older methods by provide a default 
> implementation for them and add the new methods.
> Ideally we should have provided default implementations, but since we are on 
> Java 7, we are unable to do that.



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

Reply via email to