Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/ --- (Updated Jan. 19, 2017, 12:33 a.m.) Review request for hive. Changes ---

Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/ --- (Updated Jan. 10, 2017, 9:09 p.m.) Review request for hive. Changes ---

Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/#review160691 --- Ship it! Thank you for the update. I have no more nitpicks. -

Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/ --- (Updated Jan. 5, 2017, 6:45 p.m.) Review request for hive. Changes ---

Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

> On Jan. 5, 2017, 9:38 a.m., Barna Zsombor Klara wrote: > > ql/src/java/org/apache/hadoop/hive/ql/hooks/ATSHook.java, line 208 > > > > > > Should this be logged at INFO lvl? I understand that the hive query can >

Re: Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/#review160571 --- ql/src/java/org/apache/hadoop/hive/ql/hooks/ATSHook.java (line

Review Request 55194: HIVE-15541: Hive OOM when ATSHook enabled and ATS goes down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55194/ --- Review request for hive. Bugs: HIVE-15541