/TestHiveSQLException.java
PRE-CREATION
shims/0.20S/src/main/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java
e6493eb
Diff: https://reviews.apache.org/r/21922/diff/
Testing
---
Thanks,
Navis Ryu
/
Testing
---
Thanks,
Navis Ryu
/java/org/apache/hadoop/hive/ql/plan/TableDesc.java 39f1793
Diff: https://reviews.apache.org/r/21903/diff/
Testing
---
Thanks,
Navis Ryu
---
On May 23, 2014, 2:14 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21841
://reviews.apache.org/r/21886/diff/
Testing
---
Thanks,
Navis Ryu
---
On April 2, 2014, 9:07 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16034/
---
(Updated April 2
/AuthorizationFactory.java
PRE-CREATION
ql/src/test/queries/clientpositive/authorization_explain.q PRE-CREATION
ql/src/test/results/clientpositive/authorization_explain.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/16034/diff/
Testing
---
Thanks,
Navis Ryu
34224e0
Diff: https://reviews.apache.org/r/20399/diff/
Testing
---
Thanks,
Navis Ryu
://reviews.apache.org/r/21834/diff/
Testing
---
Thanks,
Navis Ryu
/ql/session/SessionState.java dab8610
ql/src/test/org/apache/hadoop/hive/ql/session/TestSessionState.java 7df7ecc
Diff: https://reviews.apache.org/r/21841/diff/
Testing
---
Thanks,
Navis Ryu
/results/clientpositive/show_create_table_alter.q.out c4baa5b
ql/src/test/results/clientpositive/show_tblproperties.q.out c60e710
ql/src/test/results/clientpositive/unset_table_view_property.q.out f4f1e91
Diff: https://reviews.apache.org/r/21748/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20399/#review42627
---
On May 7, 2014, 4:06 a.m., Navis Ryu wrote
/
Testing
---
Thanks,
Navis Ryu
e706f52
Diff: https://reviews.apache.org/r/21549/diff/
Testing
---
Thanks,
Navis Ryu
/hive/ql/session/LineageState.java e706f52
Diff: https://reviews.apache.org/r/21549/diff/
Testing
---
Thanks,
Navis Ryu
/column_access_stats.q.out 7eee4ba
Diff: https://reviews.apache.org/r/20399/diff/
Testing
---
Thanks,
Navis Ryu
/apache/hive/beeline/Commands.java 493f963
beeline/src/main/resources/BeeLine.properties 697c29a
Diff: https://reviews.apache.org/r/19984/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19984/#review40101
---
On April 8, 2014, 2:07 a.m., Navis Ryu wrote:
---
This is an automatically generated e
/results/clientpositive/column_access_stats.q.out 7eee4ba
Diff: https://reviews.apache.org/r/20399/diff/
Testing
---
Thanks,
Navis Ryu
/drop_partitions_partialspec.q.out
PRE-CREATION
Diff: https://reviews.apache.org/r/19903/diff/
Testing
---
Thanks,
Navis Ryu
---
On April 2, 2014, 6:28 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179
/apache/hive/beeline/Commands.java 493f963
beeline/src/main/resources/BeeLine.properties 697c29a
Diff: https://reviews.apache.org/r/19984/diff/
Testing
---
Thanks,
Navis Ryu
/drop_partitions_partialspec.q.out
PRE-CREATION
Diff: https://reviews.apache.org/r/19903/diff/
Testing
---
Thanks,
Navis Ryu
/
Testing
---
Thanks,
Navis Ryu
/authorization/DelegatableAuthorizationProvider.java
PRE-CREATION
ql/src/test/queries/clientpositive/authorization_explain.q PRE-CREATION
ql/src/test/results/clientpositive/authorization_explain.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/16034/diff/
Testing
---
Thanks,
Navis
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19903/#review39347
---
On April 2, 2014, 6:04 a.m., Navis Ryu wrote
/drop_partition_filter_failure.q.out
0ab5e02
ql/src/test/results/clientpositive/drop_multi_partitions.q.out 735920b
ql/src/test/results/clientpositive/drop_partitions_partialspec.q.out
PRE-CREATION
Diff: https://reviews.apache.org/r/19903/diff/
Testing
---
Thanks,
Navis Ryu
0.14.0-SNAPSHOT by Apache Hive
...
{code}
Diffs
-
beeline/src/java/org/apache/hive/beeline/BeeLine.java 5773109
beeline/src/main/resources/BeeLine.properties 697c29a
Diff: https://reviews.apache.org/r/19984/diff/
Testing
---
Thanks,
Navis Ryu
/apache/hive/service/cli/session/HiveSessionImpl.java
f730119
service/src/java/org/apache/hive/service/cli/session/SessionManager.java
7f6687e
Diff: https://reviews.apache.org/r/15449/diff/
Testing
---
Confirmed in the local environment.
Thanks,
Navis Ryu
-CREATION
Diff: https://reviews.apache.org/r/19865/diff/
Testing
---
Thanks,
Navis Ryu
---
On March 25, 2014, 8:02 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
,
Navis Ryu
/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19387/#review37796
---
On March 19, 2014, 1:36 a.m., Navis Ryu wrote
/TestGenericUDFOPNumeric.java
2ada2ff
service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java
ace791a
service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java
ef5b5c6
Diff: https://reviews.apache.org/r/19387/diff/
Testing
---
Thanks,
Navis Ryu
/results/clientnegative/external1.q.out 696beaa
ql/src/test/results/clientnegative/external2.q.out a604885
ql/src/test/results/clientnegative/insertexternal1.q.out 3df5013
Diff: https://reviews.apache.org/r/19344/diff/
Testing
---
Thanks,
Navis Ryu
://reviews.apache.org/r/19387/diff/
Testing
---
Thanks,
Navis Ryu
implementation here to HBaseCompositeKeyFactory? Serialize() seems
seems generic enough to expect from all key factories. Doing this will
eliminate HBaseWritableKeyFactory and use of the class to detect what
method to call.
Navis Ryu wrote:
If the default serialization can be done
---
On March 7, 2014, 7:46 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179
://reviews.apache.org/r/18179/#review36711
---
On March 7, 2014, 7:46 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179
---
On March 7, 2014, 7:46 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179
/java/org/apache/hadoop/hive/ql/DriverContext.java c51a9c8
ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java 854cd52
ql/src/java/org/apache/hadoop/hive/ql/exec/TaskRunner.java ead7b59
Diff: https://reviews.apache.org/r/15873/diff/
Testing
---
Thanks,
Navis Ryu
/apache/hadoop/hive/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
---
On March 7, 2014, 3:32 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18179/
---
(Updated March 7, 2014, 3:32 a.m
/serde2/lazybinary/LazyBinaryObject.java
598683f
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
collection classes are being used here, I
don't see other concurrency issues that would make it necessary to make all
these functions synchronized.
Navis Ryu wrote:
It just only polls status of running tasks and goes into wait state quite
quickly, so it would not hinder
/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
c7591a4
Diff: https://reviews.apache.org/r/16281/diff/
Testing
---
Thanks,
Navis Ryu
://reviews.apache.org/r/15873/#review35625
---
On Feb. 27, 2014, 7:06 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r
---
Thanks,
Navis Ryu
://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
://reviews.apache.org/r/18459/#comment66256
Current reset command resets all overridden confs ignoring following
literals. But this looks like it would be good to support specified only. You
can ignore this.
- Navis Ryu
On Feb. 26, 2014, 4:37 p.m., Ashutosh Chauhan wrote
---
On Dec. 30, 2013, 6:57 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15873
/java/org/apache/hadoop/hive/ql/DriverContext.java c51a9c8
ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java 854cd52
ql/src/java/org/apache/hadoop/hive/ql/exec/TaskRunner.java ead7b59
Diff: https://reviews.apache.org/r/15873/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15873/#review35371
---
On Dec. 30, 2013, 6:57 a.m., Navis Ryu wrote
868dd4c
serde/src/test/org/apache/hadoop/hive/serde2/lazybinary/TestLazyBinarySerDe.java
69c891d
Diff: https://reviews.apache.org/r/17652/diff/
Testing
---
Thanks,
Navis Ryu
., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17694/
---
(Updated Feb. 19, 2014, 7:05 a.m.)
Review request
-CREATION
ql/src/test/results/clientpositive/show_conf.q.out PRE-CREATION
service/src/java/org/apache/hive/service/cli/CLIService.java 56b357a
Diff: https://reviews.apache.org/r/17694/diff/
Testing
---
Thanks,
Navis Ryu
=4)
Closing: org.apache.hive.jdbc.HiveConnection
{code}
The same query works fine if I replace tabs with some spaces.
Diffs
-
beeline/src/java/org/apache/hive/beeline/BeeLine.java 563d242
Diff: https://reviews.apache.org/r/18256/diff/
Testing
---
Thanks,
Navis Ryu
nit. indent
- Navis Ryu
On Feb. 19, 2014, 12:28 a.m., Ashutosh Chauhan wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18250
On Feb. 19, 2014, 2:04 a.m., Navis Ryu wrote:
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java,
line 457
https://reviews.apache.org/r/18250/diff/1/?file=497250#file497250line457
should this be doesUserHasGrantOption
,
Navis Ryu
/mapjoin_test_outer.q.out 36deabb
ql/src/test/results/clientpositive/multiMapJoin2.q.out c59a407
ql/src/test/results/clientpositive/subquery_multiinsert.q.out fdafea3
Diff: https://reviews.apache.org/r/18177/diff/
Testing
---
Thanks,
Navis Ryu
/compiler/parse/union.q.out d9be739
Diff: https://reviews.apache.org/r/17887/diff/
Testing
---
Thanks,
Navis Ryu
f8de5fc
ql/src/test/results/compiler/parse/udf_when.q.out a527fa4
ql/src/test/results/compiler/parse/union.q.out d9be739
Diff: https://reviews.apache.org/r/17887/diff/
Testing
---
Thanks,
Navis Ryu
f8de5fc
ql/src/test/results/compiler/parse/udf_when.q.out a527fa4
ql/src/test/results/compiler/parse/union.q.out d9be739
Diff: https://reviews.apache.org/r/17887/diff/
Testing
---
Thanks,
Navis Ryu
/subquery_multiinsert.q.out fdafea3
Diff: https://reviews.apache.org/r/18177/diff/
Testing
---
Thanks,
Navis Ryu
ql/src/test/results/clientpositive/multiMapJoin2.q.out c59a407
ql/src/test/results/clientpositive/subquery_multiinsert.q.out fdafea3
Diff: https://reviews.apache.org/r/18177/diff/
Testing
---
Thanks,
Navis Ryu
:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18122/
---
(Updated Feb. 14, 2014, 3:19 p.m.)
Review request for hive, Brock Noland, Navis Ryu, and Swarnim Kulkarni.
Bugs: HIVE-6411
/LazySimpleStructObjectInspector.java
8a5386a
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryObject.java
598683f
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryStruct.java
caf3517
Diff: https://reviews.apache.org/r/18179/diff/
Testing
---
Thanks,
Navis Ryu
/clientpositive/union24.q.out a88951a
ql/src/test/results/clientpositive/union_ppr.q.out f639960
Diff: https://reviews.apache.org/r/17887/diff/
Testing
---
Thanks,
Navis Ryu
.q.out eb5ad5c
ql/src/test/results/clientpositive/union24.q.out a88951a
ql/src/test/results/clientpositive/union_ppr.q.out f639960
Diff: https://reviews.apache.org/r/17887/diff/
Testing
---
Thanks,
Navis Ryu
/authorization_view.q.out a872ce8
Diff: https://reviews.apache.org/r/17732/diff/
Testing
---
Thanks,
Navis Ryu
/17732/#review33775
---
On Feb. 6, 2014, 8:22 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17732
/authorization/DelegatableAuthorizationProvider.java
PRE-CREATION
ql/src/test/queries/clientpositive/authorization_explain.q PRE-CREATION
ql/src/test/results/clientpositive/authorization_explain.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/16034/diff/
Testing
---
Thanks,
Navis
/cli/session/TestSessionHooks.java
2fac800
Diff: https://reviews.apache.org/r/17694/diff/
Testing
---
Thanks,
Navis Ryu
2fac800
Diff: https://reviews.apache.org/r/17694/diff/
Testing
---
Thanks,
Navis Ryu
/udf_current_database.q.out PRE-CREATION
ql/src/test/results/compiler/errors/invalid_select.q.out af165bb
Diff: https://reviews.apache.org/r/16900/diff/
Testing
---
Thanks,
Navis Ryu
981060f
ql/src/test/results/clientpositive/authorization_view.q.out 23a1c0b
Diff: https://reviews.apache.org/r/17732/diff/
Testing
---
Thanks,
Navis Ryu
/serde2/lazydio/LazyDioShort.java
137d0c8
serde/src/test/org/apache/hadoop/hive/serde2/lazy/TestLazyPrimitive.java
868dd4c
serde/src/test/org/apache/hadoop/hive/serde2/lazybinary/TestLazyBinarySerDe.java
69c891d
Diff: https://reviews.apache.org/r/17652/diff/
Testing
---
Thanks,
Navis
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16900/#review31885
---
On Jan. 15, 2014, 6:22 a.m., Navis Ryu wrote
/show_functions.q.out 57c9036
ql/src/test/results/clientpositive/udf_current_database.q.out PRE-CREATION
ql/src/test/results/compiler/errors/invalid_select.q.out af165bb
Diff: https://reviews.apache.org/r/16900/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17004/#review33305
---
On Jan. 29, 2014, 7:36 a.m., Navis Ryu wrote
69c891d
Diff: https://reviews.apache.org/r/17652/diff/
Testing
---
Thanks,
Navis Ryu
/test/results/clientpositive/view_cast.q.out ed1e410
ql/src/test/results/clientpositive/view_inputs.q.out 5ea7589
ql/src/test/results/clientpositive/windowing.q.out bbd169f
Diff: https://reviews.apache.org/r/17004/diff/
Testing
---
Thanks,
Navis Ryu
service/src/test/org/apache/hive/service/auth/TestPlainSaslHelper.java
8fa4afd
Diff: https://reviews.apache.org/r/17437/diff/
Testing
---
Thanks,
Navis Ryu
---
Thanks,
Navis Ryu
/subquery_in_having.q.out b293ed8
Diff: https://reviews.apache.org/r/16728/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16728/#review32885
---
On Jan. 28, 2014, 1:37 a.m., Navis Ryu wrote
/java/org/apache/hive/service/cli/session/HiveSessionImpl.java
445c858
service/src/java/org/apache/hive/service/cli/session/SessionManager.java
bfe0e7b
Diff: https://reviews.apache.org/r/15449/diff/
Testing
---
Confirmed in the local environment.
Thanks,
Navis Ryu
/17002/diff/
Testing
---
Thanks,
Navis Ryu
/
Testing
---
Thanks,
Navis Ryu
/clientnegative/alter_partition_coltype_2columns.q.out
e1f9a27
ql/src/test/results/clientpositive/alter_partition_coltype.q.out 685bf88
Diff: https://reviews.apache.org/r/17002/diff/
Testing
---
Thanks,
Navis Ryu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15449/#review28855
---
On Nov. 18, 2013, 2:32 a.m., Navis Ryu wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15449/#review28950
---
On Nov. 18, 2013, 2:32 a.m., Navis Ryu wrote
/session/SessionManager.java
bfe0e7b
Diff: https://reviews.apache.org/r/15449/diff/
Testing
---
Confirmed in the local environment.
Thanks,
Navis Ryu
/DDLTask.java 92ed55b
ql/src/java/org/apache/hadoop/hive/ql/metadata/DefaultStorageHandler.java
83772bd
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 0e2d555
Diff: https://reviews.apache.org/r/17178/diff/
Testing
---
Thanks,
Navis Ryu
/hive/ql/parse/HiveParser.g 1ce6bf3
Diff: https://reviews.apache.org/r/17179/diff/
Testing
---
Thanks,
Navis Ryu
---
On Jan. 22, 2014, 4:42 a.m., Navis Ryu wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15449/
---
(Updated
201 - 300 of 389 matches
Mail list logo