[fix](Nereids) make TVF's distribution spec always be RANDOM #24020
Conversation
|
run buildall |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
|
run buildall |
morrySnow
force-pushed
the
two_number_in_one
branch
from
454943c to
93e7b61
Compare
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
|
PR approved by anyone and no changes requested. |
|
run feut |
|
|
||
| // when we set numbers to gather, coordinator could not process and set none scan range in thrift | ||
| // so we add this test case to ensure this sql do not core dump | ||
| sql """ |
There was a problem hiding this comment.
what is the purpose of this sql?
github-actions
bot
added
the
approved
|
PR approved by at least one committer and no changes requested. |
Nereids make TVF number as Gather distribution if backend num = 1. But coordinator could not process gather fragment with scan node. In the long run, we need to get coordinators to support this scenario. But it is need a lot of refactor. So, we just forbid Gather distribution for ScanNode now.
Proposed changes
Issue Number: close #xxx
Nereids make TVF number as Gather distribution if backend num = 1.
But coordinator could not process gather fragment with scan node.
In the long run, we need to get coordinators to support this scenario.
But it is need a lot of refactor. So, we just forbid Gather
distribution for ScanNode now.
Further comments
If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...