[fix])(nereids)Support select catalog.db.table.column from xxx for nereids planner. #23221
Conversation
|
run buildall |
github-actions
bot
added
the
approved
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
| return Utils.qualifiedNameParts(qualifier, table.getName()); | ||
| } | ||
|
|
||
| /** | ||
| * Full qualified table name, concat qualifier and name with `.` as separator. | ||
| */ | ||
| public String qualifiedName() { | ||
| if (qualifier.size() == 3) { |
There was a problem hiding this comment.
why we use special branch?
There was a problem hiding this comment.
qualifier may only contain db name or contain full name (catalog.db.table)，for the first case, need to concat the table name here, for the latter case, we don't need to.
…reids planner. #23221 Nereids doesn't support select table.* from table, this pr is to fix this bug. Support three layer qualifier. (catalog.database.table)
Nereids doesn't support
select table.* from table, this pr is to fix this bug.Support three layer qualifier. (catalog.database.table)
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...