sankarh commented on a change in pull request #537: HIVE-19430 : ObjectStore.cleanNotificationEvents OutOfMemory on large number of pending events URL: https://github.com/apache/hive/pull/537#discussion_r257201191
########## File path: hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java ########## @@ -1142,10 +1142,8 @@ public void run() { } catch (Exception ex) { //catching exceptions here makes sure that the thread doesn't die in case of unexpected //exceptions - LOG.warn( - "Exception received while cleaning notifications. More details can be found in debug mode" - + ex.getMessage()); - LOG.debug(ex.getMessage(), ex); Review comment: Why this debug log is removed? Not sure, if it's there for any security issues. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services