[GitHub] [doris] hello-stephen commented on pull request #16152: [Bug](decimalv3) fix BE crash for function `if`

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16152: URL: https://github.com/apache/doris/pull/16152#issuecomment-1407325087 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 33.27 seconds load time: 491 seconds storage size: 17120625313 Bytes https://doris-co

[GitHub] [doris] SaintBacchus closed pull request #16149: [MTMV](WIP)Support master and follow change in multi fe for mtmv

2023-01-28 Thread via GitHub
SaintBacchus closed pull request #16149: [MTMV](WIP)Support master and follow change in multi fe for mtmv URL: https://github.com/apache/doris/pull/16149 -- 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

[GitHub] [doris] BiteTheDDDDt opened a new pull request, #16157: [Bug](exec) enable warning on ignoring function return value for vctx

2023-01-28 Thread via GitHub
BiteThet opened a new pull request, #16157: URL: https://github.com/apache/doris/pull/16157 # Proposed changes enable warning on ignoring function return value for vctx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect th

[GitHub] [doris] yiguolei merged pull request #16083: [improvement](memory) load support overcommit memory

2023-01-28 Thread via GitHub
yiguolei merged PR #16083: URL: https://github.com/apache/doris/pull/16083 -- 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 (49395390be -> 949a065f22)

