> On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 390 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line390> > > > > some existing configuration is probably needed (and better)
IIUC, hive configurations are in the server side and HiveQueryResultSet is in the client side. So I don't understand how to load hive configurations in HiveQueryResultSet. Is there any idea? > On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 402 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line402> > > > > also, I wonder if it makes sense to add rows to rowset immediately > > after reading. Storing all rows and then adding them all to rowset stores > > all rows twice in memory. Thank you for the advice. I'll think it over. > On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 456 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line456> > > > > nit: why object? I will change to string. > On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 466 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line466> > > > > I wonder why we need to have thrift serialization? One of the goals is > > to avoid it. Perhaps it can be done in a follow-up JIRA Sorry to confuse you, what I want to do here is to convert a object to a standard Java object. I think we can just use ObjectInspectorUtils#copyToStandardObject here. > On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 486 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line486> > > > > nit: useless check, schema is already used above to get descriptors I will fix it. > On 12月 3, 2015, 9:40 p.m., Sergey Shelukhin wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java, line 492 > > <https://reviews.apache.org/r/40867/diff/1/?file=1150526#file1150526line492> > > > > nit: get(pos) can be done once I will fix it. - Takanobu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40867/#review108889 ----------------------------------------------------------- On 12月 2, 2015, 12:52 p.m., Takanobu Asanuma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40867/ > ----------------------------------------------------------- > > (Updated 12月 2, 2015, 12:52 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > This is a WIP patch for HIVE-11527 > > * I added a new configuration whose name is > hive.server2.webhdfs.bypass.enabled. The default is false. When this value is > true, clients use the bypass. > > * I still have not considered security such as Kerberos and SSL at present. > > * I have not implement Statement#setFetchSize for bypass yet. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java db942b0 > jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java 245c6a3 > jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java 180f99e8 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java 8fafd61 > ql/src/java/org/apache/hadoop/hive/ql/exec/FetchTask.java 1634143 > service/if/TCLIService.thrift baf583f > service/src/gen/thrift/gen-cpp/TCLIService_types.h b078c99 > service/src/gen/thrift/gen-cpp/TCLIService_types.cpp b852379 > > service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TExecuteStatementResp.java > 0b9aa0f > > service/src/gen/thrift/gen-javabean/org/apache/hive/service/cli/thrift/TProtocolVersion.java > c936ada > service/src/gen/thrift/gen-py/TCLIService/ttypes.py ef5f5f5 > service/src/gen/thrift/gen-py/hive_service/ThriftHive-remote e167d5b > service/src/gen/thrift/gen-rb/t_c_l_i_service_types.rb f004ec4 > service/src/java/org/apache/hive/service/cli/CLIService.java adc9809 > service/src/java/org/apache/hive/service/cli/operation/Operation.java > 25cefc2 > > service/src/java/org/apache/hive/service/cli/operation/OperationManager.java > b0bd351 > service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java > 1331a99 > service/src/java/org/apache/hive/service/cli/session/HiveSession.java > 4f4e92d > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > a14908b > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 8434965 > > Diff: https://reviews.apache.org/r/40867/diff/ > > > Testing > ------- > > I have tested few simple queries and they worked well. But I think there are > some problems for some queries. I'm going to test more queries and fix bugs. > I'm also going to add unit tests. > > > Thanks, > > Takanobu Asanuma > >
