Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-06 Thread via GitHub
yujun777 commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2573132140 #46482 compare literal don't call toLegacyLiteral -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-06 Thread via GitHub
yujun777 closed pull request #46350: [opt](nereids) cache legacy literal for reducing sort literals time URL: https://github.com/apache/doris/pull/46350 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
github-actions[bot] commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568681671 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
github-actions[bot] commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568681658 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
morrySnow commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568681505 could we avoid use legacy literal to do sort? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
yujun777 commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568678347 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To uns

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
yujun777 commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568676750 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To uns

Re: [PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
Thearas commented on PR #46350: URL: https://github.com/apache/doris/pull/46350#issuecomment-2568676725 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+y

[PR] [opt](nereids) cache legacy literal for reducing sort literals time [doris]

2025-01-02 Thread via GitHub
yujun777 opened a new pull request, #46350: URL: https://github.com/apache/doris/pull/46350 ### What problem does this PR solve? Issue Number: close #xxx Related PR: #xxx Problem Summary: as #45181 mention, when sorting literals, toLegacyLiteral may cost a lot o