2023-01-28 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 49395390be [bugfix](metareader) meta reader could not load image (#16148) add 949a065f22 [improvement](memory)

[GitHub] [doris] github-actions[bot] commented on pull request #16157: [Bug](exec) enable warning on ignoring function return value for vctx

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16157: URL: https://github.com/apache/doris/pull/16157#issuecomment-1407328157 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 opened a new pull request, #16158: Upgrade springboot version to 2.7.8

2023-01-28 Thread via GitHub
hf200012 opened a new pull request, #16158: URL: https://github.com/apache/doris/pull/16158 # Proposed changes Upgrade the springboot version to 2.7.8, and fix other problems that springboot depends on Issue Number: close #xxx ## Problem summary Describe your chan

[GitHub] [doris] Gabriel39 opened a new pull request, #16159: [datev2](function) make function nullable DEPEND_ON_ARGUMENT

2023-01-28 Thread via GitHub
Gabriel39 opened a new pull request, #16159: URL: https://github.com/apache/doris/pull/16159 # Proposed changes Set all functions' nullable mode for `datev2` to `DEPEND_ON_ARGUMENT` ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it

[GitHub] [doris] httpshirley opened a new pull request, #16160: [docs]update en docs

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

[GitHub] [doris] github-actions[bot] commented on pull request #16132: [fix](datatype) fix some bugs about data type array datetimev2 and decimalv3

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16132: URL: https://github.com/apache/doris/pull/16132#issuecomment-1407340190 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 #16153: [bug](cooldown)add config for Cooldown Job

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16153: URL: https://github.com/apache/doris/pull/16153#issuecomment-1407342007 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.96 seconds load time: 526 seconds storage size: 17123103742 Bytes https://doris-co

[GitHub] [doris] luzhijing commented on pull request #16160: [docs](docs)update en docs

2023-01-28 Thread via GitHub
luzhijing commented on PR #16160: URL: https://github.com/apache/doris/pull/16160#issuecomment-1407343066 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] Tanya-W opened a new pull request, #16161: [fix](inverted index) fulltext match query should not prune by zone map

2023-01-28 Thread via GitHub
Tanya-W opened a new pull request, #16161: URL: https://github.com/apache/doris/pull/16161 # Proposed changes zone map prune data by min/max value, should not prune by zone map when fulltext match term is substring in a string type column's value, otherwise, some data meeting the conditi

[GitHub] [doris] morningman commented on a diff in pull request #16160: [docs](docs)update en docs

2023-01-28 Thread via GitHub
morningman commented on code in PR #16160: URL: https://github.com/apache/doris/pull/16160#discussion_r1089690101 ## docs/en/docs/lakehouse/multi-catalog/multi-catalog.md: ## @@ -261,7 +261,7 @@ See [Hudi](./hudi) ### Connect to Elasticsearch -See [Elasticsearch](./elastics

[GitHub] [doris] morningman commented on a diff in pull request #16160: [docs](docs)update en docs

2023-01-28 Thread via GitHub
morningman commented on code in PR #16160: URL: https://github.com/apache/doris/pull/16160#discussion_r1089690156 ## docs/en/docs/lakehouse/multi-catalog/hive.md: ## @@ -26,4 +26,149 @@ under the License. # Hive -TODO: translate +Once Doris is connected to Hive Metastore or

[GitHub] [doris] github-actions[bot] commented on pull request #16161: [fix](inverted index) fulltext match query should not prune by zone map

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16161: URL: https://github.com/apache/doris/pull/16161#issuecomment-1407346107 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 #16161: [fix](inverted index) fulltext match query should not prune by zone map

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16161: URL: https://github.com/apache/doris/pull/16161#issuecomment-1407346437 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 #16158: [deps](http)Upgrade springboot version to 2.7.8

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16158: URL: https://github.com/apache/doris/pull/16158#issuecomment-1407347233 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 #16158: [deps](http)Upgrade springboot version to 2.7.8

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16158: URL: https://github.com/apache/doris/pull/16158#issuecomment-1407347238 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [doris] yiguolei opened a new pull request, #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread via GitHub
yiguolei opened a new pull request, #16162: URL: https://github.com/apache/doris/pull/16162 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Ye

[GitHub] [doris] github-actions[bot] commented on pull request #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16162: URL: https://github.com/apache/doris/pull/16162#issuecomment-1407355573 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] sohardforaname opened a new pull request, #16163: [Fix](Nereids)changing test data for more effectively testing for nereids_function_p0.

2023-01-28 Thread via GitHub
sohardforaname opened a new pull request, #16163: URL: https://github.com/apache/doris/pull/16163 # Proposed changes Issue Number: close #xxx ## Problem summary changing test data for more effectively testing for nereids_function_p0. ## Checklist(Required)

[GitHub] [doris] hello-stephen commented on pull request #16158: [deps](http)Upgrade springboot version to 2.7.8

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16158: URL: https://github.com/apache/doris/pull/16158#issuecomment-1407360132 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.31 seconds load time: 493 seconds storage size: 17122289820 Bytes https://doris-co

[GitHub] [doris] hello-stephen commented on pull request #16157: [Bug](exec) enable warning on ignoring function return value for vctx

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16157: URL: https://github.com/apache/doris/pull/16157#issuecomment-1407363069 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.56 seconds load time: 543 seconds storage size: 17122534818 Bytes https://doris-co

[GitHub] [doris] Gabriel39 merged pull request #16102: [Bug](runtimefilter) Fix wrong runtime filter on datetime

2023-01-28 Thread via GitHub
Gabriel39 merged PR #16102: URL: https://github.com/apache/doris/pull/16102 -- 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](runtimefilter) Fix wrong runtime filter on datetime (#16102)

2023-01-28 Thread gabriellee
This is an automated email from the ASF dual-hosted git repository. gabriellee 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 7cf7706eb1 [Bug](runtimefilter) Fix wrong run

[GitHub] [doris] github-actions[bot] commented on pull request #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16162: URL: https://github.com/apache/doris/pull/16162#issuecomment-1407366751 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 #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16162: URL: https://github.com/apache/doris/pull/16162#issuecomment-1407366761 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] jackwener opened a new pull request, #16164: [fix](Nereids): fix Ceiling.

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

[GitHub] [doris] github-actions[bot] commented on pull request #16129: [cleanup](fe) cleanup useless code.

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16129: URL: https://github.com/apache/doris/pull/16129#issuecomment-1407371127 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

[doris] branch master updated: [cleanup](fe) cleanup useless code. (#16129)

2023-01-28 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 7e7fd5d049 [cleanup](fe) cleanup useless code.

[GitHub] [doris] github-actions[bot] commented on pull request #16129: [cleanup](fe) cleanup useless code.

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16129: URL: https://github.com/apache/doris/pull/16129#issuecomment-1407371121 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] yiguolei merged pull request #16129: [cleanup](fe) cleanup useless code.

2023-01-28 Thread via GitHub
yiguolei merged PR #16129: URL: https://github.com/apache/doris/pull/16129 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apa

[GitHub] [doris] jacktengg opened a new issue, #16165: [Bug] join produces too many rows causing be coredump

2023-01-28 Thread via GitHub
jacktengg opened a new issue, #16165: URL: https://github.com/apache/doris/issues/16165 ### 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 7e7fd5d0496dc54

[GitHub] [doris] jacktengg opened a new pull request, #16166: [fix](hashjoin) join produce blocks with rows larger than batch size

2023-01-28 Thread via GitHub
jacktengg opened a new pull request, #16166: URL: https://github.com/apache/doris/pull/16166 # Proposed changes Issue Number: close #16165 ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ]

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16166: [fix](hashjoin) join produce blocks with rows larger than batch size

2023-01-28 Thread github-actions
github-actions[bot] commented on code in PR #16166: URL: https://github.com/apache/doris/pull/16166#discussion_r1089717648 ## be/src/vec/exec/join/process_hash_table_probe_impl.h: ## @@ -208,123 +207,180 @@ Status ProcessHashTableProbe::do_process(HashTableType& hash_table_c

[GitHub] [doris] Tanya-W opened a new pull request, #16167: [Feature-WIP] (inverted index) Filter out and remain predicates that do not support apply by inverted index, and add inverted index regressi

2023-01-28 Thread via GitHub
Tanya-W opened a new pull request, #16167: URL: https://github.com/apache/doris/pull/16167 # Proposed changes 1. Filter out and remain predicates that do not support apply by inverted index, like `BF` predicate, `IS_NULL` predicate, `IS_NOT_NULL` predicate. 2. Add inverted index regres

[GitHub] [doris] hello-stephen commented on pull request #16161: [fix](inverted index) fulltext match query should not prune by zone map

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16161: URL: https://github.com/apache/doris/pull/16161#issuecomment-1407376959 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.29 seconds load time: 539 seconds storage size: 17122430817 Bytes https://doris-co

[GitHub] [doris] github-actions[bot] commented on pull request #16167: [Feature-WIP](inverted index) Filter out and remain predicates that do not support apply by inverted index, and add inverted inde

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16167: URL: https://github.com/apache/doris/pull/16167#issuecomment-1407377037 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 #16161: [fix](inverted index) fulltext match query should not prune by zone map

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16161: URL: https://github.com/apache/doris/pull/16161#issuecomment-1407377323 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 #16167: [Feature-WIP](inverted index) Filter out and remain predicates that do not support apply by inverted index, and add inverted inde

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16167: URL: https://github.com/apache/doris/pull/16167#issuecomment-1407377423 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 #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16162: URL: https://github.com/apache/doris/pull/16162#issuecomment-1407379910 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.08 seconds load time: 489 seconds storage size: 17122304340 Bytes https://doris-co

[GitHub] [doris] hello-stephen commented on pull request #16156: [refactor](Nereids) bind slot, function and do type coercion in one rule

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16156: URL: https://github.com/apache/doris/pull/16156#issuecomment-1407380117 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.17 seconds load time: 524 seconds storage size: 17122260141 Bytes https://doris-co

[GitHub] [doris] Gabriel39 closed issue #15986: [Bug] if 函数后计算结果发生错误

2023-01-28 Thread via GitHub
Gabriel39 closed issue #15986: [Bug] if 函数后计算结果发生错误 URL: https://github.com/apache/doris/issues/15986 -- 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: c

[GitHub] [doris] Gabriel39 merged pull request #16152: [Bug](decimalv3) fix BE crash for function `if`

2023-01-28 Thread via GitHub
Gabriel39 merged PR #16152: URL: https://github.com/apache/doris/pull/16152 -- 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 (7e7fd5d049 -> 26fc7c8196)

2023-01-28 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 7e7fd5d049 [cleanup](fe) cleanup useless code. (#16129) add 26fc7c8196 [Bug](decimalv3) fix BE crash for funct

[GitHub] [doris] github-actions[bot] commented on pull request #16167: [Feature-WIP](inverted index) Filter out and remain predicates that do not support apply by inverted index, and add inverted inde

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16167: URL: https://github.com/apache/doris/pull/16167#issuecomment-1407380183 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 #16164: [fix](Nereids): fix Ceiling.

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16164: URL: https://github.com/apache/doris/pull/16164#issuecomment-1407383197 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.34 seconds load time: 490 seconds storage size: 17122375691 Bytes https://doris-co

[GitHub] [doris] yiguolei merged pull request #16162: [refactor](remove unused code) remove partition info from datastream sender

2023-01-28 Thread via GitHub
yiguolei merged PR #16162: URL: https://github.com/apache/doris/pull/16162 -- 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 (26fc7c8196 -> 241a956b20)

2023-01-28 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 26fc7c8196 [Bug](decimalv3) fix BE crash for function `if` (#16152) add 241a956b20 [refactor](remove unused code

[GitHub] [doris] github-actions[bot] commented on pull request #16153: [bug](cooldown)add config for Cooldown Job

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16153: URL: https://github.com/apache/doris/pull/16153#issuecomment-1407383521 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 #16153: [bug](cooldown)add config for Cooldown Job

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16153: URL: https://github.com/apache/doris/pull/16153#issuecomment-1407383534 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [doris] yiguolei merged pull request #16153: [bug](cooldown)add config for Cooldown Job

2023-01-28 Thread via GitHub
yiguolei merged PR #16153: URL: https://github.com/apache/doris/pull/16153 -- 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: [bug](cooldown)add config for Cooldown Job

2023-01-28 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 c506b4a1e3 [bug](cooldown)add config for Cooldo

[GitHub] [doris] BiteTheDDDDt opened a new pull request, #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread via GitHub
BiteThet opened a new pull request, #16168: URL: https://github.com/apache/doris/pull/16168 # Proposed changes remove datev2 signature of hour_ceil/hour_floor ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16159: [datev2](function) make function nullable DEPEND_ON_ARGUMENT

2023-01-28 Thread github-actions
github-actions[bot] commented on code in PR #16159: URL: https://github.com/apache/doris/pull/16159#discussion_r1089723206 ## be/src/vec/functions/function_timestamp.cpp: ## @@ -721,37 +837,59 @@ static Status execute_impl(FunctionContext* context, Block& block,

[GitHub] [doris] github-actions[bot] commented on pull request #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407384603 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 #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407384757 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 #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407384767 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 #16159: [datev2](function) make function nullable DEPEND_ON_ARGUMENT

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16159: URL: https://github.com/apache/doris/pull/16159#issuecomment-1407385770 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 a diff in pull request #16159: [datev2](function) make function nullable DEPEND_ON_ARGUMENT

2023-01-28 Thread github-actions
github-actions[bot] commented on code in PR #16159: URL: https://github.com/apache/doris/pull/16159#discussion_r1089723868 ## be/src/vec/functions/function_timestamp.cpp: ## @@ -721,37 +837,59 @@ struct MondayImpl { static Status execute_impl(FunctionContext* context, Block

[GitHub] [doris] yiguolei merged pull request #16158: [deps](http)Upgrade springboot version to 2.7.8

2023-01-28 Thread via GitHub
yiguolei merged PR #16158: URL: https://github.com/apache/doris/pull/16158 -- 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: [deps](http)Upgrade springboot version to 2.7.8 (#16158)

2023-01-28 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 da28d2faee [deps](http)Upgrade springboot versi

[GitHub] [doris] yiguolei merged pull request #16145: [bugfix](compaction) fix segment writer finalize footer twice

2023-01-28 Thread via GitHub
yiguolei merged PR #16145: URL: https://github.com/apache/doris/pull/16145 -- 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 (da28d2faee -> 8ca3aa85e2)

2023-01-28 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from da28d2faee [deps](http)Upgrade springboot version to 2.7.8 (#16158) add 8ca3aa85e2 [bugfix](compaction) fix segm

[GitHub] [doris] Kikyou1997 opened a new pull request, #16169: [fix](nereids) Delete invalid signature of nvl function

2023-01-28 Thread via GitHub
Kikyou1997 opened a new pull request, #16169: URL: https://github.com/apache/doris/pull/16169 1. delete invalid signature of nvl function 2. fix some test cases that failed because of malformed function name # Proposed changes Issue Number: close #xxx ## Problem summar

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16157: [Bug](exec) enable warning on ignoring function return value for vctx

2023-01-28 Thread github-actions
github-actions[bot] commented on code in PR #16157: URL: https://github.com/apache/doris/pull/16157#discussion_r1089725536 ## be/test/vec/exprs/vexpr_test.cpp: ## @@ -83,8 +86,11 @@ TEST(TEST_VEXPR, ABSTEST2) { DescriptorTbl desc_tbl; Review Comment: warning: calling a

[GitHub] [doris] jackwener merged pull request #16164: [fix](Nereids): fix Ceiling.

2023-01-28 Thread via GitHub
jackwener merged PR #16164: URL: https://github.com/apache/doris/pull/16164 -- 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: [fix](Nereids): fix Ceiling. (#16164)

2023-01-28 Thread jakevin
This is an automated email from the ASF dual-hosted git repository. jakevin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 3151d94e9e [fix](Nereids): fix Ceiling. (#16164)

[GitHub] [doris] hello-stephen commented on pull request #16166: [fix](hashjoin) join produce blocks with rows larger than batch size

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16166: URL: https://github.com/apache/doris/pull/16166#issuecomment-1407389152 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.44 seconds load time: 525 seconds storage size: 17122289679 Bytes https://doris-co

[GitHub] [doris] hello-stephen commented on pull request #16167: [Feature-WIP](inverted index) Filter out and remain predicates that do not support apply by inverted index, and add inverted index regr

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16167: URL: https://github.com/apache/doris/pull/16167#issuecomment-1407391651 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.6 seconds load time: 491 seconds storage size: 17123012999 Bytes https://doris-com

[GitHub] [doris] jackwener opened a new pull request, #16170: [fix](Nereids): fix convert_tz() SIGNATURES.

2023-01-28 Thread via GitHub
jackwener opened a new pull request, #16170: URL: https://github.com/apache/doris/pull/16170 # Proposed changes we shouldn't add `DateV2` in `SIGNATURES`, because when meet `DateV2`, it will cast to `DateTimeV2Type` implicitly. ## Problem summary Describe your changes.

[GitHub] [doris] github-actions[bot] commented on pull request #16169: [fix](nereids) Delete invalid signature of nvl function

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16169: URL: https://github.com/apache/doris/pull/16169#issuecomment-1407394710 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 #16169: [fix](nereids) Delete invalid signature of nvl function

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16169: URL: https://github.com/apache/doris/pull/16169#issuecomment-1407394721 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 #16170: [fix](Nereids): fix convert_tz() SIGNATURES.

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16170: URL: https://github.com/apache/doris/pull/16170#issuecomment-1407397943 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 #16170: [fix](Nereids): fix convert_tz() SIGNATURES.

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16170: URL: https://github.com/apache/doris/pull/16170#issuecomment-1407397954 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] morrySnow merged pull request #16163: [Fix](Nereids)changing test data for more effectively testing for nereids_function_p0.

2023-01-28 Thread via GitHub
morrySnow merged PR #16163: URL: https://github.com/apache/doris/pull/16163 -- 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: [test](Nereids) changing test data for more effectively testing for nereids_function_p0 (#16163)

2023-01-28 Thread morrysnow
This is an automated email from the ASF dual-hosted git repository. morrysnow 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 b7379daffa [test](Nereids) changing test data

[GitHub] [doris] hello-stephen commented on pull request #16159: [datev2](function) make function nullable DEPEND_ON_ARGUMENT

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16159: URL: https://github.com/apache/doris/pull/16159#issuecomment-1407398765 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.51 seconds load time: 524 seconds storage size: 17122962414 Bytes https://doris-co

[GitHub] [doris] sohardforaname opened a new pull request, #16171: [Fix](Nereids)fix error of X-Z function for nereids

2023-01-28 Thread via GitHub
sohardforaname opened a new pull request, #16171: URL: https://github.com/apache/doris/pull/16171 # Proposed changes Issue Number: close #xxx ## Problem summary fix error of X-Z function for nereids ## Checklist(Required) 1. Does it affect the original behav

[GitHub] [doris] hello-stephen commented on pull request #16169: [fix](nereids) Delete invalid signature of nvl function

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16169: URL: https://github.com/apache/doris/pull/16169#issuecomment-1407406308 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.13 seconds load time: 517 seconds storage size: 17122314581 Bytes https://doris-co

[GitHub] [doris] github-actions[bot] commented on pull request #15924: [improvement](agg-function) remove the limit on the max number of agg function parameters

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #15924: URL: https://github.com/apache/doris/pull/15924#issuecomment-1407409582 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 #16170: [fix](Nereids): remove `DataV2Type` in ConvertTz, DayCeil and DayFloor SIGNATURES

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16170: URL: https://github.com/apache/doris/pull/16170#issuecomment-1407410262 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.13 seconds load time: 507 seconds storage size: 17122819584 Bytes https://doris-co

[GitHub] [doris] github-actions[bot] commented on pull request #15924: [improvement](agg-function) Increase the limit maximum number of agg function parameters

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #15924: URL: https://github.com/apache/doris/pull/15924#issuecomment-1407410562 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, #16172: [fix-core](block) clear block row_same_bit when block reuse

2023-01-28 Thread via GitHub
nextdreamblue opened a new pull request, #16172: URL: https://github.com/apache/doris/pull/16172 Signed-off-by: nextdreamblue # Proposed changes Issue Number: close #16155 ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it af

[GitHub] [doris] github-actions[bot] commented on pull request #16172: [fix-core](block) clear block row_same_bit when block reuse

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16172: URL: https://github.com/apache/doris/pull/16172#issuecomment-1407415617 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 #16172: [fix-core](block) clear block row_same_bit when block reuse

2023-01-28 Thread github-actions
github-actions[bot] commented on PR #16172: URL: https://github.com/apache/doris/pull/16172#issuecomment-1407415991 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] eldenmoon opened a new pull request, #16173: [Bug](topn-opt) filter condition for analytic info for two phase read…

2023-01-28 Thread via GitHub
eldenmoon opened a new pull request, #16173: URL: https://github.com/apache/doris/pull/16173 … opt two phase read optimization should not be enabled when query has analytic info # Proposed changes Issue Number: close #xxx ## Problem summary Describe your ch

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #15718: [fix](schema scanner)change schema_scanner::get_next_row to get_next_block

2023-01-28 Thread github-actions
github-actions[bot] commented on code in PR #15718: URL: https://github.com/apache/doris/pull/15718#discussion_r1089757755 ## be/src/vec/exec/vschema_scan_node.cpp: ## @@ -255,74 +257,61 @@ Status VSchemaScanNode::get_next(RuntimeState* state, vectorized::Block* block,

[GitHub] [doris] jackwener opened a new pull request, #16174: [fix](Nereids): fix dceil() dfloor()

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

[GitHub] [doris] hello-stephen commented on pull request #16172: [fix-core](block) clear block row_same_bit when block reuse

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16172: URL: https://github.com/apache/doris/pull/16172#issuecomment-1407425534 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.27 seconds load time: 522 seconds storage size: 17120405529 Bytes https://doris-co

[GitHub] [doris] jackwener commented on pull request #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread via GitHub
jackwener commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407428342 Maybe need notice: old optimizer `ScalarBuiltins.java` contains the removed type SIGNATURES. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [doris] jackwener commented on pull request #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread via GitHub
jackwener commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407428430 #16174 is same with this PR -- 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] BiteTheDDDDt commented on pull request #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread via GitHub
BiteThet commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407430631 > In this case we have wrong signature(the implementation of hour_ceil/hour_floor in `be` is inconsistent with the signature), so I remove it. -- This is an automated mess

[GitHub] [doris] BiteTheDDDDt commented on pull request #16170: [fix](Nereids): remove `DataV2Type` in ConvertTz, DayCeil and DayFloor SIGNATURES

2023-01-28 Thread via GitHub
BiteThet commented on PR #16170: URL: https://github.com/apache/doris/pull/16170#issuecomment-1407431450 On the one hand, you should make sure that the signature in nereids is consistent with the signature in doris_builtins_functions.py. On the other hand, whether a function like DayCei

[GitHub] [doris] hello-stephen commented on pull request #16173: [Bug](topn-opt) filter condition for analytic info for two phase read…

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16173: URL: https://github.com/apache/doris/pull/16173#issuecomment-1407432708 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.68 seconds load time: 494 seconds storage size: 17121999283 Bytes https://doris-co

[GitHub] [doris] jackwener commented on pull request #16168: [Bug](function) remove datev2 signature of hour_ceil/hour_floor

2023-01-28 Thread via GitHub
jackwener commented on PR #16168: URL: https://github.com/apache/doris/pull/16168#issuecomment-1407433615 > Maybe need notice: old optimizer `ScalarBuiltins.java` contains the removed type SIGNATURES. signature in it also need to be removed -- This is an automated message from the

[GitHub] [doris] hello-stephen commented on pull request #16174: [fix](Nereids): fix dceil() dfloor()

2023-01-28 Thread via GitHub
hello-stephen commented on PR #16174: URL: https://github.com/apache/doris/pull/16174#issuecomment-1407433947 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 36.49 seconds load time: 491 seconds storage size: 17123140409 Bytes https://doris-co

[GitHub] [doris] morningman merged pull request #16160: [docs](docs)update en docs

2023-01-28 Thread via GitHub
morningman merged PR #16160: URL: https://github.com/apache/doris/pull/16160 -- 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: [docs](multi-catalog)update en docs (#16160)

2023-01-28 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 46ce66cbd8 [docs](multi-catalog)update en doc

  1   2   3   >