Re: Review Request 63427: HIVE-17396

2017-11-02 Thread Deepak Jaiswal
> On Nov. 1, 2017, 3:48 a.m., Lefty Leverenz wrote: > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > > Lines 3017-3018 (patched) > > > > > > Please spell out the acronyms TS, DPP, and nDVs in the

Re: Review Request 63427: HIVE-17396

2017-11-02 Thread Lefty Leverenz
> On Nov. 1, 2017, 3:48 a.m., Lefty Leverenz wrote: > > Nit: This RB summary has the wrong JIRA number -- it should be 17936, not 17396. But the first line of the description has the correct JIRA text, so it's easy enough to find with a search. - Lefty

Re: Review Request 63427: HIVE-17396

2017-11-02 Thread Lefty Leverenz
> On Nov. 1, 2017, 3:48 a.m., Lefty Leverenz wrote: > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > > Lines 3017-3018 (patched) > > > > > > Please spell out the acronyms TS, DPP, and nDVs in the

Re: Review Request 63427: HIVE-17396

2017-11-01 Thread Deepak Jaiswal
> On Nov. 1, 2017, 3:48 a.m., Lefty Leverenz wrote: > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > > Lines 3017-3018 (patched) > > > > > > Please spell out the acronyms TS, DPP, and nDVs in the

Re: Review Request 63427: HIVE-17396

2017-10-31 Thread Lefty Leverenz
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63427/#review189778 --- common/src/java/org/apache/hadoop/hive/conf/HiveConf.java Lines

Re: Review Request 63427: HIVE-17396

2017-10-30 Thread Deepak Jaiswal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63427/ --- (Updated Oct. 31, 2017, 1:19 a.m.) Review request for hive, Ashutosh Chauhan

Re: Review Request 63427: HIVE-17396

2017-10-30 Thread Deepak Jaiswal
> On Oct. 30, 2017, 11:19 p.m., Jason Dere wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java > > Line 1417 (original), 1417 (patched) > > > > > > Trying to think about how to think about this

Re: Review Request 63427: HIVE-17396

2017-10-30 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63427/#review189657 --- ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java

Review Request 63427: HIVE-17396

2017-10-30 Thread Deepak Jaiswal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63427/ --- Review request for hive and Jason Dere. Repository: hive-git Description