[GLUTEN-7078][CORE] The fallback check for Scan should not be skipped when DPP is present
What changes were proposed in this pull request?
(Fixes: #7078) performing fallback detection for Scan in advance is good and does not introduce any side effects.
How was this patch tested?
manual tests
Thanks for opening a pull request!
Could you open an issue for this pull request on Github Issues?
https://github.com/apache/incubator-gluten/issues
Then could you also rename commit message and pull request title in the following format?
[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}
See also:
Run Gluten Clickhouse CI
https://github.com/apache/incubator-gluten/issues/7078
it better to add some ut for this scenario.
it better to add some ut for this scenario.
ok, thank you for the response.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
Do we have any updates on this PR?
I think the if conditions are no longer needed after clean up https://github.com/apache/incubator-gluten/pull/6035? Let's merge to see if everything goes fine. cc @zzcclp