Closed qw4990 closed 1 week ago
Hi @qw4990. Thanks for your PR.
PRs from untrusted users cannot be marked as trusted with /ok-to-test
in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all
.
I understand the commands that are listed here.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 55.9319%. Comparing base (
5333ac3
) to head (8b3ce50
). Report is 12 commits behind head on master.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: AilinKid, elsa0520
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
/retest
/cherry-pick release-8.1
@qw4990: new pull request created to branch release-8.1
: #53326.
What problem does this PR solve?
Issue Number: close #46556
Problem Summary: planner: add projections to keep join keys as
col=col
What changed and how does it work?
In some cases, after creating new EQ function, the 2 args might not be column anymore, which breaks the assumption that join eq keys must be
col=col
, to handle this, inject 2 projections.Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.