[jira] [Work logged] (HIVE-25143) Improve ERROR Logging in QL Package

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HIVE-25143?focusedWorklogId=710196&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-710196 ]

ASF GitHub Bot logged work on HIVE-25143:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 18/Jan/22 00:13
            Start Date: 18/Jan/22 00:13
    Worklog Time Spent: 10m 
      Work Description: github-actions[bot] commented on pull request #2301:
URL: https://github.com/apache/hive/pull/2301#issuecomment-1014967189


   This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
   Feel free to reach out on the dev@hive.apache.org list if the patch is in need of reviews.


-- 
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: gitbox-unsubscribe@hive.apache.org

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


Issue Time Tracking
-------------------

    Worklog Id:     (was: 710196)
    Time Spent: 40m  (was: 0.5h)

> Improve ERROR Logging in QL Package
> -----------------------------------
>
>                 Key: HIVE-25143
>                 URL: https://issues.apache.org/jira/browse/HIVE-25143
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> I went through and reviewed all of the ERROR logging in the HS2 {{ql}} module and I removed (most of) the following bad habits:
>  
>  * Log-and-Throw (log or throw, not both)
>  * Pass in the Exception to the logging framework instead of logging its toString() : LOG.error("alter table update columns: {}", e);
>  * Add additional context instead of copying the message from the wrapped Exception : throw new SemanticException(e.getMessage(), e);
>  * The wrapped exception is being lost in some case, though the message survives :  throw new HiveException(e.getMessage());
>  * Remove new-lines from Exception messages, this is annoying as log messages should all be on a single line for GREP
>  * Not logging the Exception stack trace :  LOG.error("Error in close loader: " + ie);
>  * Logging information but not passing it into an Exception for bubbling up:  LOG.error("Failed to return session: {} to pool", session, e); throw e;
>  * Other miscellaneous improvements



--
This message was sent by Atlassian Jira
(v8.20.1#820001)