[jira] [Commented] (DRILL-7764) Cleanup warning messages in GuavaPatcher class

[ASF GitHub Bot (Jira)]

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

ASF GitHub Bot commented on DRILL-7764:
---------------------------------------

KazydubB opened a new pull request #2093:
URL: https://github.com/apache/drill/pull/2093


   # [DRILL-7764](https://issues.apache.org/jira/browse/DRILL-7764): Cleanup 
warning messages in GuavaPatcher class
   
   ## Description
   
   Removed unnecessary logging of stack trace (on `WARN` level) when patching 
Guava fails.
   
   ## Documentation
   N/A
   
   ## Testing
   Ran all existing tests.
   


----------------------------------------------------------------
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.

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


> Cleanup warning messages in GuavaPatcher class
> ----------------------------------------------
>
>                 Key: DRILL-7764
>                 URL: https://issues.apache.org/jira/browse/DRILL-7764
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Bohdan Kazydub
>            Assignee: Bohdan Kazydub
>            Priority: Minor
>
> Currently GuavaPatcher contains
> {code}
> logger.warn("Unable to patch Guava classes.", e);
> {code}
> which outputs whole exception stack trace to logs which is unnecessary 
> alarming.
> This log message will be changed to 
> {code}
> logger.warn("Unable to patch Guava classes: {}", e.getMessage());
> logger.debug("Exception:", e);
> {code}
> logging the stack trace only in debug mode.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)