Re: Review Request 51046: Support explain analyze in Hive

2016-08-30 Thread pengcheng xiong
> On Aug. 26, 2016, 11:44 p.m., Gabor Szadovszky wrote: > > ql/src/java/org/apache/hadoop/hive/ql/hooks/ATSHook.java, line 132 > > > > > > nit: Seems to break the previous commenting structure by adding the two > >

Re: Review Request 51046: Support explain analyze in Hive

2016-08-26 Thread Gabor Szadovszky
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/#review147033 --- Some nits and minor findings. LGTM, otherwise. Thanks for the patc

Re: Review Request 51046: Support explain analyze in Hive

2016-08-26 Thread pengcheng xiong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/ --- (Updated Aug. 26, 2016, 8:28 p.m.) Review request for hive and Ashutosh Chauhan

Re: Review Request 51046: Support explain analyze in Hive

2016-08-26 Thread pengcheng xiong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/ --- (Updated Aug. 26, 2016, 6:56 p.m.) Review request for hive and Ashutosh Chauhan

Re: Review Request 51046: Support explain analyze in Hive

2016-08-24 Thread pengcheng xiong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/ --- (Updated Aug. 24, 2016, 5:02 p.m.) Review request for hive and Ashutosh Chauhan

Re: Review Request 51046: Support explain analyze in Hive

2016-08-19 Thread pengcheng xiong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/ --- (Updated Aug. 19, 2016, 11:58 p.m.) Review request for hive and Ashutosh Chauha

Review Request 51046: Support explain analyze in Hive

2016-08-12 Thread pengcheng xiong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51046/ --- Review request for hive and Ashutosh Chauhan. Repository: hive-git Descriptio