[GitHub] [doris] github-actions[bot] commented on pull request #16742: [fix](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets of cold data compaction

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16742: URL: https://github.com/apache/doris/pull/16742#issuecomment-1429287902 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 #16742: [fix](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets of cold data compaction

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16742: URL: https://github.com/apache/doris/pull/16742#issuecomment-1429287784 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] wangbo commented on a diff in pull request #16459: [fix](planner)Fix colocate query failed

2023-02-14 Thread via GitHub
wangbo commented on code in PR #16459: URL: https://github.com/apache/doris/pull/16459#discussion_r1105427246 ## fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java: ## @@ -321,6 +321,10 @@ public void useBaseIndexId() { this.selectedIndexId = olapTable.

[GitHub] [doris] xiaojunjie opened a new issue, #16747: [Bug] There is a problem with NamedExpressionUtil.newExprId when trace enable

2023-02-14 Thread via GitHub
xiaojunjie opened a new issue, #16747: URL: https://github.com/apache/doris/issues/16747 ### 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] BiteTheDDDDt opened a new pull request, #16748: [Chore](build) enable fallthrough check annd fix some fallthrough bug

2023-02-14 Thread via GitHub
BiteThet opened a new pull request, #16748: URL: https://github.com/apache/doris/pull/16748 # Proposed changes 1. enable fallthrough check 2. fix some fallthrough bug 3. remove or modify some unused code ## Problem summary Describe your changes. ## Checkl

[doris] branch master updated (4a6fd7cc30 -> ea78184551)

