Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-26 Thread via GitHub
yiguolei merged PR #28968: URL: https://github.com/apache/doris/pull/28968 -- 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] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
github-actions[bot] commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868938120 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] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
bobhan1 commented on code in PR #28968: URL: https://github.com/apache/doris/pull/28968#discussion_r1436052330 ## fe/fe-core/src/main/java/org/apache/doris/analysis/DeleteStmt.java: ## @@ -157,7 +157,7 @@ private void constructInsertStmt() throws AnalysisException {

Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
doris-robot commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868857994 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.8 seconds stream load tsv: 563 seconds loaded 74807831229 Bytes, a

Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
doris-robot commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868851248 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit ccfcada943d56b61f4cc9b3222d1af2b46dbe1eb, data reload: false run tpc

Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
morrySnow commented on code in PR #28968: URL: https://github.com/apache/doris/pull/28968#discussion_r1436007911 ## fe/fe-core/src/main/java/org/apache/doris/analysis/DeleteStmt.java: ## @@ -157,7 +157,7 @@ private void constructInsertStmt() throws AnalysisException {

Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-25 Thread via GitHub
bobhan1 commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868835758 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 unsu

Re: [PR] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-24 Thread via GitHub
github-actions[bot] commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868834207 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] [fix](planner) Fix delete statement on MOR unique table [doris]

2023-12-24 Thread via GitHub
github-actions[bot] commented on PR #28968: URL: https://github.com/apache/doris/pull/28968#issuecomment-1868834189 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