[GitHub] [doris] github-actions[bot] commented on pull request #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430872053 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

[GitHub] [doris] github-actions[bot] commented on pull request #16758: [improve](inverted index) Add element count limit for inverted index searcher cache

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1430875477 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

[GitHub] [doris] zclllyybb opened a new pull request, #16777: [feature] Make VUnionNode support conjecture logic

2023-02-14 Thread via GitHub
zclllyybb opened a new pull request, #16777: URL: https://github.com/apache/doris/pull/16777 ## Problem Summary Add conjection logic to VUnionNode. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document

[GitHub] [doris] bowenliang123 commented on pull request #16736: [chore](dependency) show progress bar when downloading third-party dependencies

2023-02-14 Thread via GitHub
bowenliang123 commented on PR #16736: URL: https://github.com/apache/doris/pull/16736#issuecomment-1430877375 Cleaned up code with mering #https://github.com/apache/doris/pull/16754 from master branch and squashed commits. -- This is an automated message from the Apache Git Service. To re

[GitHub] [doris] github-actions[bot] commented on pull request #16777: [feature] Make VUnionNode support conjecture logic

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16777: URL: https://github.com/apache/doris/pull/16777#issuecomment-1430878851 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

[GitHub] [doris] github-actions[bot] commented on pull request #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430879185 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

[GitHub] [doris] github-actions[bot] commented on pull request #16036: [feature] Support mutable property for partition

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16036: URL: https://github.com/apache/doris/pull/16036#issuecomment-1430879958 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

[GitHub] [doris] github-actions[bot] commented on pull request #16639: [Improvement](brpc) Using a thread pool for RPC service avoiding std::mutex block brpc::bthread

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16639: URL: https://github.com/apache/doris/pull/16639#issuecomment-1430880388 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

[GitHub] [doris] github-actions[bot] commented on pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16383: URL: https://github.com/apache/doris/pull/16383#issuecomment-1430880971 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

[GitHub] [doris] github-actions[bot] commented on pull request #16777: [feature] Make VUnionNode support conjecture logic

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16777: URL: https://github.com/apache/doris/pull/16777#issuecomment-1430882130 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

[GitHub] [doris] englefly commented on a diff in pull request #16761: [feature](nereids)eliminate inner/semi/cross join when any child is EmptyRelation.

2023-02-14 Thread via GitHub
englefly commented on code in PR #16761: URL: https://github.com/apache/doris/pull/16761#discussion_r1106751609 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/EliminateInnerJoinByEmptyRelation.java: ## @@ -0,0 +1,61 @@ +// Licensed to the Apache Softw

[GitHub] [doris] github-actions[bot] commented on pull request #16466: [Pipeline][WIP] Introduce pipeline tracing

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16466: URL: https://github.com/apache/doris/pull/16466#issuecomment-1430884436 `sh-checker report` To get the full details, please check in the [job]("https://github.com/apache/doris/actions/runs/4181641108";) output. shellcheck err

[GitHub] [doris] sohardforaname opened a new pull request, #16778: [FIx](planner)fix create table with null_type cause be core bug

2023-02-14 Thread via GitHub
sohardforaname opened a new pull request, #16778: URL: https://github.com/apache/doris/pull/16778 # Proposed changes Issue Number: close #xxx ## Problem summary sql: create table t as select null as k will cause be core sometime. now we change it null_type to tinyint n

[GitHub] [doris] Gabriel39 opened a new pull request, #16779: [pipeline](shuffle) Improve broadcast shuffle

2023-02-14 Thread via GitHub
Gabriel39 opened a new pull request, #16779: URL: https://github.com/apache/doris/pull/16779 # Proposed changes Issue Number: close #xxx ## Problem summary Now we reuse buffer pool for broadcast shuffle on pipeline engine. This PR ensures that a pipeline with a broadcast

[GitHub] [doris] englefly commented on a diff in pull request #16761: [feature](nereids)eliminate inner/semi/cross join when any child is EmptyRelation.

2023-02-14 Thread via GitHub
englefly commented on code in PR #16761: URL: https://github.com/apache/doris/pull/16761#discussion_r1106751609 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/EliminateInnerJoinByEmptyRelation.java: ## @@ -0,0 +1,61 @@ +// Licensed to the Apache Softw

[GitHub] [doris] github-actions[bot] commented on pull request #16779: [pipeline](shuffle) Improve broadcast shuffle

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16779: URL: https://github.com/apache/doris/pull/16779#issuecomment-1430892872 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

[GitHub] [doris] pengxiangyu opened a new pull request, #16780: [fix](cooldown)fix bug, rename remote compaction thread

2023-02-14 Thread via GitHub
pengxiangyu opened a new pull request, #16780: URL: https://github.com/apache/doris/pull/16780 # Proposed changes Issue Number: close #xxx ## Problem summary fix bug, rename remote compaction thread ## Checklist(Required) * [ ] Does it affect the original be

[GitHub] [doris] Jibing-Li commented on a diff in pull request #16763: [Fix](multi catalog)Fix partition case bug.

2023-02-14 Thread via GitHub
Jibing-Li commented on code in PR #16763: URL: https://github.com/apache/doris/pull/16763#discussion_r1106763228 ## fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java: ## @@ -119,6 +119,12 @@ public static List parseColumnsFromPath( if (columnsFromPat

[GitHub] [doris] github-actions[bot] commented on pull request #16780: [fix](cooldown)fix bug, rename remote compaction thread

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16780: URL: https://github.com/apache/doris/pull/16780#issuecomment-1430898005 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

[GitHub] [doris] englefly commented on a diff in pull request #16761: [feature](nereids) WIP eliminate inner/semi/cross join when any child is EmptyRelation.

2023-02-14 Thread via GitHub
englefly commented on code in PR #16761: URL: https://github.com/apache/doris/pull/16761#discussion_r1106751609 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/EliminateInnerJoinByEmptyRelation.java: ## @@ -0,0 +1,61 @@ +// Licensed to the Apache Softw

[GitHub] [doris] github-actions[bot] commented on pull request #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430898350 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

[GitHub] [doris] github-actions[bot] commented on pull request #16780: [fix](cooldown)fix bug, rename remote compaction thread

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16780: URL: https://github.com/apache/doris/pull/16780#issuecomment-1430899324 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

[GitHub] [doris] github-actions[bot] commented on pull request #16780: [fix](cooldown)fix bug, rename remote compaction thread

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16780: URL: https://github.com/apache/doris/pull/16780#issuecomment-1430899381 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

[GitHub] [doris] yiguolei merged pull request #16748: [Chore](build) enable fallthrough check annd fix some fallthrough bug

2023-02-14 Thread via GitHub
yiguolei merged PR #16748: URL: https://github.com/apache/doris/pull/16748 -- 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

[doris] branch master updated (585b74b175 -> f50edff59d)

2023-02-14 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 585b74b175 [chore](workflow) Fix Ubuntu package conflicts by skipping apt upgrade (#16754) add f50edff59d [Chor

<    1   2   3