2023-02-14 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 4a6fd7cc30 [feature](docker) Add Docker BE computer node Interface (#16630) add ea78184551 [Feature](Materiali

[GitHub] [doris] HappenLee merged pull request #16378: [Feature](Materialized-View) support multiple slot on one column in materialized view

2023-02-14 Thread via GitHub
HappenLee merged PR #16378: URL: https://github.com/apache/doris/pull/16378 -- 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] TangSiyang2001 commented on issue #16741: [Bug] create table could create column with type 'ALL'

2023-02-14 Thread via GitHub
TangSiyang2001 commented on issue #16741: URL: https://github.com/apache/doris/issues/16741#issuecomment-1429304124 I'll try to fix it. -- 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 specifi

[GitHub] [doris] stalary opened a new issue, #16749: [Enhancement](Es) Support array inner type in fe

2023-02-14 Thread via GitHub
stalary opened a new issue, #16749: URL: https://github.com/apache/doris/issues/16749 ### 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 Support array inner

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16748: URL: https://github.com/apache/doris/pull/16748#issuecomment-1429307868 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] morrySnow commented on a diff in pull request #16411: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin

2023-02-14 Thread via GitHub
morrySnow commented on code in PR #16411: URL: https://github.com/apache/doris/pull/16411#discussion_r1105428879 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java: ## @@ -101,6 +101,9 @@ public enum RuleType { ELIMINATE_GROUP_BY_CONSTANT(RuleTypeClass

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16661: [Enhencement](CsvReader) Optimize CsvReader

2023-02-14 Thread via GitHub
github-actions[bot] commented on code in PR #16661: URL: https://github.com/apache/doris/pull/16661#discussion_r1105458194 ## be/src/vec/exec/format/file_reader/new_plain_text_line_reader.h: ## @@ -88,6 +104,25 @@ class NewPlainTextLineReader : public LineReader { size_t

[GitHub] [doris] morrySnow commented on a diff in pull request #16459: [fix](planner)Fix colocate query failed

2023-02-14 Thread via GitHub
morrySnow commented on code in PR #16459: URL: https://github.com/apache/doris/pull/16459#discussion_r1105458743 ## fe/fe-core/src/main/java/org/apache/doris/planner/DistributedPlanner.java: ## @@ -539,6 +539,14 @@ private boolean dataDistributionMatchEqPredicate(List eqJoinPre

[GitHub] [doris] BiteTheDDDDt opened a new pull request, #16750: [Feature](Materialized-View) change mv rewrite from bottom up to up bottom && Compatible with old …

2023-02-14 Thread via GitHub
BiteThet opened a new pull request, #16750: URL: https://github.com/apache/doris/pull/16750 # Proposed changes 1. change mv rewrite from bottom up to up bottom 2. compatible with old version mv 3. restore some ut codes (but disable) ## Problem summary Describe y

[GitHub] [doris] yagagagaga opened a new pull request, #16751: [typo](docs) Fix some misspelled words

2023-02-14 Thread via GitHub
yagagagaga opened a new pull request, #16751: URL: https://github.com/apache/doris/pull/16751 # Proposed changes Issue Number: close #xxx ## Problem summary I found some misspellings in the docs, and I fixed it. ## Checklist(Required) * [ ] Does it affect th

[doris] branch master updated (ea78184551 -> 784c27deeb)

2023-02-14 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 ea78184551 [Feature](Materialized-View) support multiple slot on one column in materialized view (#16378) add

[GitHub] [doris] Gabriel39 merged pull request #16685: [Bug](shuffle) fix mem leak in data stream sender

2023-02-14 Thread via GitHub
Gabriel39 merged PR #16685: URL: https://github.com/apache/doris/pull/16685 -- 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 #16383: [Improvement](thrift) optimize thrift message

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16383: URL: https://github.com/apache/doris/pull/16383#issuecomment-1429335860 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 commented on a diff in pull request #16411: [feature](Nereids): Infer isNotNull from filter and eliminate OuterJoin

2023-02-14 Thread via GitHub
jackwener commented on code in PR #16411: URL: https://github.com/apache/doris/pull/16411#discussion_r1105477872 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/RuleType.java: ## @@ -101,6 +101,9 @@ public enum RuleType { ELIMINATE_GROUP_BY_CONSTANT(RuleTypeClass

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

2023-02-14 Thread via GitHub
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

[GitHub] [doris] github-actions[bot] commented on pull request #16751: [typo](docs) Fix some misspelled words

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16751: URL: https://github.com/apache/doris/pull/16751#issuecomment-1429349569 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 #16751: [typo](docs) Fix some misspelled words

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16751: URL: https://github.com/apache/doris/pull/16751#issuecomment-1429349640 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] hello-stephen commented on pull request #16737: [Enhance](ComputeNode) change logic of BeSelectionPolicy.getCandidateBackends

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

[GitHub] [doris] starocean999 merged pull request #16436: [enhancement](Nereids): avoid contruct groupExpr in graph-simplifier

2023-02-14 Thread via GitHub
starocean999 merged PR #16436: URL: https://github.com/apache/doris/pull/16436 -- 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: avoid contruct groupExpr in graph-simplifier (#16436)

2023-02-14 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 abc828 avoid contruct groupExpr in grap

[GitHub] [doris] pengxiangyu opened a new pull request, #16752: [fix](cooldown)move remove_unused_remote_files

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

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

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

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16752: [fix](cooldown)move remove_unused_remote_files

2023-02-14 Thread via GitHub
github-actions[bot] commented on code in PR #16752: URL: https://github.com/apache/doris/pull/16752#discussion_r1105512142 ## be/src/olap/tablet_manager.cpp: ## @@ -1372,4 +1372,161 @@ std::set TabletManager::check_all_tablet_segment(bool repair) { return bad_tablets; }

[GitHub] [doris] BiteTheDDDDt commented on a diff in pull request #16734: [Opt](exec) opt aggreate function performance in nullable column

2023-02-14 Thread via GitHub
BiteThet commented on code in PR #16734: URL: https://github.com/apache/doris/pull/16734#discussion_r1105512439 ## be/src/vec/aggregate_functions/helpers.h: ## @@ -125,70 +123,32 @@ static IAggregateFunction* create_with_numeric_type(const IDataType& argument_ty re

[GitHub] [doris] BiteTheDDDDt commented on a diff in pull request #16734: [Opt](exec) opt aggreate function performance in nullable column

2023-02-14 Thread via GitHub
BiteThet commented on code in PR #16734: URL: https://github.com/apache/doris/pull/16734#discussion_r1105514952 ## be/src/vec/data_types/data_type_nullable.cpp: ## @@ -158,4 +158,16 @@ DataTypePtr remove_nullable(const DataTypePtr& type) { return type; } +DataTypes r

[GitHub] [doris] pengxiangyu closed pull request #16752: [fix](cooldown)move remove_unused_remote_files

2023-02-14 Thread via GitHub
pengxiangyu closed pull request #16752: [fix](cooldown)move remove_unused_remote_files URL: https://github.com/apache/doris/pull/16752 -- 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] morrySnow commented on a diff in pull request #16449: [enhancement](nereids)enhance join reorder rule with project

2023-02-14 Thread via GitHub
morrySnow commented on code in PR #16449: URL: https://github.com/apache/doris/pull/16449#discussion_r1104455166 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/InApplyToJoin.java: ## @@ -63,7 +63,9 @@ public Rule build() { throw new A

[GitHub] [doris] ZhangGuoqiang666 opened a new pull request, #16753: test ignore p0 pipeline when change only docker file

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

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

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

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

2023-02-14 Thread via GitHub
bowenliang123 commented on PR #16754: URL: https://github.com/apache/doris/pull/16754#issuecomment-1429417103 This problem is encountered when I am doing another PR (https://github.com/apache/doris/pull/16736), which brings no dependency change bug failed in `Build Third Party Libraries (Li

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

2023-02-14 Thread via GitHub
bowenliang123 commented on PR #16754: URL: https://github.com/apache/doris/pull/16754#issuecomment-1429420137 cc @adonis0147 -- 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] stalary opened a new pull request, #16755: [Fix](CMakeLists): Reduce clang version

2023-02-14 Thread via GitHub
stalary opened a new pull request, #16755: URL: https://github.com/apache/doris/pull/16755 # Proposed changes Issue Number: close #xxx ## Problem summary Reduce clang version ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit

[GitHub] [doris] github-actions[bot] commented on pull request #16755: [Fix](CMakeLists): Reduce clang version

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16755: URL: https://github.com/apache/doris/pull/16755#issuecomment-1429429468 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 #16755: [Fix](CMakeLists): Reduce clang version

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16755: URL: https://github.com/apache/doris/pull/16755#issuecomment-1429430695 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 #16755: [Fix](CMakeLists): Reduce clang version

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16755: URL: https://github.com/apache/doris/pull/16755#issuecomment-1429430793 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 #16736: [chore](dependency) show progress bar when downloading third-party dependencies

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16736: URL: https://github.com/apache/doris/pull/16736#issuecomment-1429436406 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 #16736: [chore](dependency) show progress bar when downloading third-party dependencies

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

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

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16754: URL: https://github.com/apache/doris/pull/16754#issuecomment-1429439119 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] TangSiyang2001 opened a new pull request, #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
TangSiyang2001 opened a new pull request, #16757: URL: https://github.com/apache/doris/pull/16757 # Proposed changes Issue Number: close #16741 ## Problem summary Pre-block create stmt with `ALL` column type. ## Checklist(Required) * [ ] Does it affect the o

[GitHub] [doris] github-actions[bot] commented on pull request #16742: [fix](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets of cold data compaction

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16742: URL: https://github.com/apache/doris/pull/16742#issuecomment-1429470672 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] TangSiyang2001 commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
TangSiyang2001 commented on PR #16757: URL: https://github.com/apache/doris/pull/16757#issuecomment-1429473769 CREATE stmt with `ALL` type will be blocked. ```bash mysql> create table test (c1 all) distributed by hash(c1) properties('replication_num'='1'); ERROR 1105 (HY000): errCod

[GitHub] [doris] Tanya-W opened a new pull request, #16758: [improve](inverted index) Add element count limit for inverted index searcher cache

2023-02-14 Thread via GitHub
Tanya-W opened a new pull request, #16758: URL: https://github.com/apache/doris/pull/16758 # Proposed changes Issue Number: close #xxx ## Problem summary The element in InvertedIndexSearcherCache is inverted index searcher, which is a file descriptor of inverted index file,

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1429485331 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] catpineapple opened a new pull request, #16759: [feature](planner)fix multi partition support datetime column

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

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16758: URL: https://github.com/apache/doris/pull/16758#issuecomment-1429494632 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] TangSiyang2001 commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
TangSiyang2001 commented on PR #16757: URL: https://github.com/apache/doris/pull/16757#issuecomment-1429502073 > could you plz add some test? What kind of test? My test result was like below, it did work then: ```bash mysql> create table test (c1 all) distributed by hash(c1) pro

[GitHub] [doris] github-actions[bot] commented on pull request #14063: [Feature](ipv6)Support IPV6

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #14063: URL: https://github.com/apache/doris/pull/14063#issuecomment-1429512777 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 closed issue #16458: [Bug] Query same tablet but hits different rollup should not use colocate join

2023-02-14 Thread via GitHub
yiguolei closed issue #16458: [Bug] Query same tablet but hits different rollup should not use colocate join URL: https://github.com/apache/doris/issues/16458 -- 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

[GitHub] [doris] yiguolei merged pull request #16459: [fix](planner)Fix colocate query failed

2023-02-14 Thread via GitHub
yiguolei merged PR #16459: URL: https://github.com/apache/doris/pull/16459 -- 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](planner)Fix colocate query failed #16459

2023-02-14 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 acf5540a9f [fix](planner)Fix colocate query fai

[GitHub] [doris] Jibing-Li opened a new pull request, #16760: [Fix](multi catalog)Fix partition case bug.

2023-02-14 Thread via GitHub
Jibing-Li opened a new pull request, #16760: URL: https://github.com/apache/doris/pull/16760 # Proposed changes Issue Number: close #xxx ## Problem summary Set column names from path to lower case in case-insensitive case. This is for Iceberg columns from path. Iceber

[GitHub] [doris] Yukang-Lian commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
Yukang-Lian commented on PR #16757: URL: https://github.com/apache/doris/pull/16757#issuecomment-1429521487 > > could you plz add some test? > > What kind of test? My manual test result was like below,and it did work then: > > ```shell > mysql> create table test (c1 all) dis

[GitHub] [doris-website] morningman merged pull request #189: [doc] add 1.2.2 download link

2023-02-14 Thread via GitHub
morningman merged PR #189: URL: https://github.com/apache/doris-website/pull/189 -- 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: [doc] add 1.2.2 download link (#189)

2023-02-14 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 32735acc9b [doc] add 1.2.2 download l

[GitHub] [doris] TangSiyang2001 commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
TangSiyang2001 commented on PR #16757: URL: https://github.com/apache/doris/pull/16757#issuecomment-1429529028 > > > could you plz add some test? > > > > > > What kind of test? My manual test result was like below,and it did work then: > > ```shell > > mysql> create table te

[GitHub] [doris] pengxiangyu closed pull request #16258: [feature](cooldown)Add cooldown delete

2023-02-14 Thread via GitHub
pengxiangyu closed pull request #16258: [feature](cooldown)Add cooldown delete URL: https://github.com/apache/doris/pull/16258 -- 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] hello-stephen commented on pull request #16755: [Fix](CMakeLists): Reduce clang version

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

[GitHub] [doris] englefly commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
englefly commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105751625 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105756133 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] englefly commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
englefly commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105751625 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] englefly commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
englefly commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105751625 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105756133 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105756133 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] github-actions[bot] commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16757: URL: https://github.com/apache/doris/pull/16757#issuecomment-1429683853 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] englefly commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
englefly commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105751625 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] englefly commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
englefly commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105766515 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16731: [WIP] segcompaction feat. verticalcompaction

