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

2023-02-15 Thread via GitHub
platoneko commented on PR #16780: URL: https://github.com/apache/doris/pull/16780#issuecomment-1430909185 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 unsubscrib

[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-15 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 #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1430919618 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] Yulei-Yang opened a new pull request, #16781: [fix](doc) add essential property for hive catalog on Kerberosied hms

2023-02-15 Thread via GitHub
Yulei-Yang opened a new pull request, #16781: URL: https://github.com/apache/doris/pull/16781 # Proposed changes property `hive.metastore.kerberos.principal` is essential when the principal of hms you are connecting is not the default value: hive-metastore/_HOST@your_realms。otherwise, yo

[GitHub] [doris] Jenson97 opened a new pull request, #16782: Features wangtao01

2023-02-15 Thread via GitHub
Jenson97 opened a new pull request, #16782: URL: https://github.com/apache/doris/pull/16782 # Proposed changes ## Problem summary I fixed an error in the document, which helped improve the comprehensibility of the document. ## Checklist(Required) * [ ] Does it aff

[GitHub] [doris] xy720 commented on a diff in pull request #16776: [Feature](map-type) Support stream load and fix some bugs for map type

2023-02-15 Thread via GitHub
xy720 commented on code in PR #16776: URL: https://github.com/apache/doris/pull/16776#discussion_r1106747039 ## be/src/vec/data_types/data_type_map.cpp: ## @@ -21,13 +21,22 @@ #include "vec/columns/column_array.h" #include "vec/columns/column_map.h" #include "vec/common/asser

[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-15 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430943138 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 #16649: [doc](filecache) add file cache document

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16649: URL: https://github.com/apache/doris/pull/16649#issuecomment-1430945552 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 #16649: [doc](filecache) add file cache document

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16649: URL: https://github.com/apache/doris/pull/16649#issuecomment-1430945609 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 #15339: [enhancement-wip](aggregate-function) enhance aggregate funtion collect and add group_array aliases

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430948931 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] xy720 commented on a diff in pull request #16776: [Feature](map-type) Support stream load and fix some bugs for map type

2023-02-15 Thread via GitHub
xy720 commented on code in PR #16776: URL: https://github.com/apache/doris/pull/16776#discussion_r1106817125 ## be/src/vec/data_types/data_type_map.cpp: ## @@ -91,57 +162,51 @@ Status DataTypeMap::from_string(ReadBuffer& rb, IColumn* column) const {

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

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

[GitHub] [doris] xy720 closed issue #16547: [Roadmap] Support Struct+Map Type

2023-02-15 Thread via GitHub
xy720 closed issue #16547: [Roadmap] Support Struct+Map Type URL: https://github.com/apache/doris/issues/16547 -- 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

[doris] branch master updated (f50edff59d -> 0c56a4622c)

2023-02-15 Thread xuyang
This is an automated email from the ASF dual-hosted git repository. xuyang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from f50edff59d [Chore](build) enable fallthrough check annd fix some fallthrough bug (#16748) add 0c56a4622c [Feature

[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-15 Thread via GitHub
github-actions[bot] commented on PR #15339: URL: https://github.com/apache/doris/pull/15339#issuecomment-1430984055 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] isHuangXin commented on issue #16560: [Enhancement] Support microseconds in function `unix_timestamp`

2023-02-15 Thread via GitHub
isHuangXin commented on issue #16560: URL: https://github.com/apache/doris/issues/16560#issuecomment-1430989813 Plz, assign it to me. -- 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

[GitHub] [doris] github-actions[bot] commented on pull request #16750: [Feature](Materialized-View) change mv rewrite from bottom up to up bottom && Compatible with old …

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16750: URL: https://github.com/apache/doris/pull/16750#issuecomment-1430997144 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 #16750: [Feature](Materialized-View) change mv rewrite from bottom up to up bottom && Compatible with old …

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16750: URL: https://github.com/apache/doris/pull/16750#issuecomment-1430997228 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] caiconghui commented on pull request #16619: Revert "[Performance] Optimize performance of like expr using strstr()"

2023-02-15 Thread via GitHub
caiconghui commented on PR #16619: URL: https://github.com/apache/doris/pull/16619#issuecomment-1431002660 cc @weizuo93 -- 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] BiteTheDDDDt merged pull request #16750: [Feature](Materialized-View) change mv rewrite from bottom up to up bottom && Compatible with old …

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

[doris] branch master updated: [Feature](Materialized-View) change mv rewrite from bottom up to up bottom && Compatible with old … (#16750)

2023-02-15 Thread panxiaolei
This is an automated email from the ASF dual-hosted git repository. panxiaolei 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 f4ed52906a [Feature](Materialized-View) chang

[GitHub] [doris] github-actions[bot] commented on pull request #15509: [feature](partition)add default list partition

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #15509: URL: https://github.com/apache/doris/pull/15509#issuecomment-1431012126 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] starocean999 opened a new pull request, #16783: [enhancement](nereids)add a session variable to control join reorder algorithm

2023-02-15 Thread via GitHub
starocean999 opened a new pull request, #16783: URL: https://github.com/apache/doris/pull/16783 # Proposed changes 1. disable join reorder in nereids if **disable_join_reorder** is true. 2. add a session variable **max_table_count_use_cascades_join_reorder** to control join reorder alg

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

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16779: URL: https://github.com/apache/doris/pull/16779#issuecomment-1431021443 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] ByteYue opened a new pull request, #16784: [enhancement](profile) add profile for json& csv file converting to doris column

2023-02-15 Thread via GitHub
ByteYue opened a new pull request, #16784: URL: https://github.com/apache/doris/pull/16784 # Proposed changes Issue Number: close #xxx Previously there are no profile to monitor the time consumption of data converting to doris column, this pr mainly adds profile for json file and c

[GitHub] [doris] github-actions[bot] commented on pull request #16784: [enhancement](profile) add profile for json& csv file converting to doris column

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16784: URL: https://github.com/apache/doris/pull/16784#issuecomment-1431030752 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-15 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1431031705 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 #16784: [enhancement](profile) add profile for json& csv file converting to doris column

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16784: URL: https://github.com/apache/doris/pull/16784#issuecomment-1431044597 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] HappenLee opened a new pull request, #16785: [Opt](exec) Refactor the code and logical functions to SIMD the code

2023-02-15 Thread via GitHub
HappenLee opened a new pull request, #16785: URL: https://github.com/apache/doris/pull/16785 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has

[GitHub] [doris] hello-stephen commented on pull request #16777: [feature] Make VUnionNode support conjecture logic

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16777: URL: https://github.com/apache/doris/pull/16777#issuecomment-1431078592 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 33.94 seconds stream load tsv: 479 seconds loaded 74807831229 Bytes, about 148 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16785: [Opt](exec) Refactor the code and logical functions to SIMD the code

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16785: URL: https://github.com/apache/doris/pull/16785#issuecomment-1431080738 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] nextdreamblue opened a new pull request, #16786: [fix](subquery) fix bug of using constexpr and some agg func(like cou…

2023-02-15 Thread via GitHub
nextdreamblue opened a new pull request, #16786: URL: https://github.com/apache/doris/pull/16786 …nt,max) as subquery's output (#16579) # Proposed changes cherry-pick from #16579 ## Problem summary Describe your changes. ## Checklist(Required) * [ ] D

[GitHub] [doris] yorkexing commented on issue #16765: [Bug] construct-docker-image error

2023-02-15 Thread via GitHub
yorkexing commented on issue #16765: URL: https://github.com/apache/doris/issues/16765#issuecomment-1431117744 https://github.com/apache/doris/blob/master/docker/runtime/be/resource/init_be.sh 这个文件有问题 #!/bin/env bash 开始的 改为 #!/bin/bash 我是这么解决的 -- This is an automate

[GitHub] [doris] adonis0147 opened a new pull request, #16787: [chore](tools) Support starting multiple FEs on single node

2023-02-15 Thread via GitHub
adonis0147 opened a new pull request, #16787: URL: https://github.com/apache/doris/pull/16787 # Proposed changes ~~Issue Number: close #xxx~~ ## Problem summary Introduce a tool to start multiple FEs on single node. ## Use case ```shell $ ./multi-fe ./mul

[GitHub] [doris] yorkexing opened a new issue, #16788: [Bug] 1.2.1版本 构建 doris borker sh脚本问题

2023-02-15 Thread via GitHub
yorkexing opened a new issue, #16788: URL: https://github.com/apache/doris/issues/16788 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Version 1.2.1 ### What

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

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

[doris] branch master updated: fix bug, rename thread (#16780)

2023-02-15 Thread pengxiangyu
This is an automated email from the ASF dual-hosted git repository. pengxiangyu 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 fe9b2fb803 fix bug, rename thread (#16780) f

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

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16767: URL: https://github.com/apache/doris/pull/16767#issuecomment-1431150059 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 #16775: [enhance](util) Split PriorityWorkStealingThreadPool into impl cpp && modernize-pass-by-value blocking_put

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16775: URL: https://github.com/apache/doris/pull/16775#issuecomment-1431150346 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 #16775: [enhance](util) Split PriorityWorkStealingThreadPool into impl cpp && modernize-pass-by-value blocking_put

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16775: URL: https://github.com/apache/doris/pull/16775#issuecomment-1431150430 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] yangzhg commented on a diff in pull request #16571: [Enhancement](HttpServer) Add http interface authentication

2023-02-15 Thread via GitHub
yangzhg commented on code in PR #16571: URL: https://github.com/apache/doris/pull/16571#discussion_r1106974240 ## fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/BackendsAction.java: ## @@ -70,9 +67,6 @@ public class BackendsAction extends RestBaseController { @Reques

[GitHub] [doris] zddr opened a new pull request, #16789: [Enhance](ComputeNode) K8sDeployManager support computeNode

2023-02-15 Thread via GitHub
zddr opened a new pull request, #16789: URL: https://github.com/apache/doris/pull/16789 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. 1.allow have no ELECTABLE or BACKEND 2.add cn NodeType 3.delete deprecated co

[GitHub] [doris] hello-stephen commented on pull request #16779: [pipeline](shuffle) Improve broadcast shuffle

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16779: URL: https://github.com/apache/doris/pull/16779#issuecomment-1431163283 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.1 seconds stream load tsv: 474 seconds loaded 74807831229 Bytes, about 150 MB/s

[GitHub] [doris] TangSiyang2001 commented on issue #16741: [Bug] create table could create column with type 'ALL'

2023-02-15 Thread via GitHub
TangSiyang2001 commented on issue #16741: URL: https://github.com/apache/doris/issues/16741#issuecomment-1431178299 Job done, PTAL. -- 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 co

[GitHub] [doris] xy720 opened a new pull request, #16790: [regression](struct-type) add more regression tests for struct-type

2023-02-15 Thread via GitHub
xy720 opened a new pull request, #16790: URL: https://github.com/apache/doris/pull/16790 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [x] Has unit

[GitHub] [doris] github-actions[bot] commented on pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16676: URL: https://github.com/apache/doris/pull/16676#issuecomment-1431193275 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 #16676: [feature](Nereids): add OuterJoinAssoc rule

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16676: URL: https://github.com/apache/doris/pull/16676#issuecomment-1431193395 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 merged pull request #16676: [feature](Nereids): add OuterJoinAssoc rule

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

[doris] branch master updated: [feature](Nereids): add OuterJoinAssoc rule (#16676)

2023-02-15 Thread starocean999
This is an automated email from the ASF dual-hosted git repository. starocean999 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 ecadd4b392 [feature](Nereids): add OuterJoi

[GitHub] [doris-flink-connector] GoGoWen opened a new pull request, #102: [Fix](flink-connector)fix stream load exit unnormally as concurrency issue

2023-02-15 Thread via GitHub
GoGoWen opened a new pull request, #102: URL: https://github.com/apache/doris-flink-connector/pull/102 # Proposed changes Issue Number: close #100 ## Problem Summary: Describe the overview of changes. ## Checklist(Required) 1. Does it affect the original beh

[GitHub] [doris] morningman commented on a diff in pull request #16733: [feature](docker)Add Docker Broker Init Script

2023-02-15 Thread via GitHub
morningman commented on code in PR #16733: URL: https://github.com/apache/doris/pull/16733#discussion_r1107000991 ## docker/runtime/broker/resource/init_broker.sh: ## @@ -16,95 +16,178 @@ # specific language governing permissions and limitations # under the License. -FE_SERV

[GitHub] [doris] github-actions[bot] commented on pull request #16733: [feature](docker)Add Docker Broker Init Script

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16733: URL: https://github.com/apache/doris/pull/16733#issuecomment-1431209183 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 #16733: [feature](docker)Add Docker Broker Init Script

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

[doris] branch master updated (ecadd4b392 -> 611d9aca10)

2023-02-15 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from ecadd4b392 [feature](Nereids): add OuterJoinAssoc rule (#16676) add 611d9aca10 [feature](docker)Add Docker Bro

[GitHub] [doris] morningman merged pull request #16733: [feature](docker)Add Docker Broker Init Script

2023-02-15 Thread via GitHub
morningman merged PR #16733: URL: https://github.com/apache/doris/pull/16733 -- 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] FreeOnePlus opened a new pull request, #16791: [fix](docker)Fix Dockerfile logic

2023-02-15 Thread via GitHub
FreeOnePlus opened a new pull request, #16791: URL: https://github.com/apache/doris/pull/16791 # Proposed changes Error log logic fix. Remove Chinese annotations. ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the or

[GitHub] [doris-flink-connector] JNSimba commented on issue #100: [Bug] (concurrency) Thread safety problem of loading data to Doris

2023-02-15 Thread via GitHub
JNSimba commented on issue #100: URL: https://github.com/apache/doris-flink-connector/issues/100#issuecomment-1431240428 > But method DorisStreamLoad.handlePreCommitResponse() may be invoked by exception checker thread before loading is set to true by main thread. In the **checkdone

[GitHub] [doris] morrySnow opened a new pull request, #16792: Revert "[test](pipeline) Run nereids cases in p1/p2 (#16130)"

2023-02-15 Thread via GitHub
morrySnow opened a new pull request, #16792: URL: https://github.com/apache/doris/pull/16792 This reverts commit b480db2e119ac0516e8621ea3d53c40f250c1d24. remove nereids p1/p2 cases provisionally. add it back batch-wise # Proposed changes Issue Number: close #xxx #

[GitHub] [doris] github-actions[bot] commented on pull request #16792: Revert "[test](pipeline) Run nereids cases in p1/p2 (#16130)"

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16792: URL: https://github.com/apache/doris/pull/16792#issuecomment-1431253168 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 #16792: Revert "[test](pipeline) Run nereids cases in p1/p2 (#16130)"

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16792: URL: https://github.com/apache/doris/pull/16792#issuecomment-1431253219 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-flink-connector] GoGoWen commented on issue #100: [Bug] (concurrency) Thread safety problem of loading data to Doris

2023-02-15 Thread via GitHub
GoGoWen commented on issue #100: URL: https://github.com/apache/doris-flink-connector/issues/100#issuecomment-1431256720 this should be an issue like https://github.com/apache/doris-flink-connector/pull/102. -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [doris] github-actions[bot] commented on pull request #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1431267879 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] yixiutt opened a new pull request, #16793: [bugfix](reader) make segment_overlapping meta correct

2023-02-15 Thread via GitHub
yixiutt opened a new pull request, #16793: URL: https://github.com/apache/doris/pull/16793 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has un

[GitHub] [doris] github-actions[bot] commented on pull request #16793: [bugfix](reader) make segment_overlapping meta correct

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16793: URL: https://github.com/apache/doris/pull/16793#issuecomment-1431278191 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] hello-stephen commented on pull request #16783: [enhancement](nereids)add a session variable to control join reorder algorithm

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16783: URL: https://github.com/apache/doris/pull/16783#issuecomment-1431289438 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.87 seconds stream load tsv: 477 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

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16734: URL: https://github.com/apache/doris/pull/16734#issuecomment-1431297833 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] jackwener opened a new pull request, #16794: [fix](Nereids): fix Master Bors problem.

2023-02-15 Thread via GitHub
jackwener opened a new pull request, #16794: URL: https://github.com/apache/doris/pull/16794 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [x] Does it affect the original behavior * [ ] Has

[GitHub] [doris] luozenglin opened a new pull request, #16795: [fix](insert) fix be core dump due to using the wrong schema to create column writers

2023-02-15 Thread via GitHub
luozenglin opened a new pull request, #16795: URL: https://github.com/apache/doris/pull/16795 # Proposed changes Issue Number: close #16658 ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [

[GitHub] [doris] luzhijing opened a new pull request, #16796: [comunity](team) add new PMC members and committers

2023-02-15 Thread via GitHub
luzhijing opened a new pull request, #16796: URL: https://github.com/apache/doris/pull/16796 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has

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

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16779: URL: https://github.com/apache/doris/pull/16779#issuecomment-1431300374 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 #16779: [pipeline](shuffle) Improve broadcast shuffle

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16779: URL: https://github.com/apache/doris/pull/16779#issuecomment-1431300434 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 #16795: [fix](insert) fix be core dump due to using the wrong schema to create column writers

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16795: URL: https://github.com/apache/doris/pull/16795#issuecomment-1431303360 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 #16796: [comunity](team) add new PMC members and committers

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16796: URL: https://github.com/apache/doris/pull/16796#issuecomment-1431304125 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 #16796: [comunity](team) add new PMC members and committers

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16796: URL: https://github.com/apache/doris/pull/16796#issuecomment-1431304180 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] stalary closed issue #16749: [Enhancement](Es) Support array inner type in fe

2023-02-15 Thread via GitHub
stalary closed issue #16749: [Enhancement](Es) Support array inner type in fe URL: https://github.com/apache/doris/issues/16749 -- 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.

[GitHub] [doris] hello-stephen commented on pull request #16787: [chore](tools) Support starting multiple FEs on single node

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16787: URL: https://github.com/apache/doris/pull/16787#issuecomment-1431315412 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.52 seconds stream load tsv: 473 seconds loaded 74807831229 Bytes, about 150 MB/s

[GitHub] [doris-flink-connector] DarvenDuan commented on issue #100: [Bug] (concurrency) Thread safety problem of loading data to Doris

2023-02-15 Thread via GitHub
DarvenDuan commented on issue #100: URL: https://github.com/apache/doris-flink-connector/issues/100#issuecomment-1431320283 > > But method DorisStreamLoad.handlePreCommitResponse() may be invoked by exception checker thread before loading is set to true by main thread. > > In the **c

[GitHub] [doris] XieJiann opened a new pull request, #16797: [feature](Nereids): add rule split limit into two phase

2023-02-15 Thread via GitHub
XieJiann opened a new pull request, #16797: URL: https://github.com/apache/doris/pull/16797 # Proposed changes Add a rule split limit, like before: Limit(Origin) after: Limit(Global) -> Gather -> Limit(Local) And because this rule is always beneficial, we add a rule in rew

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

2023-02-15 Thread via GitHub
platoneko commented on code in PR #16767: URL: https://github.com/apache/doris/pull/16767#discussion_r1107093171 ## be/src/io/cache/block/block_file_cache.cpp: ## @@ -32,6 +32,9 @@ namespace fs = std::filesystem; namespace doris { namespace io { +const std::string IFileCache

[GitHub] [doris] hello-stephen commented on pull request #16785: [Opt](exec) Refactor the code and logical functions to SIMD the code

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16785: URL: https://github.com/apache/doris/pull/16785#issuecomment-1431333846 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.96 seconds stream load tsv: 484 seconds loaded 74807831229 Bytes, about 147 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16472: [improvement](meta) Enhance Doris's fault tolerance to disk error

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16472: URL: https://github.com/apache/doris/pull/16472#issuecomment-1431334078 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] yixiutt opened a new pull request, #16798: [doc](enhancement) fix some style

2023-02-15 Thread via GitHub
yixiutt opened a new pull request, #16798: URL: https://github.com/apache/doris/pull/16798 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has un

[GitHub] [doris] github-actions[bot] commented on pull request #16793: [bugfix](reader) make segment_overlapping meta correct

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16793: URL: https://github.com/apache/doris/pull/16793#issuecomment-1431368841 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 #16793: [bugfix](reader) make segment_overlapping meta correct

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16793: URL: https://github.com/apache/doris/pull/16793#issuecomment-1431368909 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] liaoxin01 opened a new pull request, #16799: [enhancement](merge-on-write) do compaction with merge on read

2023-02-15 Thread via GitHub
liaoxin01 opened a new pull request, #16799: URL: https://github.com/apache/doris/pull/16799 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has

[GitHub] [doris] github-actions[bot] commented on pull request #16799: [enhancement](merge-on-write) do compaction with merge on read

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16799: URL: https://github.com/apache/doris/pull/16799#issuecomment-1431377043 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] hello-stephen commented on pull request #16789: [Enhance](ComputeNode) K8sDeployManager support computeNode

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16789: URL: https://github.com/apache/doris/pull/16789#issuecomment-1431377602 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.9 seconds stream load tsv: 479 seconds loaded 74807831229 Bytes, about 148 MB/s

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

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1431387966 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 #16785: [Opt](exec) Refactor the code and logical functions to SIMD the code

2023-02-15 Thread via GitHub
github-actions[bot] commented on PR #16785: URL: https://github.com/apache/doris/pull/16785#issuecomment-1431406984 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] hello-stephen commented on pull request #16793: [bugfix](reader) make segment_overlapping meta correct

2023-02-15 Thread via GitHub
hello-stephen commented on PR #16793: URL: https://github.com/apache/doris/pull/16793#issuecomment-1431408276 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 36.88 seconds stream load tsv: 471 seconds loaded 74807831229 Bytes, about 151 MB/s

[GitHub] [doris] HappenLee commented on a diff in pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-15 Thread via GitHub
HappenLee commented on code in PR #16383: URL: https://github.com/apache/doris/pull/16383#discussion_r1107166746 ## be/src/exec/data_sink.cpp: ## @@ -177,6 +177,146 @@ Status DataSink::create_data_sink(ObjectPool* pool, const TDataSink& thrift_sink return Status::OK(); }

[GitHub] [doris] Gabriel39 merged pull request #16779: [pipeline](shuffle) Improve broadcast shuffle

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

[doris] branch master updated (611d9aca10 -> dd06cc7609)

2023-02-15 Thread gabriellee
This is an automated email from the ASF dual-hosted git repository. gabriellee pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 611d9aca10 [feature](docker)Add Docker Broker Init Script (#16733) add dd06cc7609 [pipeline](shuffle) Improve

[GitHub] [doris] HappenLee commented on a diff in pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-15 Thread via GitHub
HappenLee commented on code in PR #16383: URL: https://github.com/apache/doris/pull/16383#discussion_r1107167131 ## be/src/pipeline/pipeline_fragment_context.cpp: ## @@ -272,6 +273,139 @@ Status PipelineFragmentContext::prepare(const doris::TExecPlanFragmentParams& re retu

[GitHub] [doris] HappenLee commented on a diff in pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-15 Thread via GitHub
HappenLee commented on code in PR #16383: URL: https://github.com/apache/doris/pull/16383#discussion_r1107171484 ## gensrc/thrift/PaloInternalService.thrift: ## @@ -553,3 +553,45 @@ struct TExportStatusResult { 2: required Types.TExportState state 3: optional list file

[GitHub] [doris] HappenLee commented on a diff in pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-15 Thread via GitHub
HappenLee commented on code in PR #16383: URL: https://github.com/apache/doris/pull/16383#discussion_r1107171836 ## gensrc/thrift/PaloInternalService.thrift: ## @@ -553,3 +553,45 @@ struct TExportStatusResult { 2: required Types.TExportState state 3: optional list file

[GitHub] [doris] HappenLee commented on a diff in pull request #16383: [Improvement](thrift) optimize thrift messages

2023-02-15 Thread via GitHub
HappenLee commented on code in PR #16383: URL: https://github.com/apache/doris/pull/16383#discussion_r1107171836 ## gensrc/thrift/PaloInternalService.thrift: ## @@ -553,3 +553,45 @@ struct TExportStatusResult { 2: required Types.TExportState state 3: optional list file

[GitHub] [doris] github-actions[bot] commented on pull request #15509: [feature](partition)add default list partition

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

  1   2   3   >