Re: Review Request 71784: HiveProtoLoggingHook might consume lots of memory

2019-11-27 Thread Harish Jaiprakash via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71784/#review218841 --- Ship it! LGTM. - Harish Jaiprakash On Nov. 21, 2019, 3:10

Re: Review Request 71784: HiveProtoLoggingHook might consume lots of memory

2019-11-20 Thread Harish Jaiprakash via Review Board
> On Nov. 20, 2019, 7:32 a.m., Harish Jaiprakash wrote: > > ql/src/test/org/apache/hadoop/hive/ql/hooks/TestHiveProtoLoggingHook.java > > Lines 176 (patched) > > > > > > We expect the dequeue to have not happened by