BiteTheDDDDt opened a new pull request, #63969:
URL: https://github.com/apache/doris/pull/63969
### What problem does this PR solve?
Issue Number: N/A
Related PR: #59088
Problem Summary: #59088 changed TopN runtime predicate target initialization
to rely on a storage column id. For targets that cannot create a storage column
predicate, such as non-pushdown TopN predicates or unsupported storage columns,
init_target returned before marking the target as detected. That left
RuntimePredicate disabled, so the scan side ignored the TopN source even though
FE had sent the source id. This PR keeps the target detected when no storage
predicate is created, removes obsolete compatibility skips for missing runtime
predicate descs, and adds FE/BE coverage for the source marking and no-column
target paths.
### Release note
None
### Check List (For Author)
- Test: Unit Test
- ./run-fe-ut.sh --run
org.apache.doris.qe.runtime.ThriftPlansBuilderTest,org.apache.doris.qe.CoordinatorTest
- ./run-be-ut.sh --run --filter=RuntimePredicateTest.*
- cd fe && mvn checkstyle:check -pl fe-core -q
- build-support/check-format.sh
- Behavior changed: No
- Does this need documentation: No
--
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]
