David Mollitor created HIVE-24772:
-------------------------------------

             Summary: Revamp Server Error Logging
                 Key: HIVE-24772
                 URL: https://issues.apache.org/jira/browse/HIVE-24772
             Project: Hive
          Issue Type: Improvement
            Reporter: David Mollitor
            Assignee: David Mollitor


Most of the action takes place in {{ThriftCLIService}} where errors are logged 
in response to client requests (though I know in many instances things are 
logged multiple times).

I propose to improve this on multiple fronts:

# Many log messages have the word "Error" in it, but log at the WARN level.  I 
have changed all relevant logging to be at ERROR level and removed the word 
"Error" from the message
# Some of the error message in the logging code had copy & paste errors where 
they printed the wrong request name
# Print the actual request object in the error message
# Big one for me: Do not pass a stack trace to the client.  This is bad 
practice from a security perspective,... clients should not know that level of 
detail of the server, and also it's very confusing for the client perspective 
to understand that the stack trace is actually from the remote server, not the 
local client, and finally, it's too messy for a typical user to deal with 
anyway.  Stack trace should be presented in the HS2 logs only.
# Various clean up
# Log an IP address for the client as part of standard operating procedure



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

Reply via email to