Zoltan Borok-Nagy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24133 )

Change subject: IMPALA-14523: JDBC Scan Parallelism with Shared Cursor Fetching
......................................................................


Patch Set 14: Code-Review+1

(1 comment)

Left a minor comment, otherwise LGTM!

http://gerrit.cloudera.org:8080/#/c/24133/14/common/thrift/PlanNodes.thrift
File common/thrift/PlanNodes.thrift:

http://gerrit.cloudera.org:8080/#/c/24133/14/common/thrift/PlanNodes.thrift@306
PS14, Line 306: bool is_data_source_scan
The new field should be the last field and have the field id '7' instead of 
re-assigning fied id '6'.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3b25b99f5cb77d32c111ba37c0a01378ffdc1107
Gerrit-Change-Number: 24133
Gerrit-PatchSet: 14
Gerrit-Owner: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Tue, 26 May 2026 14:58:58 +0000
Gerrit-HasComments: Yes

Reply via email to