----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65075/ -----------------------------------------------------------
(Updated Jan. 23, 2018, 8:27 p.m.) Review request for hive, Aihua Xu, Andrew Sherman, and Sergio Pena. Changes ------- Addressed comments by adding new test to verify HushableRandomAccessFileAppender. 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/3/ Changes: https://reviews.apache.org/r/65075/diff/2-3/ Testing ------- Made sure that the new tests updated to verify this change are passing. Thanks, kalyan kumar kalvagadda