hubgeter opened a new pull request, #68871:
URL: https://github.com/apache/doris/pull/68871
### What problem does this PR solve?
Issue Number: N/A
Problem Summary:
`PhysicalLazyMaterializeTVFScan.accept()` dispatched to
`visitPhysicalTVFRelation`, so the dedicated
`PhysicalPlanTranslator.visitPhysicalLazyMaterializeTVFScan` was unreachable
dead code and every TVF scan under TopN lazy materialization was translated by
the parent method.
This PR dispatches `accept()` to `visitPhysicalLazyMaterializeTVFScan`,
consistent with `PhysicalLazyMaterializeOlapScan`, so the dedicated translation
path stays reachable. No behavior change today since both translator methods
are identical, and the added unit test pins the dispatch.
### Release note
None
### Check List (For Author)
- Test
- [ ] Regression test
- [x] Unit Test
- `PhysicalLazyMaterializeTVFScanVisitorTest` (new), plus
`TopnLazyMaterializeTest`, `MaterializeProbeVisitorTest`,
`LazyMaterializeTopNTest`, `TopNRuntimeFilterTest` (27 tests, all pass)
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- Behavior changed:
- [x] No.
- [ ] Yes.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
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]
