suxiaogang223 commented on code in PR #68837:
URL: https://github.com/apache/doris/pull/68837#discussion_r4245963618
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java:
##########
@@ -3805,9 +3824,9 @@ private DataPartition toDataPartition(DistributionSpec
distributionSpec/* target
partitionExprs.add(context.findSlotRef(partitionExprId));
}
}
- return new
DataPartition(TPartitionType.HIVE_TABLE_SINK_HASH_PARTITIONED, partitionExprs);
+ return createHiveTableSinkDataPartition(partitionExprs,
Config.be_exec_version);
Review Comment:
Fixed in 6b2161c6b9d. Nereids now captures `be_exec_version` in the
statement `TQueryOptions`; both `PlanTranslatorContext` and
`CoordinatorContext` consume that same snapshot instead of rereading the
mutable global config. Added focused coverage for a global version change
between planning and dispatch.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
