github-actions[bot] commented on code in PR #66413:
URL: https://github.com/apache/doris/pull/66413#discussion_r3751288175
##########
fe/fe-connector/fe-connector-paimon/src/main/java/org/apache/doris/connector/paimon/PaimonScanPlanProvider.java:
##########
@@ -703,6 +712,10 @@ private List<ConnectorScanRange> planScanInternal(
// Non-DataSplit → always JNI
for (Split split : nonDataSplits) {
+ if (hasVariantProjection) {
Review Comment:
[P2] Apply IGNORE_JNI before rejecting Variant splits
The method documents `IGNORE_JNI` as dropping both JNI arms, but this
Variant check runs before the `continue` below; the DataSplit JNI arm has the
same ordering, and the force-JNI system-table guard throws even earlier. As a
result, a scan that deliberately ignores JNI-only ranges now fails planning
whenever its projection contains Variant, even though no JNI reader or Variant
carrier would be used for those discarded ranges. Please resolve the escape
hatch before these compatibility checks and skip ignored JNI ranges first,
while retaining the exception for any JNI range that would actually be emitted.
--
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]
