On Sept. 16, 2014, 6:42 a.m., Thejas Nair wrote:
ql/src/java/org/apache/hadoop/hive/ql/Driver.java, line 741
https://reviews.apache.org/r/25616/diff/2/?file=690379#file690379line741
should we skip it from ReadEntity if none of the columns are being used
? Though, that case is not
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25616/
---
(Updated Sept. 16, 2014, 7:37 p.m.)
Review request for hive and Thejas Nair.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25616/#review53580
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25616/#review53593
---
Ship it!
Ship It!
- Thejas Nair
On Sept. 16, 2014, 7:37 p.m.,
On Sept. 16, 2014, 6:42 a.m., Thejas Nair wrote:
ql/src/java/org/apache/hadoop/hive/ql/Driver.java, line 741
https://reviews.apache.org/r/25616/diff/2/?file=690379#file690379line741
should we skip it from ReadEntity if none of the columns are being used
? Though, that case is not
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25616/#review53316
---
On Sept. 15, 2014, 7:24 a.m., Thejas Nair wrote:
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/authorization/plugin/TestHiveAuthorizerCheckInvocation.java,
line 272
https://reviews.apache.org/r/25616/diff/1/?file=688987#file688987line272
Wouldn't select
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25616/#review53277
---
On Sept. 14, 2014, 7:13 a.m., Thejas Nair wrote:
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/authorization/plugin/TestHiveAuthorizerCheckInvocation.java,
line 272
https://reviews.apache.org/r/25616/diff/1/?file=688987#file688987line272
It would be good to