-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65075/
-----------------------------------------------------------

(Updated Jan. 24, 2018, 5:44 p.m.)


Review request for hive, Aihua Xu, Andrew Sherman, and Sergio Pena.


Changes
-------

Added code changes to addess the codestyle errors


Bugs: HIVE-18426
    https://issues.apache.org/jira/browse/HIVE-18426


Repository: hive-git


Description
-------

Each new operation creates new entry in the ConcurrentMap in RoutingAppender 
but when the operation ends, AppenderControl stored in the map is retrieved and 
stopped but the entry in ConcurrentMap is never cleaned up.


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/LogUtils.java 
5c7ec69f04587ab18704a10fade6e3a47601a87f 
  
itests/hive-unit/src/test/java/org/apache/hive/service/cli/operation/TestOperationLoggingLayout.java
 8febe3e79ff892c54b696b6c6ef92f7026c46033 


Diff: https://reviews.apache.org/r/65075/diff/4/

Changes: https://reviews.apache.org/r/65075/diff/3-4/


Testing
-------

Made sure that the new tests updated to verify this change are passing.


Thanks,

kalyan kumar kalvagadda

Reply via email to