2023-02-14 Thread via GitHub
github-actions[bot] commented on code in PR #16731: URL: https://github.com/apache/doris/pull/16731#discussion_r1105775343 ## be/src/olap/rowset/beta_rowset_writer.cpp: ## @@ -116,17 +117,17 @@ Status BetaRowsetWriter::add_block(const vectorized::Block* block) { return _ad

[GitHub] [doris] hello-stephen commented on pull request #16757: [fix](Stmt)pre-block create stmt with `ALL` column type

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

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105788563 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105788563 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] XieJiann commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
XieJiann commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105788563 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/SortPhase.java: ## @@ -0,0 +1,45 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] englefly opened a new pull request, #16761: [feature](nereids)add rule eliminate inner join when any child is EmptyRelation.

2023-02-14 Thread via GitHub
englefly opened a new pull request, #16761: URL: https://github.com/apache/doris/pull/16761 # Proposed changes add rule to eliminate inner join when any child is EmptyRelation: * before: * inner-join * / \ * NodeEmptyRelation * * after: * Empty

[GitHub] [doris] hello-stephen commented on pull request #16759: [fix](planner)fix multi partition support datetime column

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

[GitHub] [doris] dataroaring merged pull request #16742: [fix](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets of cold data compaction

2023-02-14 Thread via GitHub
dataroaring merged PR #16742: URL: https://github.com/apache/doris/pull/16742 -- 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](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets of cold data compaction (#16742)

