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

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

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

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

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

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] github-actions[bot] commented on pull request #16780: [fix](cooldown)fix bug, rename remote compaction thread

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 #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

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

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.

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] Jibing-Li commented on a diff in pull request #16763: [Fix](multi catalog)Fix partition case bug.

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] pengxiangyu opened a new pull request, #16780: [fix](cooldown)fix bug, rename remote compaction thread

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] github-actions[bot] commented on pull request #16779: [pipeline](shuffle) Improve broadcast shuffle

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] englefly commented on a diff in pull request #16761: [feature](nereids)eliminate inner/semi/cross join when any child is EmptyRelation.

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] Gabriel39 opened a new pull request, #16779: [pipeline](shuffle) Improve broadcast shuffle

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] sohardforaname opened a new pull request, #16778: [FIx](planner)fix create table with null_type cause be core bug

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] github-actions[bot] commented on pull request #16466: [Pipeline][WIP] Introduce pipeline tracing

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] englefly commented on a diff in pull request #16761: [feature](nereids)eliminate inner/semi/cross join when any child is EmptyRelation.

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 #16777: [feature] Make VUnionNode support conjecture logic

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] github-actions[bot] commented on pull request #16383: [Improvement](thrift) optimize thrift messages

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 #16639: [Improvement](brpc) Using a thread pool for RPC service avoiding std::mutex block brpc::bthread

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 #16036: [feature] Support mutable property for partition

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 #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

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 #16777: [feature] Make VUnionNode support conjecture logic

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] bowenliang123 commented on pull request #16736: [chore](dependency) show progress bar when downloading third-party dependencies

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] zclllyybb opened a new pull request, #16777: [feature] Make VUnionNode support conjecture logic

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] github-actions[bot] commented on pull request #16758: [improve](inverted index) Add element count limit for inverted index searcher cache

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] github-actions[bot] commented on pull request #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

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

github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1430871799 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

