Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1806731367 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

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
xiaokang merged PR #26771: URL: https://github.com/apache/doris/pull/26771 -- 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:

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
xiaokang merged PR #26780: URL: https://github.com/apache/doris/pull/26780 -- 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:

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
xiaokang commented on PR #26780: URL: https://github.com/apache/doris/pull/26780#issuecomment-1806730816 #26771 -- 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

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26780: URL: https://github.com/apache/doris/pull/26780#issuecomment-1805738245 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 48.62 seconds stream load tsv: 575 seconds loaded 74807831229 Bytes,

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26780: URL: https://github.com/apache/doris/pull/26780#issuecomment-1805734350 TeamCity be ut coverage result: Function Coverage: 38.33% (7955/20755) Line Coverage: 30.18% (64530/213825) Region Coverage: 29.27% (33307/113778) Branch Coverage:

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26780: URL: https://github.com/apache/doris/pull/26780#issuecomment-1805671819 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

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1805666494 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

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

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

[PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
eldenmoon opened a new pull request, #26780: URL: https://github.com/apache/doris/pull/26780 Other wise the key loaded cache with is without sequence column, and lead to stale data read ## Proposed changes Issue Number: close #xxx ## Further comments If

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1805540119 TeamCity be ut coverage result: Function Coverage: 36.77% (8404/22856) Line Coverage: 29.28% (68135/232682) Region Coverage: 27.91% (35214/126188) Branch Coverage:

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1805502537 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 4c3ba8a043e5adceed1510835fa6a744b50de44b, data reload: false

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1805501320 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.4 seconds stream load tsv: 558 seconds loaded 74807831229 Bytes,

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

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

Re: [PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26771: URL: https://github.com/apache/doris/pull/26771#issuecomment-1805440191 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

[PR] [Fix](row store) cache invalidate key should not include sequence column [doris]

2023-11-10 Thread via GitHub
eldenmoon opened a new pull request, #26771: URL: https://github.com/apache/doris/pull/26771 Other wise the key loaded cache with is without sequence column, and lead to stale data read ## Proposed changes Issue Number: close #xxx ## Further comments If