Steve Carlin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23900 )

Change subject: IMPALA-14488: Calcite planner: Support fallback to Original 
planner
......................................................................


Patch Set 4:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/23900/4/common/thrift/ImpalaService.thrift
File common/thrift/ImpalaService.thrift:

http://gerrit.cloudera.org:8080/#/c/23900/4/common/thrift/ImpalaService.thrift@1033
PS4, Line 1033:   RUNTIME_PLANNERS = 191
> Enum item USE_CALCITE_PLANNER=191 changed to RUNTIME_PLANNERS=191 in TImpal
This is ok. CALCITE_PLANNER is still pre-beta mode and nobody should be using 
it.


http://gerrit.cloudera.org:8080/#/c/23900/4/common/thrift/Query.thrift
File common/thrift/Query.thrift:

http://gerrit.cloudera.org:8080/#/c/23900/4/common/thrift/Query.thrift@790
PS4, Line 790:   192: optional list<PlanNodes.TPlannerType> runtime_planners = 
[TPlannerType.ORIGINAL]
> Renaming field 'use_calcite_planner' to 'runtime_planners' in TQueryOptions
This is ok. CALCITE_PLANNER is still pre-beta mode and nobody should be using 
it.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id1fdc5ef92fff84e89af0e19c4246cc15e2ea823
Gerrit-Change-Number: 23900
Gerrit-PatchSet: 4
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
Gerrit-Comment-Date: Mon, 26 Jan 2026 15:42:19 +0000
Gerrit-HasComments: Yes

Reply via email to