[doris] branch master updated: [chore](workflow) Fix Ubuntu package conflicts by skipping apt upgrade (#16754)

This is an automated email from the ASF dual-hosted git repository. adonisling pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 585b74b175 [chore](workflow) Fix Ubuntu packa

[GitHub] [doris] adonis0147 merged pull request #16754: [chore](workflow) Fix Ubuntu package conflicts by skipping `apt upgrade`

adonis0147 merged PR #16754: URL: https://github.com/apache/doris/pull/16754 -- 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.a

[GitHub] [doris] hello-stephen commented on pull request #16775: [enhance](util) Split PriorityWorkStealingThreadPool into impl cpp && modernize-pass-by-value blocking_put

hello-stephen commented on PR #16775: URL: https://github.com/apache/doris/pull/16775#issuecomment-1430868621 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 33.86 seconds stream load tsv: 478 seconds loaded 74807831229 Bytes, about 149 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16613: [Feature](struct-type) Add implicitly cast for struct-type

github-actions[bot] commented on PR #16613: URL: https://github.com/apache/doris/pull/16613#issuecomment-1430864885 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] github-actions[bot] commented on pull request #16613: [Feature](struct-type) Add implicitly cast for struct-type

github-actions[bot] commented on PR #16613: URL: https://github.com/apache/doris/pull/16613#issuecomment-1430864840 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 #16767: [improvement](filecache) split file cache into sharding directories

github-actions[bot] commented on PR #16767: URL: https://github.com/apache/doris/pull/16767#issuecomment-1430862609 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 #16754: [chore](dependency) fix ubuntu package conflict by skipping `apt upgrade` in build-thirdparty workflow

github-actions[bot] commented on PR #16754: URL: https://github.com/apache/doris/pull/16754#issuecomment-1430861902 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] AshinGau commented on a diff in pull request #16767: [improvement](filecache) split file cache into sharding directories

AshinGau commented on code in PR #16767: URL: https://github.com/apache/doris/pull/16767#discussion_r1106726945 ## be/src/io/cache/block/block_lru_file_cache.cpp: ## @@ -634,63 +637,113 @@ void LRUFileCache::remove(const Key& key, bool is_persistent, size_t offset, } void L

[GitHub] [doris] github-actions[bot] commented on pull request #16776: [Feature](map-type) Support stream load and fix some bugs for map type

github-actions[bot] commented on PR #16776: URL: https://github.com/apache/doris/pull/16776#issuecomment-1430846525 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] amorynan opened a new pull request, #16776: [Feature](map-type) Support stream load and fix some bugs for map type

amorynan opened a new pull request, #16776: URL: https://github.com/apache/doris/pull/16776 # Proposed changes 1. support stream load with json, csv format for map 2. fix olap convertor when compaction action in map column which has null Add implicitly cast for struct-type.

[GitHub] [doris] Tanya-W commented on a diff in pull request #16758: [improve](inverted index) Add element count limit for inverted index searcher cache

Tanya-W commented on code in PR #16758: URL: https://github.com/apache/doris/pull/16758#discussion_r1106717034 ## be/src/olap/rowset/segment_v2/inverted_index_cache.cpp: ## @@ -49,6 +49,19 @@ void InvertedIndexSearcherCache::create_global_instance(size_t capacity, uint32_ Inve

[GitHub] [doris] github-actions[bot] commented on pull request #16748: [Chore](build) enable fallthrough check annd fix some fallthrough bug

github-actions[bot] commented on PR #16748: URL: https://github.com/apache/doris/pull/16748#issuecomment-1430842200 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] github-actions[bot] commented on pull request #16748: [Chore](build) enable fallthrough check annd fix some fallthrough bug

github-actions[bot] commented on PR #16748: URL: https://github.com/apache/doris/pull/16748#issuecomment-1430842162 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] Tanya-W commented on a diff in pull request #16758: [improve](inverted index) Add element count limit for inverted index searcher cache

Tanya-W commented on code in PR #16758: URL: https://github.com/apache/doris/pull/16758#discussion_r1106715012 ## be/src/olap/lru_cache.h: ## @@ -322,6 +324,9 @@ class LRUCache { // Separate from constructor so caller can easily make an array of LRUCache void set_cap

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

luozenglin commented on PR #16748: URL: https://github.com/apache/doris/pull/16748#issuecomment-1430839908 LGTM -- 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 unsubscri

[GitHub] [doris] bowenliang123 commented on a diff in pull request #16768: [fix](docs) Fix broken graph in en/broker-load-manual.md

bowenliang123 commented on code in PR #16768: URL: https://github.com/apache/doris/pull/16768#discussion_r1106706838 ## docs/en/docs/data-operate/import/import-way/broker-load-manual.md: ## @@ -43,38 +43,38 @@ After the user submits the import task, FE will generate the corresp

[GitHub] [doris] bowenliang123 commented on a diff in pull request #16768: [fix](docs) Fix broken graph in en/broker-load-manual.md

bowenliang123 commented on code in PR #16768: URL: https://github.com/apache/doris/pull/16768#discussion_r1106706838 ## docs/en/docs/data-operate/import/import-way/broker-load-manual.md: ## @@ -43,38 +43,38 @@ After the user submits the import task, FE will generate the corresp

[GitHub] [doris] bowenliang123 commented on a diff in pull request #16768: [fix](docs) Fix broken graph in en/broker-load-manual.md

bowenliang123 commented on code in PR #16768: URL: https://github.com/apache/doris/pull/16768#discussion_r1106706838 ## docs/en/docs/data-operate/import/import-way/broker-load-manual.md: ## @@ -43,38 +43,38 @@ After the user submits the import task, FE will generate the corresp

[doris] branch master updated: [Fix](profile) fix `/query_profile` action. (#16540)

This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new a6bda81dba [Fix](profile) fix `/query_profile`

[GitHub] [doris] yiguolei merged pull request #16540: [Fix](profile) fix `/query_profile` action.

yiguolei merged PR #16540: URL: https://github.com/apache/doris/pull/16540 -- 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

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16775: [enhance](util) Split PriorityWorkStealingThreadPool into impl cpp && modernize-pass-by-value blocking_put

github-actions[bot] commented on code in PR #16775: URL: https://github.com/apache/doris/pull/16775#discussion_r1106701569 ## be/src/util/priority_work_stealing_thread_pool.cpp: ## @@ -0,0 +1,113 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contr

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

carlvinhust2012 commented on code in PR #16763: URL: https://github.com/apache/doris/pull/16763#discussion_r1106701378 ## fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java: ## @@ -119,6 +119,12 @@ public static List parseColumnsFromPath( if (columnsF

[GitHub] [doris] github-actions[bot] commented on pull request #16540: [Fix](profile) fix `/query_profile` action.

github-actions[bot] commented on PR #16540: URL: https://github.com/apache/doris/pull/16540#issuecomment-1430813866 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] github-actions[bot] commented on pull request #16540: [Fix](profile) fix `/query_profile` action.

github-actions[bot] commented on PR #16540: URL: https://github.com/apache/doris/pull/16540#issuecomment-1430813826 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] carlvinhust2012 commented on a diff in pull request #16768: [fix](docs) Fix broken graph in en/broker-load-manual.md

carlvinhust2012 commented on code in PR #16768: URL: https://github.com/apache/doris/pull/16768#discussion_r1106699670 ## docs/en/docs/data-operate/import/import-way/broker-load-manual.md: ## @@ -43,38 +43,38 @@ After the user submits the import task, FE will generate the corre

[GitHub] [doris] morningman commented on a diff in pull request #16767: [improvement](filecache) split file cache into sharding directories

morningman commented on code in PR #16767: URL: https://github.com/apache/doris/pull/16767#discussion_r1106698431 ## be/src/io/cache/block/block_lru_file_cache.cpp: ## @@ -634,63 +637,113 @@ void LRUFileCache::remove(const Key& key, bool is_persistent, size_t offset, } void

[GitHub] [doris] JackDrogon opened a new pull request, #16775: [enhance](util) Split PriorityWorkStealingThreadPool into impl cpp && modernize-pass-by-value blocking_put

JackDrogon opened a new pull request, #16775: URL: https://github.com/apache/doris/pull/16775 ## Problem summary - Split PriorityWorkStealingThreadPool into impl cpp to reduce compile time - Refactor record_batch_queue blocking_queue blocking_put to pass-by-value ## Checklist

[GitHub] [doris] morningman commented on a diff in pull request #16649: [doc](filecache) add file cache document

morningman commented on code in PR #16649: URL: https://github.com/apache/doris/pull/16649#discussion_r1106691731 ## docs/en/docs/lakehouse/filecache.md: ## @@ -0,0 +1,89 @@ +--- +{ +"title": "File Cache", +"language": "en" +} +--- + + + +# File Cache + + Review Comment

[GitHub] [doris] starocean999 opened a new pull request, #16774: [enhancement](nereids)add eliminate left nullaware anti join rule

starocean999 opened a new pull request, #16774: URL: https://github.com/apache/doris/pull/16774 # Proposed changes Issue Number: close #xxx ## Problem summary if no join conjunct is nullable, the left null aware anti join can be converted to left anti join ## Chec

[GitHub] [doris] yagagagaga closed pull request #16751: [typo](docs) Fix some misspelled words

yagagagaga closed pull request #16751: [typo](docs) Fix some misspelled words URL: https://github.com/apache/doris/pull/16751 -- 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

[GitHub] [doris] github-actions[bot] commented on pull request #16772: [pipeline](profile) Support real-time profile report in pipeline

github-actions[bot] commented on PR #16772: URL: https://github.com/apache/doris/pull/16772#issuecomment-1430791283 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] Gabriel39 opened a new pull request, #16772: [pipeline](profile) Support real-time profile report in pipeline

Gabriel39 opened a new pull request, #16772: URL: https://github.com/apache/doris/pull/16772 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has

[GitHub] [doris] starocean999 commented on a diff in pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

starocean999 commented on code in PR #16676: URL: https://github.com/apache/doris/pull/16676#discussion_r1106667717 ## fe/fe-core/src/main/java/org/apache/doris/nereids/util/Utils.java: ## @@ -79,6 +79,30 @@ public static R execWithReturnVal(Supplier f) { return (R) an

[GitHub] [doris] bowenliang123 commented on pull request #16754: [chore](dependency) fix ubuntu package conflict by skipping `apt upgrade` in build-thirdparty workflow

bowenliang123 commented on PR #16754: URL: https://github.com/apache/doris/pull/16754#issuecomment-1430783580 Hi, kind ping to @adonis0147 again. I have force pushed to trigger the CI again to pass the P0 Rregression. Nothing new changed. -- This is an automated message from the Apache Gi

[GitHub] [doris] starocean999 commented on a diff in pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

starocean999 commented on code in PR #16676: URL: https://github.com/apache/doris/pull/16676#discussion_r1106667717 ## fe/fe-core/src/main/java/org/apache/doris/nereids/util/Utils.java: ## @@ -79,6 +79,30 @@ public static R execWithReturnVal(Supplier f) { return (R) an

[GitHub] [doris] jackwener closed issue #16707: outer join elimination

jackwener closed issue #16707: outer join elimination URL: https://github.com/apache/doris/issues/16707 -- 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:

[doris] branch master updated: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin (#16411)

This is an automated email from the ASF dual-hosted git repository. jakevin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new ad46e529d8 [feature](Nereids): Infer isNotNull f

[GitHub] [doris] jackwener merged pull request #16411: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin

jackwener merged PR #16411: URL: https://github.com/apache/doris/pull/16411 -- 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

[GitHub] [doris] github-actions[bot] commented on pull request #16411: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin

github-actions[bot] commented on PR #16411: URL: https://github.com/apache/doris/pull/16411#issuecomment-1430780328 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 #16411: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin

github-actions[bot] commented on PR #16411: URL: https://github.com/apache/doris/pull/16411#issuecomment-1430780366 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] starocean999 commented on a diff in pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

starocean999 commented on code in PR #16676: URL: https://github.com/apache/doris/pull/16676#discussion_r1106665552 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/exploration/join/OuterJoinLAsscomProject.java: ## @@ -74,95 +69,56 @@ public Rule build() {

[GitHub] [doris] starocean999 commented on a diff in pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

starocean999 commented on code in PR #16676: URL: https://github.com/apache/doris/pull/16676#discussion_r1106664901 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/exploration/join/OuterJoinAssoc.java: ## @@ -0,0 +1,112 @@ +// Licensed to the Apache Software Foundatio

[GitHub] [doris] hello-stephen commented on pull request #16771: [enhancement](exception safe) make function state exception safe

hello-stephen commented on PR #16771: URL: https://github.com/apache/doris/pull/16771#issuecomment-1430777894 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 33.81 seconds stream load tsv: 476 seconds loaded 74807831229 Bytes, about 149 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16734: [Opt](exec) opt aggreate function performance in nullable column

github-actions[bot] commented on PR #16734: URL: https://github.com/apache/doris/pull/16734#issuecomment-1430777642 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] github-actions[bot] commented on pull request #16734: [Opt](exec) opt aggreate function performance in nullable column

github-actions[bot] commented on PR #16734: URL: https://github.com/apache/doris/pull/16734#issuecomment-1430777614 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] JNSimba merged pull request #16769: [fix](docs)Supplement show processlist documentation

JNSimba merged PR #16769: URL: https://github.com/apache/doris/pull/16769 -- 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.apac

[doris] branch master updated: fix doc (#16769)

This is an automated email from the ASF dual-hosted git repository. diwu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 0664186efe fix doc (#16769) 0664186efe is described

[GitHub] [doris] jackwener closed pull request #14906: [feature](Nereids): OuterJoinAssoc Rule.

jackwener closed pull request #14906: [feature](Nereids): OuterJoinAssoc Rule. URL: https://github.com/apache/doris/pull/14906 -- 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. T

[GitHub] [doris] jackwener commented on a diff in pull request #16761: [feature](nereids)add rule eliminate inner join when any child is EmptyRelation.

jackwener commented on code in PR #16761: URL: https://github.com/apache/doris/pull/16761#discussion_r1106660704 ## regression-test/suites/nereids_tpch_p0/nereids_emptyset_prune.groovy: ## @@ -0,0 +1,56 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or mor

[GitHub] [doris] jackwener commented on a diff in pull request #16761: [feature](nereids)add rule eliminate inner join when any child is EmptyRelation.

jackwener commented on code in PR #16761: URL: https://github.com/apache/doris/pull/16761#discussion_r1106660071 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/EliminateInnerJoinByEmptyRelation.java: ## @@ -0,0 +1,61 @@ +// Licensed to the Apache Soft

[GitHub] [doris] BiteTheDDDDt closed pull request #16654: [Feature](Materialized-View) testing

BiteThet closed pull request #16654: [Feature](Materialized-View) testing URL: https://github.com/apache/doris/pull/16654 -- 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

[doris] branch master updated: [fix](fe)should check slot from both lhs and rhs of outputSmap of join node for colocate join (#16738)

This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 13134c1bfe [fix](fe)should check slot from both

[GitHub] [doris] yiguolei merged pull request #16738: [fix](fe)should check slot from both lhs and rhs of outputSmap of join node for colocate join

yiguolei merged PR #16738: URL: https://github.com/apache/doris/pull/16738 -- 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

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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430738432 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-thirdparty] kaka11chen opened a new pull request, #37: [Enhancement] If kms url is empty, do not call getfileStatus() rpc.

kaka11chen opened a new pull request, #37: URL: https://github.com/apache/doris-thirdparty/pull/37 If kms url is empty, do not call `getfileStatus()` rpc. -- 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 t

[GitHub] [doris] github-actions[bot] commented on pull request #16771: [enhancement](exception safe) make function state exception safe

github-actions[bot] commented on PR #16771: URL: https://github.com/apache/doris/pull/16771#issuecomment-1430736737 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 #16770: [improve](point query) reuse rowset from lookup_row_key to eliminate …

github-actions[bot] commented on PR #16770: URL: https://github.com/apache/doris/pull/16770#issuecomment-1430734565 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] yiguolei opened a new pull request, #16771: [enhancement](exception safe) make function state exception safe

yiguolei opened a new pull request, #16771: URL: https://github.com/apache/doris/pull/16771 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has u

[GitHub] [doris] eldenmoon opened a new pull request, #16770: [improve](point query) reuse rowset from lookup_row_key to eliminate …

eldenmoon opened a new pull request, #16770: URL: https://github.com/apache/doris/pull/16770 …tablet lock Reuse rowset for 2 reasons: 1. eliminate tablet lock for performance issue, if other thread hold the lock too long could affect point query latency 2. rowset should be acqui

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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430730562 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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430727755 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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430727032 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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430723901 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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430723619 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

github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430718478 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] LemonLiTree opened a new pull request, #16769: [fix](docs)Supplement show processlist documentation

LemonLiTree opened a new pull request, #16769: URL: https://github.com/apache/doris/pull/16769 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Ha

[GitHub] [doris] zhannngchen commented on a diff in pull request #16689: [Enchancement](merge-on-write) check the correctness of rowid conversion after compaction

zhannngchen commented on code in PR #16689: URL: https://github.com/apache/doris/pull/16689#discussion_r1106616091 ## be/src/olap/compaction.cpp: ## @@ -390,11 +391,12 @@ Status Compaction::modify_rowsets() { if (_tablet->keys_type() == KeysType::UNIQUE_KEYS &&

[GitHub] [doris] zhannngchen commented on a diff in pull request #16689: [Enchancement](merge-on-write) check the correctness of rowid conversion after compaction

zhannngchen commented on code in PR #16689: URL: https://github.com/apache/doris/pull/16689#discussion_r1106616091 ## be/src/olap/compaction.cpp: ## @@ -390,11 +391,12 @@ Status Compaction::modify_rowsets() { if (_tablet->keys_type() == KeysType::UNIQUE_KEYS &&

[doris] branch master updated (9b8c91e18c -> 69c70d27bd)

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 9b8c91e18c [improvement](rowset reader) fix possible memleak (#16680) add 69c70d27bd [Refactor](auth) Add Access

[GitHub] [doris] yiguolei merged pull request #16679: [Refactor](auth) Add AccessController to support customized authorization

yiguolei merged PR #16679: URL: https://github.com/apache/doris/pull/16679 -- 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

[GitHub] [doris] SaintBacchus commented on pull request #16461: [fix](compute_node) fix backend_num_for_federation bug for federation…

SaintBacchus commented on PR #16461: URL: https://github.com/apache/doris/pull/16461#issuecomment-1430702673 LGTM -- 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 unsubsc

[GitHub] [doris] github-actions[bot] commented on pull request #16738: [fix](fe)should check slot from both lhs and rhs of outputSmap of join node for colocate join

github-actions[bot] commented on PR #16738: URL: https://github.com/apache/doris/pull/16738#issuecomment-1430697846 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] github-actions[bot] commented on pull request #16738: [fix](fe)should check slot from both lhs and rhs of outputSmap of join node for colocate join

github-actions[bot] commented on PR #16738: URL: https://github.com/apache/doris/pull/16738#issuecomment-1430697820 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 a diff in pull request #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

github-actions[bot] commented on code in PR #15339: URL: https://github.com/apache/doris/pull/15339#discussion_r1106604940 ## be/src/vec/exec/join/process_hash_table_probe_impl.h: ## @@ -607,6 +607,7 @@ Status ProcessHashTableProbe::do_process_with_other_join_conjuncts(

  1   2   3   >