2023-02-14 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 7482b6bad2 [fix](cooldown) Add cold_compacti

[GitHub] [doris] morningman merged pull request #14063: [Feature](ipv6)Support IPV6

2023-02-14 Thread via GitHub
morningman merged PR #14063: URL: https://github.com/apache/doris/pull/14063 -- 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 (7482b6bad2 -> d013d529c8)

2023-02-14 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 7482b6bad2 [fix](cooldown) Add cold_compaction_lock to serialize any operations which may delete the input rowsets

[GitHub] [doris] github-actions[bot] commented on pull request #16654: [Feature](Materialized-View) testing

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16654: URL: https://github.com/apache/doris/pull/16654#issuecomment-1429783483 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] morningman opened a new issue, #16762: [Feature] Support thirdparty authorization

2023-02-14 Thread via GitHub
morningman opened a new issue, #16762: URL: https://github.com/apache/doris/issues/16762 ### 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 In current implem

[GitHub] [doris] morningman commented on pull request #16679: [Refactor](auth) Add AccessController to support customized authorization

2023-02-14 Thread via GitHub
morningman commented on PR #16679: URL: https://github.com/apache/doris/pull/16679#issuecomment-1429815128 > If a catalog uses range authentication, what should we do when the user uses' grant 'to authorize the table under the catalog? When checking the privilege of this catalog,

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16036: URL: https://github.com/apache/doris/pull/16036#issuecomment-1429819668 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] Jibing-Li closed pull request #16760: [Fix](multi catalog) Fix partition case bug.

