----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44031/#review126757 -----------------------------------------------------------
service-rpc/if/TCLIService.thrift (line 958) <https://reviews.apache.org/r/44031/#comment189824> Please add doc comments for the two new *Req types (TGetFunctionsReq is a good example). If the behavior is expected to match the corresponding JDBC calls, please state so in the comment. - Carl Steinbach On March 29, 2016, 12:11 a.m., Hari Sankar Sivarama Subramaniyan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44031/ > ----------------------------------------------------------- > > (Updated March 29, 2016, 12:11 a.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Repository: hive-git > > > Description > ------- > > HS2 changes: API calls for retrieving primary keys and foreign keys > information. Changes without the generated thrift files. > > > Diffs > ----- > > jdbc/src/java/org/apache/hive/jdbc/HiveDatabaseMetaData.java 7e54d1f > service-rpc/if/TCLIService.thrift aa28b6e > service-rpc/src/gen/thrift/gen-cpp/TCLIService.h 3407564 > service-rpc/src/gen/thrift/gen-cpp/TCLIService.cpp fc82b88 > service-rpc/src/gen/thrift/gen-cpp/TCLIService_server.skeleton.cpp 66ed6a7 > service-rpc/src/gen/thrift/gen-cpp/TCLIService_types.h 7f1d9dd > service-rpc/src/gen/thrift/gen-cpp/TCLIService_types.cpp 3a27a60 > > service-rpc/src/gen/thrift/gen-javabean/org/apache/hive/service/rpc/thrift/TCLIService.java > c684f89 > service-rpc/src/gen/thrift/gen-php/TCLIService.php eba62f1 > service-rpc/src/gen/thrift/gen-php/Types.php b7df50a > service-rpc/src/gen/thrift/gen-py/TCLIService/TCLIService-remote 56f5c5d > service-rpc/src/gen/thrift/gen-py/TCLIService/TCLIService.py ad2d71d > service-rpc/src/gen/thrift/gen-py/TCLIService/ttypes.py c691781 > service-rpc/src/gen/thrift/gen-rb/t_c_l_i_service.rb 7d7f7a7 > service-rpc/src/gen/thrift/gen-rb/t_c_l_i_service_types.rb 07ed97c > service/src/java/org/apache/hive/service/cli/CLIService.java ab30ae2 > service/src/java/org/apache/hive/service/cli/EmbeddedCLIServiceClient.java > 9cad5be > service/src/java/org/apache/hive/service/cli/ICLIService.java 0a54bdd > > service/src/java/org/apache/hive/service/cli/operation/GetCrossReferenceOperation.java > PRE-CREATION > > service/src/java/org/apache/hive/service/cli/operation/GetPrimaryKeysOperation.java > PRE-CREATION > > service/src/java/org/apache/hive/service/cli/operation/OperationManager.java > 56a9c18 > service/src/java/org/apache/hive/service/cli/session/HiveSession.java > 4f4e92d > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > 80a1844 > > service/src/java/org/apache/hive/service/cli/thrift/RetryingThriftCLIServiceClient.java > 14191e5 > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 62fcde5 > > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIServiceClient.java > ccce6dc > > Diff: https://reviews.apache.org/r/44031/diff/ > > > Testing > ------- > > > Thanks, > > Hari Sankar Sivarama Subramaniyan > >
