-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16076/
-----------------------------------------------------------
(Updated Dec. 18, 2013, 2:02 a.m.)
Review request for hive.
Changes
-------
Rewrote whole codes to address comments. Checked jdbc2 in hive-0.12 is working
good with this (via V4).
Bugs: HIVE-3746
https://issues.apache.org/jira/browse/HIVE-3746
Repository: hive-git
Description
-------
serialize result set in columnar format
Diffs (updated)
-----
itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/TestHiveServer2.java
eb08628
jdbc/src/java/org/apache/hive/jdbc/HiveBaseResultSet.java b02f374
jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java e420b75
jdbc/src/java/org/apache/hive/jdbc/HiveDatabaseMetaData.java fe39c0c
jdbc/src/java/org/apache/hive/jdbc/HiveMetaDataResultSet.java d1ac109
jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java 061337d
ql/src/java/org/apache/hadoop/hive/ql/exec/FetchFormatter.java d369e0e
ql/src/java/org/apache/hadoop/hive/ql/exec/ListSinkOperator.java f64be31
serde/src/java/org/apache/hadoop/hive/serde2/SerDeUtils.java 18ae044
service/if/TCLIService.thrift 62a9730
service/src/gen/thrift/gen-cpp/TCLIService_types.h 853bb4c
service/src/gen/thrift/gen-cpp/TCLIService_types.cpp 7ab1310
service/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/service/ThriftHive.java
1c44789
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TBinaryColumn.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TBinaryValue.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TBoolColumn.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TByteColumn.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TColumn.java
497cc01
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TColumnValue.java
44da2cd
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TDoubleColumn.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TExecuteStatementReq.java
ea656ac
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TGetTablesReq.java
1cb5147
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TI16Column.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TI32Column.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TI64Column.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TOpenSessionReq.java
8ab8297
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TOpenSessionResp.java
688f790
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TProtocolVersion.java
8c6c4f0
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TRow.java
0b6772c
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TRowSet.java
db2262d
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TStatus.java
81c2f16
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TStringColumn.java
PRE-CREATION
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TTableSchema.java
ff5e54d
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TTypeDesc.java
251f86a
service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TTypeQualifiers.java
3935555
service/src/gen/thrift/gen-py/TCLIService/ttypes.py 185ea5b
service/src/gen/thrift/gen-rb/t_c_l_i_service_types.rb c94acbf
service/src/java/org/apache/hive/service/cli/CLIService.java 8c85386
service/src/java/org/apache/hive/service/cli/Column.java PRE-CREATION
service/src/java/org/apache/hive/service/cli/ColumnBasedSet.java PRE-CREATION
service/src/java/org/apache/hive/service/cli/ColumnValue.java cf2b3d9
service/src/java/org/apache/hive/service/cli/OperationHandle.java 8f548da
service/src/java/org/apache/hive/service/cli/Row.java 9e419e9
service/src/java/org/apache/hive/service/cli/RowBasedSet.java PRE-CREATION
service/src/java/org/apache/hive/service/cli/RowSet.java dce506d
service/src/java/org/apache/hive/service/cli/RowSetFactory.java PRE-CREATION
service/src/java/org/apache/hive/service/cli/SessionHandle.java 1ed02e2
service/src/java/org/apache/hive/service/cli/TableSchema.java 155f529
service/src/java/org/apache/hive/service/cli/operation/GetCatalogsOperation.java
70cabe3
service/src/java/org/apache/hive/service/cli/operation/GetColumnsOperation.java
8d09d1c
service/src/java/org/apache/hive/service/cli/operation/GetFunctionsOperation.java
e3b161a
service/src/java/org/apache/hive/service/cli/operation/GetSchemasOperation.java
f413116
service/src/java/org/apache/hive/service/cli/operation/GetTableTypesOperation.java
d168d5e
service/src/java/org/apache/hive/service/cli/operation/GetTablesOperation.java
c8cce08
service/src/java/org/apache/hive/service/cli/operation/GetTypeInfoOperation.java
a923199
service/src/java/org/apache/hive/service/cli/operation/HiveCommandOperation.java
e5bfd92
service/src/java/org/apache/hive/service/cli/operation/Operation.java 5d1dd5f
service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java
296f8b3
service/src/java/org/apache/hive/service/cli/session/HiveSession.java 00058cc
service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java
c0e6151
service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java
708f4e4
service/src/java/org/apache/hive/service/cli/session/SessionManager.java
e262b72
service/src/java/org/apache/hive/service/cli/thrift/EmbeddedThriftBinaryCLIService.java
83f2535
service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
9df110e
service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIServiceClient.java
9bb2a0f
service/src/test/org/apache/hive/service/cli/thrift/ThriftCLIServiceTest.java
ff7166d
Diff: https://reviews.apache.org/r/16076/diff/
Testing
-------
Thanks,
Navis Ryu