2023-02-14 Thread via GitHub
Jibing-Li closed pull request #16760: [Fix](multi catalog) Fix partition case bug. URL: https://github.com/apache/doris/pull/16760 -- 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] Jibing-Li opened a new pull request, #16763: [Fix](multi catalog)Fix partition case bug.

2023-02-14 Thread via GitHub
Jibing-Li opened a new pull request, #16763: URL: https://github.com/apache/doris/pull/16763 # Proposed changes Issue Number: close #xxx ## Problem summary Set column names from path to lower case in case-insensitive case. This is for Iceberg columns from path. Iceberg

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16036: URL: https://github.com/apache/doris/pull/16036#issuecomment-1429843197 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] Jibing-Li opened a new pull request, #16764: [Fix](multi catalog)Fix iceberg parquet file doesn't have iceberg.schema meta problem.

2023-02-14 Thread via GitHub
Jibing-Li opened a new pull request, #16764: URL: https://github.com/apache/doris/pull/16764 # Proposed changes Issue Number: close #xxx ## Problem summary To support schema evolution, Iceberg add schema information to Parquet file metadata. But for early iceberg version

[GitHub] [doris] github-actions[bot] commented on pull request #16764: [Fix](multi catalog)Fix iceberg parquet file doesn't have iceberg.schema meta problem.

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16764: URL: https://github.com/apache/doris/pull/16764#issuecomment-1429856205 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] zhilaohu32 opened a new issue, #16765: [Bug]

2023-02-14 Thread via GitHub
zhilaohu32 opened a new issue, #16765: URL: https://github.com/apache/doris/issues/16765 ### 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] morrySnow commented on a diff in pull request #16586: [refactor](Nereids): add two phase sort

2023-02-14 Thread via GitHub
morrySnow commented on code in PR #16586: URL: https://github.com/apache/doris/pull/16586#discussion_r1105957734 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/implementation/LogicalTopNToPhysicalTopN.java: ## @@ -19,20 +19,33 @@ import org.apache.doris.nereids.ru

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #15509: URL: https://github.com/apache/doris/pull/15509#issuecomment-1429933776 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 #16763: [Fix](multi catalog)Fix partition case bug.

2023-02-14 Thread via GitHub
hello-stephen commented on PR #16763: URL: https://github.com/apache/doris/pull/16763#issuecomment-1429958280 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 33.88 seconds stream load tsv: 468 seconds loaded 74807831229 Bytes, about 152 MB/s

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

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16036: URL: https://github.com/apache/doris/pull/16036#issuecomment-1429974417 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 #16734: [Opt](exec) opt aggreate function performance in nullable column

2023-02-14 Thread via GitHub
github-actions[bot] commented on PR #16734: URL: https://github.com/apache/doris/pull/16734#issuecomment-1430081440 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] luzhijing opened a new pull request, #16766: [docs](releasenote)1.2.2 release note

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

  1   2   3   >