Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20495 )

Change subject: PROTOTYPE: Update TCLIService.thrift to a more recent version
......................................................................


Patch Set 3:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/20495/3/common/thrift/hive-1-api/TCLIService.thrift
File common/thrift/hive-1-api/TCLIService.thrift:

http://gerrit.cloudera.org:8080/#/c/20495/3/common/thrift/hive-1-api/TCLIService.thrift@347
PS3, Line 347:   7: TStringValue stringVal    // STRING, LIST, MAP, STRUCT, 
UNIONTYPE, BINARY, DECIMAL, NULL, INTERVAL_YEAR_MONTH, INTERVAL_DAY_TIME
line too long (133 > 90)


http://gerrit.cloudera.org:8080/#/c/20495/3/common/thrift/hive-1-api/TCLIService.thrift@572
PS3, Line 572:   1: required TProtocolVersion client_protocol = 
TProtocolVersion.HIVE_CLI_SERVICE_PROTOCOL_V10
line too long (95 > 90)


http://gerrit.cloudera.org:8080/#/c/20495/3/common/thrift/hive-1-api/TCLIService.thrift@591
PS3, Line 591:   2: required TProtocolVersion serverProtocolVersion = 
TProtocolVersion.HIVE_CLI_SERVICE_PROTOCOL_V10
line too long (101 > 90)


http://gerrit.cloudera.org:8080/#/c/20495/3/common/thrift/hive-1-api/TCLIService.thrift@779
PS3, Line 779: // Retrieves the schema names available in this database.
line has trailing whitespace



--
To view, visit http://gerrit.cloudera.org:8080/20495
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I02320f982e86f539d8eed259a7c5adef49f81de6
Gerrit-Change-Number: 20495
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Comment-Date: Wed, 20 Sep 2023 15:41:39 +0000
Gerrit-HasComments: Yes

Reply via email to