[GitHub] [doris] morningman commented on a diff in pull request #16602: [feature-wip](iceberg) add dlf catalog impl for iceberg catalog

2023-02-16 Thread via GitHub
morningman commented on code in PR #16602: URL: https://github.com/apache/doris/pull/16602#discussion_r1108127786 ## fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/dlf/DLFCatalog.java: ## @@ -0,0 +1,224 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [doris] morningman merged pull request #16767: [improvement](filecache) split file cache into sharding directories

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

[doris] branch master updated: [improvement](filecache) split file cache into sharding directories (#16767)

2023-02-16 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman 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 e2245cbdd3 [improvement](filecache) split fil

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16734: URL: https://github.com/apache/doris/pull/16734#issuecomment-1432676098 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 #16818: [Improvement](parquet-reader) Optimize and refactor parquet reader to improve performance.

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16785: URL: https://github.com/apache/doris/pull/16785#issuecomment-1432681163 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] morningman merged pull request #16791: [fix](docker)Fix Dockerfile logic

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

[doris] branch master updated (e2245cbdd3 -> ce7791c362)

2023-02-16 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 e2245cbdd3 [improvement](filecache) split file cache into sharding directories (#16767) add ce7791c362 [fix](

[GitHub] [doris] HappenLee merged pull request #16814: [Bug](Datetime) Fix date time function mem use after free

2023-02-16 Thread via GitHub
HappenLee merged PR #16814: URL: https://github.com/apache/doris/pull/16814 -- 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: [Bug](Datetime) Fix date time function mem use after free (#16814)

2023-02-16 Thread lihaopeng
This is an automated email from the ASF dual-hosted git repository. lihaopeng 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 de1337511c [Bug](Datetime) Fix date time funct

[GitHub] [doris] zhannngchen commented on a diff in pull request #16799: [enhancement](merge-on-write) do compaction with merge on read

2023-02-16 Thread via GitHub
zhannngchen commented on code in PR #16799: URL: https://github.com/apache/doris/pull/16799#discussion_r1108144748 ## be/src/olap/compaction.cpp: ## @@ -413,19 +413,36 @@ Status Compaction::construct_input_rowset_readers() { Status Compaction::modify_rowsets() { std::vecto

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16799: URL: https://github.com/apache/doris/pull/16799#issuecomment-1432691373 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 #16799: [enhancement](merge-on-write) do compaction with merge on read

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

2023-02-16 Thread via GitHub
zddr commented on code in PR #16789: URL: https://github.com/apache/doris/pull/16789#discussion_r1108148176 ## fe/fe-core/src/main/java/org/apache/doris/deploy/DeployManager.java: ## @@ -148,38 +146,51 @@ public DeployManager(Env env, long intervalMs) { // Derived Class can

[GitHub] [doris] wsjz commented on a diff in pull request #16602: [feature-wip](iceberg) add dlf catalog impl for iceberg catalog

2023-02-16 Thread via GitHub
wsjz commented on code in PR #16602: URL: https://github.com/apache/doris/pull/16602#discussion_r1108148667 ## fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/dlf/DLFCatalog.java: ## @@ -0,0 +1,224 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16772: URL: https://github.com/apache/doris/pull/16772#issuecomment-1432697177 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] wsjz commented on a diff in pull request #16602: [feature-wip](iceberg) add dlf catalog impl for iceberg catalog

2023-02-16 Thread via GitHub
wsjz commented on code in PR #16602: URL: https://github.com/apache/doris/pull/16602#discussion_r1108150641 ## fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/dlf/DLFCatalog.java: ## @@ -0,0 +1,224 @@ +// Licensed to the Apache Software Foundation (ASF) under one +/

[GitHub] [doris] eldenmoon opened a new pull request, #16827: [chore](be-config) set disable_storage_row_cache default true to def…

2023-02-16 Thread via GitHub
eldenmoon opened a new pull request, #16827: URL: https://github.com/apache/doris/pull/16827 …ault disable row cache # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the

[GitHub] [doris] yixiutt opened a new pull request, #16828: [pick](bugfix)pick 16793

2023-02-16 Thread via GitHub
yixiutt opened a new pull request, #16828: URL: https://github.com/apache/doris/pull/16828 # 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 #16827: [chore](be-config) set disable_storage_row_cache default true to def…

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16827: URL: https://github.com/apache/doris/pull/16827#issuecomment-1432704955 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 #16827: [chore](be-config) set disable_storage_row_cache default true to def…

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16827: URL: https://github.com/apache/doris/pull/16827#issuecomment-1432705479 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 #16815: [regression pipeline]add git action to trigger teamcity pipeline 0216

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16815: URL: https://github.com/apache/doris/pull/16815#issuecomment-1432710749 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 #16815: [regression pipeline]add git action to trigger teamcity pipeline 0216

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16815: URL: https://github.com/apache/doris/pull/16815#issuecomment-1432710791 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] qidaye commented on a diff in pull request #16735: [doc](point query) modify and refine docs

2023-02-16 Thread via GitHub
qidaye commented on code in PR #16735: URL: https://github.com/apache/doris/pull/16735#discussion_r1108167002 ## docs/en/docs/advanced/hight-concurrent-point-query.md: ## @@ -0,0 +1,60 @@ +# High-concurrency point query based on primary key Review Comment: Please add header

[GitHub] [doris] dataroaring merged pull request #16815: [regression pipeline]add git action to trigger teamcity pipeline 0216

2023-02-16 Thread via GitHub
dataroaring merged PR #16815: URL: https://github.com/apache/doris/pull/16815 -- 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: [improvement][regression] add git action to trigger teamcity pipeline 0216 (#16815)

2023-02-16 Thread dataroaring
This is an automated email from the ASF dual-hosted git repository. dataroaring 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 a56f1ca6b6 [improvement][regression] add git

[GitHub] [doris] qidaye commented on a diff in pull request #16735: [doc](point query) modify and refine docs

2023-02-16 Thread via GitHub
qidaye commented on code in PR #16735: URL: https://github.com/apache/doris/pull/16735#discussion_r1108173773 ## docs/en/docs/data-table/best-practice.md: ## @@ -180,62 +180,4 @@ Users can modify the Schema of an existing table through the Schema Change opera - Adding or modif

[GitHub] [doris] morningman merged pull request #16828: [bugfix](reader) make segment_overlapping meta correct

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

[doris] branch branch-1.2-lts updated: [bugfix](reader) make segment_overlapping meta correct (#16828)

2023-02-16 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a commit to branch branch-1.2-lts in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/branch-1.2-lts by this push: new 306e004ece [bugfix](reader) m

[GitHub] [doris] ZhangGuoqiang666 commented on pull request #16815: [regression pipeline]add git action to trigger teamcity pipeline 0216

2023-02-16 Thread via GitHub
ZhangGuoqiang666 commented on PR #16815: URL: https://github.com/apache/doris/pull/16815#issuecomment-1432726741 buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16772: URL: https://github.com/apache/doris/pull/16772#issuecomment-1432728081 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] hf200012 merged pull request #16768: [fix](docs) Fix broken graph in en/broker-load-manual.md

2023-02-16 Thread via GitHub
hf200012 merged PR #16768: URL: https://github.com/apache/doris/pull/16768 -- 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: fix broken graph in broker-load-manual.md (#16768)

2023-02-16 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng 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 2f3ce39857 fix broken graph in broker-load-

[GitHub] [doris] HappenLee merged pull request #16785: [Opt](exec) Refactor the code and logical functions to SIMD the code

2023-02-16 Thread via GitHub
HappenLee merged PR #16785: URL: https://github.com/apache/doris/pull/16785 -- 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 (2f3ce39857 -> f08c1222cc)

2023-02-16 Thread lihaopeng
This is an automated email from the ASF dual-hosted git repository. lihaopeng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 2f3ce39857 fix broken graph in broker-load-manual.md (#16768) add f08c1222cc [Opt](exec) Refactor the code and

[GitHub] [doris] yunaisheng opened a new issue, #16829: [Feature] Partial column updates on aggregate model table include null value

2023-02-16 Thread via GitHub
yunaisheng opened a new issue, #16829: URL: https://github.com/apache/doris/issues/16829 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Description We require partia

[GitHub] [doris] dinggege1024 opened a new issue, #16830: [Bug] release not link 404

2023-02-16 Thread via GitHub
dinggege1024 opened a new issue, #16830: URL: https://github.com/apache/doris/issues/16830 ### 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 all ### What

[GitHub] [doris] BiteTheDDDDt opened a new pull request, #16831: [Chore](build) remove memory_copy and remove some wno cbuild check

2023-02-16 Thread via GitHub
BiteThet opened a new pull request, #16831: URL: https://github.com/apache/doris/pull/16831 # Proposed changes 1. remove memory_copy, memory_copy is not faster than memcpy now. https://user-images.githubusercontent.com/7939630/219318104-60d46380-8695-4c4f-ba59-fced0f375598.png";>

[GitHub] [doris] github-actions[bot] commented on pull request #16816: [improve](dynamic table) refine SegmentWriter columns writer generate

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16816: URL: https://github.com/apache/doris/pull/16816#issuecomment-1432747613 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 #16831: [Chore](build) remove memory_copy and remove some wno cbuild check

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16831: URL: https://github.com/apache/doris/pull/16831#issuecomment-1432751616 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-16 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1432752005 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 #16831: [Chore](build) remove memory_copy and remove some wno cbuild check

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16831: URL: https://github.com/apache/doris/pull/16831#issuecomment-1432754179 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 #16735: [doc](point query) modify and refine docs

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16735: URL: https://github.com/apache/doris/pull/16735#issuecomment-1432765351 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 #16735: [doc](point query) modify and refine docs

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16735: URL: https://github.com/apache/doris/pull/16735#issuecomment-1432765413 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 #16772: [pipeline](profile) Support real-time profile report in pipeline

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16772: URL: https://github.com/apache/doris/pull/16772#issuecomment-1432775821 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 #16772: [pipeline](profile) Support real-time profile report in pipeline

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16772: URL: https://github.com/apache/doris/pull/16772#issuecomment-1432775925 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 #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16812: URL: https://github.com/apache/doris/pull/16812#issuecomment-1432777114 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 #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16812: URL: https://github.com/apache/doris/pull/16812#issuecomment-1432778631 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] qidaye merged pull request #16735: [doc](point query) modify and refine docs

2023-02-16 Thread via GitHub
qidaye merged PR #16735: URL: https://github.com/apache/doris/pull/16735 -- 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.apach

[doris] branch master updated: [doc](point query) modify and refine docs (#16735)

2023-02-16 Thread jianliangqi
This is an automated email from the ASF dual-hosted git repository. jianliangqi 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 262a2ea10d [doc](point query) modify and ref

[GitHub] [doris] github-actions[bot] commented on pull request #16831: [Chore](build) remove memory_copy and remove some wno build check

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16831: URL: https://github.com/apache/doris/pull/16831#issuecomment-1432792339 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 #16831: [Chore](build) remove memory_copy and remove some wno build check

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16831: URL: https://github.com/apache/doris/pull/16831#issuecomment-1432792401 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] chioWong commented on issue #16403: [Bug] StreamLoad sink.enable-2pc设置不生效

2023-02-16 Thread via GitHub
chioWong commented on issue #16403: URL: https://github.com/apache/doris/issues/16403#issuecomment-1432801239 > 据的可见性依赖于checkpoint,将sink.enable-2pc设置为false时,数据最终的submit依然依赖于checkpoint > > ### What You Expected? > StreamLoad可以由用户控制数据的提交间隔 > > ### How to Reproduce? Doris

[GitHub] [doris] github-actions[bot] commented on pull request #14531: [improvement](meta) make database,table,column names to support unicode (replace PR #13467 with this)

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #14531: URL: https://github.com/apache/doris/pull/14531#issuecomment-1432801489 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] JackDan9 commented on issue #6227: Should a docker image be provided to facilitate deployment?

2023-02-16 Thread via GitHub
JackDan9 commented on issue #6227: URL: https://github.com/apache/doris/issues/6227#issuecomment-1432802615 确实应该是,现在做镜像进去部署太费劲了 -- 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 commen

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on code in PR #16812: URL: https://github.com/apache/doris/pull/16812#discussion_r1108241611 ## be/src/olap/tablet.cpp: ## @@ -1807,13 +1798,32 @@ Status Tablet::_read_cooldown_meta(io::RemoteFileSystem* fs, int64_t cooldown_re return Status::

[GitHub] [doris-website] jeffreys-cat opened a new pull request, #191: [PDF] add doris log on the cover

2023-02-16 Thread via GitHub
jeffreys-cat opened a new pull request, #191: URL: https://github.com/apache/doris-website/pull/191 - [PDF] add doris log on the cover -- 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] morrySnow commented on a diff in pull request #16797: [feature](Nereids): add rule split limit into two phase

2023-02-16 Thread via GitHub
morrySnow commented on code in PR #16797: URL: https://github.com/apache/doris/pull/16797#discussion_r1108242007 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -1338,38 +1338,15 @@ public PlanFragment visitPhysicalLimit(Ph

[GitHub] [doris] github-actions[bot] commented on pull request #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16812: URL: https://github.com/apache/doris/pull/16812#issuecomment-1432808796 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-website] morningman merged pull request #191: [PDF] add doris logo on the cover

2023-02-16 Thread via GitHub
morningman merged PR #191: URL: https://github.com/apache/doris-website/pull/191 -- 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...@dor

[doris-website] branch master updated: [PDF] add doris logo on the cover (#191)

2023-02-16 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git The following commit(s) were added to refs/heads/master by this push: new 9ac69cc64e [PDF] add doris logo on th

[GitHub] [doris] github-actions[bot] commented on pull request #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16812: URL: https://github.com/apache/doris/pull/16812#issuecomment-1432812507 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] liaoxin01 opened a new pull request, #16832: [fix](merge-on-write) fix that be may coredump when sequence column is null

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

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16776: URL: https://github.com/apache/doris/pull/16776#issuecomment-1432824123 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] liaoxin01 opened a new issue, #16833: [Bug] (merge-on-write)be may coredump when sequence column is null

2023-02-16 Thread via GitHub
liaoxin01 opened a new issue, #16833: URL: https://github.com/apache/doris/issues/16833 ### 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 master ### What

[GitHub] [doris] github-actions[bot] commented on pull request #16832: [fix](merge-on-write) fix that be may coredump when sequence column is null

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16832: URL: https://github.com/apache/doris/pull/16832#issuecomment-1432828022 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] stalary commented on a diff in pull request #16806: [Enhancement](ES): Support mapping es date format

2023-02-16 Thread via GitHub
stalary commented on code in PR #16806: URL: https://github.com/apache/doris/pull/16806#discussion_r1108262190 ## fe/fe-core/src/main/java/org/apache/doris/external/elasticsearch/EsUtil.java: ## @@ -419,6 +421,69 @@ public static List genColumnsFromEs(EsRestClient client, Strin

[GitHub] [doris] stalary commented on pull request #16806: [Enhancement](ES): Support mapping es date format

2023-02-16 Thread via GitHub
stalary commented on PR #16806: URL: https://github.com/apache/doris/pull/16806#issuecomment-1432830341 > should handle this issue: > > ```sql > // if es data fomat is default or long > select * from es_table where date_col = "2023-02-16 12:12:12" > ``` Ok, let me handl

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

2023-02-16 Thread via GitHub
JNSimba commented on issue #100: URL: https://github.com/apache/doris-flink-connector/issues/100#issuecomment-1432848516 > When doing checkpoint, method dorisStreamLoad.startLoad() will be invoked and changes pendingLoadFuture, but in checkDone() dorisStreamLoad.getPendingLoadFuture() may

[GitHub] [doris] pengxiangyu commented on a diff in pull request #16803: [fix](cooldown) Use `pending_remote_rowsets` to avoid deleting rowset files being uploaded

2023-02-16 Thread via GitHub
pengxiangyu commented on code in PR #16803: URL: https://github.com/apache/doris/pull/16803#discussion_r1108292224 ## be/src/olap/tablet.h: ## @@ -328,7 +328,11 @@ class Tablet : public BaseTablet { static void remove_unused_remote_files(); -std::shared_mutex& get_r

[GitHub] [doris] yongjinhou opened a new pull request, #16834: [Enhancement](HttpServer) Support https interface

2023-02-16 Thread via GitHub
yongjinhou opened a new pull request, #16834: URL: https://github.com/apache/doris/pull/16834 # Proposed changes Issue Number: close #15658 ## Problem summary Support https interface. ## Checklist(Required) * [x] Does it affect the original behavior * [ ]

[GitHub] [doris] pengxiangyu commented on a diff in pull request #16803: [fix](cooldown) Use `pending_remote_rowsets` to avoid deleting rowset files being uploaded

2023-02-16 Thread via GitHub
pengxiangyu commented on code in PR #16803: URL: https://github.com/apache/doris/pull/16803#discussion_r1108302426 ## be/src/olap/tablet.cpp: ## @@ -1768,7 +1765,10 @@ Status Tablet::_cooldown_data(const std::shared_ptr& dest_ UniqueId cooldown_meta_id = UniqueId::gen_uid(

[GitHub] [doris] pengxiangyu commented on a diff in pull request #16803: [fix](cooldown) Use `pending_remote_rowsets` to avoid deleting rowset files being uploaded

2023-02-16 Thread via GitHub
pengxiangyu commented on code in PR #16803: URL: https://github.com/apache/doris/pull/16803#discussion_r1108308600 ## be/src/olap/tablet.cpp: ## @@ -1768,7 +1765,10 @@ Status Tablet::_cooldown_data(const std::shared_ptr& dest_ UniqueId cooldown_meta_id = UniqueId::gen_uid(

[GitHub] [doris] XieJiann commented on a diff in pull request #16797: [feature](Nereids): add rule split limit into two phase

2023-02-16 Thread via GitHub
XieJiann commented on code in PR #16797: URL: https://github.com/apache/doris/pull/16797#discussion_r1108309568 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalLimit.java: ## @@ -44,19 +45,28 @@ * offset 100 */ public class LogicalLimit exten

[GitHub] [doris] XieJiann commented on a diff in pull request #16797: [feature](Nereids): add rule split limit into two phase

2023-02-16 Thread via GitHub
XieJiann commented on code in PR #16797: URL: https://github.com/apache/doris/pull/16797#discussion_r1108310075 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -1338,38 +1338,15 @@ public PlanFragment visitPhysicalLimit(Phy

[GitHub] [doris] ZhangGuoqiang666 opened a new issue, #16835: [Enhancement] change the teamcity pipeline trigger : triggered by github pull request comment

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

[GitHub] [doris] XieJiann commented on a diff in pull request #16797: [feature](Nereids): add rule split limit into two phase

2023-02-16 Thread via GitHub
XieJiann commented on code in PR #16797: URL: https://github.com/apache/doris/pull/16797#discussion_r1108310075 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -1338,38 +1338,15 @@ public PlanFragment visitPhysicalLimit(Phy

[GitHub] [doris] XieJiann commented on a diff in pull request #16797: [feature](Nereids): add rule split limit into two phase

2023-02-16 Thread via GitHub
XieJiann commented on code in PR #16797: URL: https://github.com/apache/doris/pull/16797#discussion_r1108310075 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -1338,38 +1338,15 @@ public PlanFragment visitPhysicalLimit(Phy

[GitHub] [doris] github-actions[bot] commented on pull request #16803: [fix](cooldown) Use `pending_remote_rowsets` to avoid deleting rowset files being uploaded

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16803: URL: https://github.com/apache/doris/pull/16803#issuecomment-1432905995 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 #16803: [fix](cooldown) Use `pending_remote_rowsets` to avoid deleting rowset files being uploaded

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16803: URL: https://github.com/apache/doris/pull/16803#issuecomment-1432906074 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 #16812: [fix](cooldown)Fix bug for single cooldown compaction, add remote meta

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16812: URL: https://github.com/apache/doris/pull/16812#issuecomment-1432913774 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] ZhangGuoqiang666 opened a new pull request, #16836: [enhancement] change the teamcity pipeline trigger : triggered by github pull request comment

2023-02-16 Thread via GitHub
ZhangGuoqiang666 opened a new pull request, #16836: URL: https://github.com/apache/doris/pull/16836 # Proposed changes Issue Number: close #16835 ## Problem summary Describe your changes. Optimized some code and Reduce invalid code,fix syntax error ## Checklist

[GitHub] [doris] hello-stephen commented on pull request #16831: [Chore](build) remove memory_copy and remove some wno build check

2023-02-16 Thread via GitHub
hello-stephen commented on PR #16831: URL: https://github.com/apache/doris/pull/16831#issuecomment-1432925543 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.83 seconds stream load tsv: 478 seconds loaded 74807831229 Bytes, about 149 MB/s

[GitHub] [doris] yiguolei merged pull request #16799: [enhancement](merge-on-write) do compaction with merge on read

2023-02-16 Thread via GitHub
yiguolei merged PR #16799: URL: https://github.com/apache/doris/pull/16799 -- 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: [enhancement](merge-on-write) do compaction with merge on read (#16799)

2023-02-16 Thread yiguolei
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 2a9e748073 [enhancement](merge-on-write) do com

[GitHub] [doris] yiguolei commented on a diff in pull request #16822: [improvement](memory) fix possible memory leak of vcollect iterator

2023-02-16 Thread via GitHub
yiguolei commented on code in PR #16822: URL: https://github.com/apache/doris/pull/16822#discussion_r1108342805 ## be/src/vec/olap/vcollect_iterator.cpp: ## @@ -166,15 +158,17 @@ Status VCollectIterator::build_heap(std::vector& rs_reade ++iter; }

[GitHub] [doris] catpineapple opened a new pull request, #16837: [fix](dbt)fix dbt incremental

2023-02-16 Thread via GitHub
catpineapple opened a new pull request, #16837: URL: https://github.com/apache/doris/pull/16837 # Proposed changes Issue Number: close #xxx ## Problem summary 1. fix dbt incremental :new ideas for no rollback and support incremental data rerun . 2. add snapshot 3.

[GitHub] [doris] yiguolei merged pull request #16568: [Improve](regression) Add timeout check in schema change regression

2023-02-16 Thread via GitHub
yiguolei merged PR #16568: URL: https://github.com/apache/doris/pull/16568 -- 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: [Improve](regression) Add timeout check in schema change regression (#16568)

2023-02-16 Thread yiguolei
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 3c641d4465 [Improve](regression) Add timeout ch

[GitHub] [doris] catpineapple closed pull request #14767: [fix](dbt)fix dbt incremental

2023-02-16 Thread via GitHub
catpineapple closed pull request #14767: [fix](dbt)fix dbt incremental URL: https://github.com/apache/doris/pull/14767 -- 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 unsub

[GitHub] [doris] catpineapple commented on pull request #14767: [fix](dbt)fix dbt incremental

2023-02-16 Thread via GitHub
catpineapple commented on PR #14767: URL: https://github.com/apache/doris/pull/14767#issuecomment-1432944005 new pull request https://github.com/apache/doris/pull/16837 replace this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [doris] airborne12 opened a new issue, #16838: [Bug](inverted index) searching inverted index of numeric array column returns error result

2023-02-16 Thread via GitHub
airborne12 opened a new issue, #16838: URL: https://github.com/apache/doris/issues/16838 ### 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 master ### Wha

[GitHub] [doris] airborne12 opened a new pull request, #16839: [Fix](inverted index) fix array inverted index error match result when doing schema change add index

2023-02-16 Thread via GitHub
airborne12 opened a new pull request, #16839: URL: https://github.com/apache/doris/pull/16839 # Proposed changes Issue Number: close #16838 ## Problem summary There is a bug in inverted_index_writer when adding multiple lines array values' index. This problem can cau

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

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16771: URL: https://github.com/apache/doris/pull/16771#issuecomment-1432950645 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 #16839: [Fix](inverted index) fix array inverted index error match result when doing schema change add index

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16839: URL: https://github.com/apache/doris/pull/16839#issuecomment-1432953497 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 #16836: [enhancement] change the teamcity pipeline trigger : triggered by github pull request comment

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16836: URL: https://github.com/apache/doris/pull/16836#issuecomment-1432965933 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 #16836: [enhancement] change the teamcity pipeline trigger : triggered by github pull request comment

2023-02-16 Thread via GitHub
github-actions[bot] commented on PR #16836: URL: https://github.com/apache/doris/pull/16836#issuecomment-1432965986 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] morningman closed issue #16835: [Enhancement] change the teamcity pipeline trigger : triggered by github pull request comment

2023-02-16 Thread via GitHub
morningman closed issue #16835: [Enhancement] change the teamcity pipeline trigger : triggered by github pull request comment URL: https://github.com/apache/doris/issues/16835 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[doris] branch master updated (3c641d4465 -> f86e8ec395)

2023-02-16 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 3c641d4465 [Improve](regression) Add timeout check in schema change regression (#16568) add f86e8ec395 [enhan

  1   2   3   4   >