Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns (#… [doris]

2024-11-26 Thread via GitHub
yiguolei merged PR #44585: URL: https://github.com/apache/doris/pull/44585 -- 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 unsubscribe, e-mail: commits-unsubscr...@doris.apa

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns (#… [doris]

2024-11-25 Thread via GitHub
doris-robot commented on PR #44585: URL: https://github.com/apache/doris/pull/44585#issuecomment-2499622320 TeamCity be ut coverage result: Function Coverage: 36.73% (9604/26145) Line Coverage: 28.14% (79007/280774) Region Coverage: 26.82% (40597/151393) Branch Coverage: 23.6

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns (#… [doris]

2024-11-25 Thread via GitHub
eldenmoon commented on PR #44585: URL: https://github.com/apache/doris/pull/44585#issuecomment-2499513296 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 un

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns (#… [doris]

2024-11-25 Thread via GitHub
doris-robot commented on PR #44585: URL: https://github.com/apache/doris/pull/44585#issuecomment-2499513290 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+proce

[PR] [opt](Variant) avoid unnecessary mem for variant extracted columns (#… [doris]

2024-11-25 Thread via GitHub
eldenmoon opened a new pull request, #44585: URL: https://github.com/apache/doris/pull/44585 cherry-pick from #43567 -- 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 unsu

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-11 Thread via GitHub
eldenmoon closed pull request #43443: [opt](Variant) avoid unnecessary mem for variant extracted columns URL: https://github.com/apache/doris/pull/43443 -- 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](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-11 Thread via GitHub
eldenmoon merged PR #43567: URL: https://github.com/apache/doris/pull/43567 -- 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 unsubscribe, e-mail: commits-unsubscr...@doris.ap

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-11 Thread via GitHub
github-actions[bot] commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467664475 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](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-11 Thread via GitHub
github-actions[bot] commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467560652 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](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
doris-robot commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467164320 TeamCity be ut coverage result: Function Coverage: 37.91% (9866/26027) Line Coverage: 29.08% (82241/282779) Region Coverage: 28.24% (42340/149931) Branch Coverage: 24.7

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
github-actions[bot] commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467148135 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
github-actions[bot] commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467147085 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
eldenmoon commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467144744 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 un

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
doris-robot commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467143342 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+proce

Re: [PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
eldenmoon commented on PR #43567: URL: https://github.com/apache/doris/pull/43567#issuecomment-2467143350 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 un

[PR] [opt](Variant) avoid unnecessary mem for variant extracted columns [doris]

2024-11-10 Thread via GitHub
eldenmoon opened a new pull request, #43567: URL: https://github.com/apache/doris/pull/43567 ### What problem does this PR solve? _field_name_to_index and _field_id_to_index is unnecessary for variant subcolumns, since they use column path as identifier Issue Number: close #xxx