bugfix: fix propagating empty_relation generates an illegal plan

[yukkit]

Which issue does this PR close?

Closes #5218 .

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

[jackwener]

LGTM, Thanks you @yukkit.

[yukkit]

Can you help merge the pr, I'm going to use this commit @jackwener

[jackwener]

Can you help merge the pr, I'm going to use this commit @jackwener

I can understand what you want.

BUT, even if I merge this PR, you also can't use it right away because it doesn't be included in release version.

If you want to use master branch to get this PR (after this PR was merged), I don't recommend you this method, because master branch isn't stable branch, which may cause other problem.

[alamb]

Thank you @yukkit and @jackwener

[alamb]

BTW if you do want to try out DataFusion directly from master you can use something like this in your Cargo.toml

[patch.crates-io]
datafusion = { git = "https://github.com/apache/arrow-datafusion.git", branch="master" }

[ursabot]

Benchmark runs are scheduled for baseline = b4cf60a and contender = e166500. e166500 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ec2-t3-xlarge-us-east-2] ec2-t3-xlarge-us-east-2
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on test-mac-arm] test-mac-arm
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ursa-i9-9960x] ursa-i9-9960x
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ursa-thinkcentre-m75q] ursa-thinkcentre-m75q
Buildkite builds:
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java