[GitHub] [doris] qinchaofeng commented on issue #12328: [Bug] The development version of the document Problems.

2022-09-05 Thread GitBox
qinchaofeng commented on issue #12328: URL: https://github.com/apache/doris/issues/12328#issuecomment-1237730469 Please view the PROD document : [Date Functions](https://doris.apache.org/docs/sql-manual/sql-functions/date-time-functions/dayname/) -- This is an automated message from t

[GitHub] [doris] github-actions[bot] commented on pull request #12376: [typo](docs)Sql blacklist documentation fix

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12376: URL: https://github.com/apache/doris/pull/12376#issuecomment-1237730091 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 #12376: [typo](docs)Sql blacklist documentation fix

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12376: URL: https://github.com/apache/doris/pull/12376#issuecomment-1237730051 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] Henry2SS commented on pull request #12348: [enhancement](load) make default load mem limit configurable

2022-09-05 Thread GitBox
Henry2SS commented on PR #12348: URL: https://github.com/apache/doris/pull/12348#issuecomment-1237725446 > We already had a `load_mem_limit` session variable for this purpose. I think we should use that instead of adding a new config. the defaut value of `load_mem_limit` is `0`. And i

[GitHub] [doris] morrySnow commented on pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
morrySnow commented on PR #12129: URL: https://github.com/apache/doris/pull/12129#issuecomment-1237725648 please add some regression test under regression-test/suites/nereids_syntax_p0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [doris] morrySnow commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
morrySnow commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963310372 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,201 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [doris] whutpencil commented on issue #12368: [Enhancement] broker load should not close file system when exception happens

2022-09-05 Thread GitBox
whutpencil commented on issue #12368: URL: https://github.com/apache/doris/issues/12368#issuecomment-1237723295 > In some senerio, `t1` is still using the file system to reading files. It means that the file system is not broken. If `t2` close the file system, it will leads `t1` to abor

[GitHub] [doris] whutpencil commented on issue #12368: [Enhancement] broker load should not close file system when exception happens

2022-09-05 Thread GitBox
whutpencil commented on issue #12368: URL: https://github.com/apache/doris/issues/12368#issuecomment-1237722982 > In some senerio, `t1` is still using the file system to reading files. It means that the file system is not broken. If `t2` close the file system, it will leads `t1` to abor

[GitHub] [doris] cambyzju opened a new pull request, #12378: [enhancement](array-type) avoid abuse of Offset and Offset64

2022-09-05 Thread GitBox
cambyzju opened a new pull request, #12378: URL: https://github.com/apache/doris/pull/12378 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. We already separate Array Offset64 and String Offset(32bit) in PR: https://github.com/

[GitHub] [doris] stalary opened a new pull request, #12377: [Enhancement](container test): Add elasticsearch container file

2022-09-05 Thread GitBox
stalary opened a new pull request, #12377: URL: https://github.com/apache/doris/pull/12377 # Proposed changes Issue Number: close #xxx ## Problem summary Add elasticsearch container file ## Checklist(Required) 1. Does it affect the original behavior:

[GitHub] [doris] Henry2SS commented on issue #12368: [Enhancement] broker load should not close file system when exception happens

2022-09-05 Thread GitBox
Henry2SS commented on issue #12368: URL: https://github.com/apache/doris/issues/12368#issuecomment-1237718977 > There is no problem with this logic. If the filesystem cannot be used, it will still report an error next time if it is not closed. In some senerio, `t1` is still using the

[doris-website] branch master updated: Sql blacklist documentation fix

2022-09-05 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-website.git The following commit(s) were added to refs/heads/master by this push: new aeb44ad5ea6 Sql blacklist documenta

[GitHub] [doris] hf200012 opened a new pull request, #12376: [typo](docs)Sql blacklist documentation fix

2022-09-05 Thread GitBox
hf200012 opened a new pull request, #12376: URL: https://github.com/apache/doris/pull/12376 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Ye

[doris] branch master updated (4a55b504c0 -> f1507f93ee)

2022-09-05 Thread jakevin
This is an automated email from the ASF dual-hosted git repository. jakevin pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 4a55b504c0 [feature-wip](parquet-reader) bug fix, get the correct group reader (#12294) add f1507f93ee [enhancem

[GitHub] [doris] jackwener merged pull request #12365: [enhancement](chore)add single empty line rule to fe check style for Nereids

2022-09-05 Thread GitBox
jackwener merged PR #12365: URL: https://github.com/apache/doris/pull/12365 -- 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] HappenLee commented on a diff in pull request #12334: [Opt](vectorized) speed up hash function compute in hash partition

2022-09-05 Thread GitBox
HappenLee commented on code in PR #12334: URL: https://github.com/apache/doris/pull/12334#discussion_r963293499 ## be/src/vec/columns/column.h: ## @@ -286,6 +299,12 @@ class IColumn : public COW { /// passed bytes to hash must identify sequence of values unambiguously.

[GitHub] [doris] github-actions[bot] commented on pull request #12371: [chore](ut) enable asan core dump when running be ut

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12371: URL: https://github.com/apache/doris/pull/12371#issuecomment-1237700648 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 #12371: [chore](ut) enable asan core dump when running be ut

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12371: URL: https://github.com/apache/doris/pull/12371#issuecomment-1237700617 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 #12373: [fix](frontend) fix peerDependencies error

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12373: URL: https://github.com/apache/doris/pull/12373#issuecomment-1237699303 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 #12373: [fix](frontend) fix peerDependencies error

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12373: URL: https://github.com/apache/doris/pull/12373#issuecomment-1237699281 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] Gabriel39 opened a new pull request, #12375: [FOLLOWUP](load) fix nullable and add regression

2022-09-05 Thread GitBox
Gabriel39 opened a new pull request, #12375: URL: https://github.com/apache/doris/pull/12375 # Proposed changes follow-up for: #12302 ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Yes

[GitHub] [doris] luozenglin commented on pull request #12290: [enhancement](tracing) Support forward to master tracing

2022-09-05 Thread GitBox
luozenglin commented on PR #12290: URL: https://github.com/apache/doris/pull/12290#issuecomment-1237696024 LGTM -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscri

[doris] branch master updated: [feature-wip](parquet-reader) bug fix, get the correct group reader (#12294)

2022-09-05 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 4a55b504c0 [feature-wip](parquet-reader) bug

[GitHub] [doris] morningman merged pull request #12294: [feature-wip](parquet-reader) bug fix, get the correct group reader

2022-09-05 Thread GitBox
morningman merged PR #12294: URL: https://github.com/apache/doris/pull/12294 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.a

[GitHub] [doris] Kikyou1997 commented on pull request #12374: [feature-wip](nereids) Overwrite nullable method for Year function and Divid expression

2022-09-05 Thread GitBox
Kikyou1997 commented on PR #12374: URL: https://github.com/apache/doris/pull/12374#issuecomment-1237668229 @morningman @jackwener @englefly -- 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 spe

[GitHub] [doris] Kikyou1997 opened a new pull request, #12374: [feature-wip](nereids) Overwrite nullable method for Year function and Divid expression

2022-09-05 Thread GitBox
Kikyou1997 opened a new pull request, #12374: URL: https://github.com/apache/doris/pull/12374 # Proposed changes Issue Number: noissue ## Problem summary The nullability of these function should always be nullable, so just return true in the overwrite method. ## Check

[GitHub] [doris] jeffreys-cat opened a new pull request, #12373: [fix](frontend) fix peerDependencies error

2022-09-05 Thread GitBox
jeffreys-cat opened a new pull request, #12373: URL: https://github.com/apache/doris/pull/12373 # Proposed changes Issue Number: close #xxx ## Problem summary ```npm install``` problem with peer dependencies in the latest version of npm (v7+) Use ```npm install --leg

[GitHub] [doris] qzsee commented on pull request #12366: [Fix] add hll param for if function

2022-09-05 Thread GitBox
qzsee commented on PR #12366: URL: https://github.com/apache/doris/pull/12366#issuecomment-1237654675 > Which SQL print the error message? SELECT hll_union_agg(if(xxx='xxx', xx_id, hll_empty())) from xxx where xxx = '2022-08-10' and xx = '11' -- This is an automated

[GitHub] [doris] whutpencil commented on issue #12368: [Enhancement] broker load should not close file system when exception happens

2022-09-05 Thread GitBox
whutpencil commented on issue #12368: URL: https://github.com/apache/doris/issues/12368#issuecomment-1237649120 There is no problem with this logic. If the filesystem cannot be used, it will still report an error next time if it is not closed. -- This is an automated message from the Apac

[GitHub] [doris] adonis0147 commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
adonis0147 commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963242111 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,184 @@ +// Licensed to the Apache Software Foundation (ASF) unde

[GitHub] [doris] adonis0147 commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
adonis0147 commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963228201 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,184 @@ +// Licensed to the Apache Software Foundation (ASF) unde

[doris] branch master updated (cf5d194fe1 -> d7dedfadad)

2022-09-05 Thread huajianlan
This is an automated email from the ASF dual-hosted git repository. huajianlan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from cf5d194fe1 [enhancement](array-type) Split Array Offsets and String Offsets (#12341) add d7dedfadad [fix](ner

[GitHub] [doris] 924060929 merged pull request #12345: [fix](nereids) fix dead loop in unnesting subquery rule

2022-09-05 Thread GitBox
924060929 merged PR #12345: URL: https://github.com/apache/doris/pull/12345 -- 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] adonis0147 commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
adonis0147 commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963226809 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,184 @@ +// Licensed to the Apache Software Foundation (ASF) unde

[GitHub] [doris] adonis0147 commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
adonis0147 commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963226691 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,184 @@ +// Licensed to the Apache Software Foundation (ASF) unde

[GitHub] [doris] github-actions[bot] commented on pull request #12240: [enhancement](test) add some p0 cases

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12240: URL: https://github.com/apache/doris/pull/12240#issuecomment-1237625271 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 #12240: [enhancement](test) add some p0 cases

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12240: URL: https://github.com/apache/doris/pull/12240#issuecomment-1237625258 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] qzsee commented on pull request #12366: [Fix] add hll param for if function

2022-09-05 Thread GitBox
qzsee commented on PR #12366: URL: https://github.com/apache/doris/pull/12366#issuecomment-1237622800 > SELECT hll_union_agg(if(xxx='xxx', xx_id, hll_empty())) from xxx where xxx = '2022-08-10' and xx = '11' -- This is an automated message from the Apache Git Servi

[GitHub] [doris] zhengshiJ opened a new pull request, #12372: [test](nereids) add subquery regression Testing

2022-09-05 Thread GitBox
zhengshiJ opened a new pull request, #12372: URL: https://github.com/apache/doris/pull/12372 # Proposed changes ## Problem summary Added regression test of sub-queries. Currently only associated sub-queries are added. Non-associated sub-queries will be added after proje

[GitHub] [doris] cambyzju commented on a diff in pull request #12334: [Opt](vectorized) speed up hash function compute in hash partition

2022-09-05 Thread GitBox
cambyzju commented on code in PR #12334: URL: https://github.com/apache/doris/pull/12334#discussion_r963222126 ## be/src/vec/columns/column.h: ## @@ -286,6 +299,12 @@ class IColumn : public COW { /// passed bytes to hash must identify sequence of values unambiguously.

[GitHub] [doris] dataroaring opened a new pull request, #12371: [chore](ut) enable asan core dump when running be ut

2022-09-05 Thread GitBox
dataroaring opened a new pull request, #12371: URL: https://github.com/apache/doris/pull/12371 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ]

[GitHub] [doris] dataroaring opened a new pull request, #12370: [fix](threadpool) threadpool schedules does not work right on concurr…

2022-09-05 Thread GitBox
dataroaring opened a new pull request, #12370: URL: https://github.com/apache/doris/pull/12370 …ent token Assuming there is a concurrent thread token whose concurrency is 2, and the 1st submit on the token is submitted to threadpool while the 2nd is not submitted due to busy. T

[doris] branch master updated: [enhancement](array-type) Split Array Offsets and String Offsets (#12341)

2022-09-05 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 cf5d194fe1 [enhancement](array-type) Split Arra

[GitHub] [doris] yiguolei merged pull request #12341: [enhancement](array-type) Split Array Offsets and String Offsets

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

[GitHub] [doris] github-actions[bot] commented on pull request #12334: [Opt](vectorized) speed up hash function compute in hash partition

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12334: URL: https://github.com/apache/doris/pull/12334#issuecomment-1237611611 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 #12334: [Opt](vectorized) speed up hash function compute in hash partition

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12334: URL: https://github.com/apache/doris/pull/12334#issuecomment-1237611594 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] cambyzju commented on a diff in pull request #12321: [Function](ELT)Add elt function for doris

2022-09-05 Thread GitBox
cambyzju commented on code in PR #12321: URL: https://github.com/apache/doris/pull/12321#discussion_r963213409 ## gensrc/script/doris_builtins_functions.py: ## @@ -2012,6 +2012,12 @@ [['concat'], 'VARCHAR', ['VARCHAR', '...'], '_ZN5doris15StringFunctions6conca

[GitHub] [doris] cambyzju commented on a diff in pull request #12321: [Function](ELT)Add elt function for doris

2022-09-05 Thread GitBox
cambyzju commented on code in PR #12321: URL: https://github.com/apache/doris/pull/12321#discussion_r963213058 ## be/src/vec/functions/function_string.h: ## @@ -494,6 +494,51 @@ class FunctionStringConcat : public IFunction { } }; +class FunctionStringElt : public IFunct

[GitHub] [doris] dataroaring opened a new pull request, #12369: (threadpool) threadpool schedules does not work right on concurrent t…

2022-09-05 Thread GitBox
dataroaring opened a new pull request, #12369: URL: https://github.com/apache/doris/pull/12369 …oken Assuming there is a concurrent thread token whose concurrency is 2, and the 1st submit on the token is submitted to threadpool while the 2nd is not submitted due to busy. The to

[GitHub] [doris] Henry2SS opened a new issue, #12368: [Enhancement] broker load should not close file system when exception happens

2022-09-05 Thread GitBox
Henry2SS opened a new issue, #12368: URL: https://github.com/apache/doris/issues/12368 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Description version:

[GitHub] [doris] Henry2SS opened a new pull request, #12367: [enhancement](broker load) file system should not be closed when exception happens

2022-09-05 Thread GitBox
Henry2SS opened a new pull request, #12367: URL: https://github.com/apache/doris/pull/12367 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Ye

[GitHub] [doris] qzsee commented on a diff in pull request #12151: [Nereids][Feature] Constant fold of an expression

2022-09-05 Thread GitBox
qzsee commented on code in PR #12151: URL: https://github.com/apache/doris/pull/12151#discussion_r963205692 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rewrite/rules/FoldConstantRule.java: ## @@ -0,0 +1,440 @@ +// Licensed to the Apache Software Foundat

[GitHub] [doris] cambyzju commented on pull request #12364: [feature-wip](unique-key-merge-on-write) update delete bitmap when increamental clone

2022-09-05 Thread GitBox
cambyzju commented on PR #12364: URL: https://github.com/apache/doris/pull/12364#issuecomment-1237598923 Please add some description or link for 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

[GitHub] [doris] qzsee commented on a diff in pull request #12151: [Nereids][Feature] Constant fold of an expression

2022-09-05 Thread GitBox
qzsee commented on code in PR #12151: URL: https://github.com/apache/doris/pull/12151#discussion_r963205097 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/literal/Literal.java: ## @@ -82,6 +107,90 @@ public R accept(ExpressionVisitor visitor, C context)

[GitHub] [doris] cambyzju commented on pull request #12366: [Fix] add hll param for if function

2022-09-05 Thread GitBox
cambyzju commented on PR #12366: URL: https://github.com/apache/doris/pull/12366#issuecomment-1237598190 Which SQL print the error message? -- 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 spe

[GitHub] [doris] qzsee commented on a diff in pull request #12151: [Nereids][Feature] Constant fold of an expression

2022-09-05 Thread GitBox
qzsee commented on code in PR #12151: URL: https://github.com/apache/doris/pull/12151#discussion_r963204841 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/literal/Literal.java: ## @@ -82,6 +107,90 @@ public R accept(ExpressionVisitor visitor, C context)

[GitHub] [doris] qzsee commented on a diff in pull request #12151: [Nereids][Feature] Constant fold of an expression

2022-09-05 Thread GitBox
qzsee commented on code in PR #12151: URL: https://github.com/apache/doris/pull/12151#discussion_r963201990 ## fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java: ## @@ -916,6 +917,9 @@ private Expression withPredicate(Expression valueExpression, P

[doris] branch master updated: [Enhancement](restore) new add the property of reserve_replica to restore statement (#11942)

2022-09-05 Thread lide
This is an automated email from the ASF dual-hosted git repository. lide 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 53b79d5a8c [Enhancement](restore) new add the prope

[GitHub] [doris] lide-reed closed issue #11941: [Enhancement] (restore) restore table or partition with the same replication_num as before the backup

2022-09-05 Thread GitBox
lide-reed closed issue #11941: [Enhancement] (restore) restore table or partition with the same replication_num as before the backup URL: https://github.com/apache/doris/issues/11941 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [doris] lide-reed merged pull request #11942: [Enhancement](restore) restore table or partition with the same replication num as before the backup

2022-09-05 Thread GitBox
lide-reed merged PR #11942: URL: https://github.com/apache/doris/pull/11942 -- 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] qzsee opened a new pull request, #12366: [Fix] add hll param for if function

2022-09-05 Thread GitBox
qzsee opened a new pull request, #12366: URL: https://github.com/apache/doris/pull/12366 # Proposed changes Issue Number: close #xxx ## Problem summary ERROR 1105 (HY000): errCode = 2, detailMessage = No matching function with signature: if(boolean, hll, hll) Desc

[GitHub] [doris] morrySnow opened a new pull request, #12365: [enhancement](chore)add single empty line rule to fe check style for Nereids

2022-09-05 Thread GitBox
morrySnow opened a new pull request, #12365: URL: https://github.com/apache/doris/pull/12365 # Proposed changes reject multi empty line ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Ye

[GitHub] [doris] yiguolei commented on a diff in pull request #12334: [Opt](vectorized) speed up hash function compute in hash partition

2022-09-05 Thread GitBox
yiguolei commented on code in PR #12334: URL: https://github.com/apache/doris/pull/12334#discussion_r963198602 ## be/src/vec/sink/vdata_stream_sender.cpp: ## @@ -466,7 +466,6 @@ Status VDataStreamSender::send(RuntimeState* state, RowBatch* batch) { Status VDataStreamSender::

[GitHub] [doris] freemandealer commented on pull request #12360: [Enhancement](log) improve error msg for delta writer fail (#12121)

2022-09-05 Thread GitBox
freemandealer commented on PR #12360: URL: https://github.com/apache/doris/pull/12360#issuecomment-1237589095 > It need be code format. sure, will do. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [doris] yiguolei merged pull request #12268: [regression](test) add tpcds sf1 unique test

2022-09-05 Thread GitBox
yiguolei merged PR #12268: URL: https://github.com/apache/doris/pull/12268 -- 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] liaoxin01 opened a new pull request, #12364: [feature-wip](unique-key-merge-on-write) update delete bitmap when increamental clone

2022-09-05 Thread GitBox
liaoxin01 opened a new pull request, #12364: URL: https://github.com/apache/doris/pull/12364 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [

[GitHub] [doris] morrySnow commented on a diff in pull request #12129: [feature-wip](nereids) Support having clause

2022-09-05 Thread GitBox
morrySnow commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r963187348 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/ResolveHaving.java: ## @@ -0,0 +1,184 @@ +// Licensed to the Apache Software Foundation (ASF) under

[doris] branch master updated (f2aa87d797 -> 86fa0e38e2)

2022-09-05 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 f2aa87d797 Add ctas support config key type ut and doc. (#12327) add 86fa0e38e2 [fix](join) hash join should use

[GitHub] [doris] yiguolei merged pull request #12261: [fix](join) hash join should use children's output tuple ids instead of output tableref ids

2022-09-05 Thread GitBox
yiguolei merged PR #12261: URL: https://github.com/apache/doris/pull/12261 -- 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] liaoxin01 opened a new pull request, #12363: [feature-wip](unique-key-merge-on-write) fix that versions of multiple replicas are inconsistent when rebalance

2022-09-05 Thread GitBox
liaoxin01 opened a new pull request, #12363: URL: https://github.com/apache/doris/pull/12363 # Proposed changes Issue Number: close #xxx ## Problem summary uniq key with merge-on-write version is not continuous when publish transaction,return an error message to FE.

[GitHub] [doris] stalary commented on pull request #12344: [regression](left join)Add left join, the left table is empty, the query result is not empty…

2022-09-05 Thread GitBox
stalary commented on PR #12344: URL: https://github.com/apache/doris/pull/12344#issuecomment-1237563475 It seems that the description is wrong, departments as the left table but not empty? -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [doris] stalary commented on pull request #12360: [Enhancement](log) improve error msg for delta writer fail (#12121)

2022-09-05 Thread GitBox
stalary commented on PR #12360: URL: https://github.com/apache/doris/pull/12360#issuecomment-1237562160 It need be code format. -- 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

[doris] branch master updated: Add ctas support config key type ut and doc. (#12327)

2022-09-05 Thread stalary
This is an automated email from the ASF dual-hosted git repository. stalary 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 f2aa87d797 Add ctas support config key type ut a

[GitHub] [doris] stalary merged pull request #12327: [Doc](CTAS) Mod ut and doc, add key type desc.

2022-09-05 Thread GitBox
stalary merged PR #12327: URL: https://github.com/apache/doris/pull/12327 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apac

[GitHub] [doris] stalary closed issue #12326: [Enhancement] CTAS support user config table key type.

2022-09-05 Thread GitBox
stalary closed issue #12326: [Enhancement] CTAS support user config table key type. URL: https://github.com/apache/doris/issues/12326 -- 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 com

[GitHub] [doris] github-actions[bot] commented on pull request #12357: [Bug](remote) Fix BE crash because of call the future's get method twice

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12357: URL: https://github.com/apache/doris/pull/12357#issuecomment-1237559987 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 #12357: [Bug](remote) Fix BE crash because of call the future's get method twice

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12357: URL: https://github.com/apache/doris/pull/12357#issuecomment-1237559971 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 #8370: Support get metric by name

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #8370: URL: https://github.com/apache/doris/pull/8370#issuecomment-1237538038 We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.

[GitHub] [doris] freemandealer opened a new pull request, #12362: [bug] fix OOM caused by pending queue in sink send (#12359)

2022-09-05 Thread GitBox
freemandealer opened a new pull request, #12362: URL: https://github.com/apache/doris/pull/12362 Each NodeChannel has its own queue, with size up to 1/20 exec_mem_limit. User will crash into OOM if set exec_mem_limit high. This commit uses fixed number to control the total max memory us

[GitHub] [doris] spaces-X commented on a diff in pull request #12340: [Enhancement](spark-load) enable spark-load implicitly cast to bitmap

2022-09-05 Thread GitBox
spaces-X commented on code in PR #12340: URL: https://github.com/apache/doris/pull/12340#discussion_r963053771 ## fe/fe-core/src/main/java/org/apache/doris/analysis/CastExpr.java: ## @@ -142,11 +142,6 @@ private static boolean disableRegisterCastingFunction(Type fromType, Type

[GitHub] [doris] spaces-X commented on a diff in pull request #12340: [Enhancement](spark-load) enable spark-load implicitly cast to bitmap

2022-09-05 Thread GitBox
spaces-X commented on code in PR #12340: URL: https://github.com/apache/doris/pull/12340#discussion_r963053771 ## fe/fe-core/src/main/java/org/apache/doris/analysis/CastExpr.java: ## @@ -142,11 +142,6 @@ private static boolean disableRegisterCastingFunction(Type fromType, Type

[GitHub] [doris] morningman opened a new pull request, #12361: [refactor] move broker scan node to file scan node

2022-09-05 Thread GitBox
morningman opened a new pull request, #12361: URL: https://github.com/apache/doris/pull/12361 # 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 #12340: [Enhancement](spark-load) enable spark-load implicitly cast to bitmap

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12340: URL: https://github.com/apache/doris/pull/12340#issuecomment-1237250627 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 #12340: [Enhancement](spark-load) enable spark-load implicitly cast to bitmap

2022-09-05 Thread GitBox
github-actions[bot] commented on PR #12340: URL: https://github.com/apache/doris/pull/12340#issuecomment-1237250589 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] ChenpiDog commented on issue #12253: [Feature] [Real time data warehouse]CDC and Materialized View

2022-09-05 Thread GitBox
ChenpiDog commented on issue #12253: URL: https://github.com/apache/doris/issues/12253#issuecomment-1237149011 @stalary 1 and 2 are enough to implement one. If the change of 1 is too large, Doris can consider attacking 2. Make the real-time materialized view as compatible with the real tabl

[GitHub] [doris] freemandealer opened a new pull request, #12360: [Enhancement] improve error msg for delta writer fail (#12121)

2022-09-05 Thread GitBox
freemandealer opened a new pull request, #12360: URL: https://github.com/apache/doris/pull/12360 Signed-off-by: freemandealer # Proposed changes Issue Number: close #12121 ## Problem summary fuzzy error msg when delta writer writing failed Describe your changes

[GitHub] [doris] freemandealer opened a new issue, #12359: [Bug] [Load] NodeChannel pending batches causing OOM with large exec_mem_limit and/or high load concurrency

2022-09-05 Thread GitBox
freemandealer opened a new issue, #12359: URL: https://github.com/apache/doris/issues/12359 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Version master

[doris] branch master updated: [enhancement](chore)add single space separator rule to fe check style (#12354)

2022-09-05 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 190717dbcc [enhancement](chore)add single spac

[GitHub] [doris] morrySnow merged pull request #12354: [enhancement](chore)add single space separator rule to fe check style

2022-09-05 Thread GitBox
morrySnow merged PR #12354: URL: https://github.com/apache/doris/pull/12354 -- 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] morrySnow opened a new pull request, #12358: [enhancement](explain)add projections and output id in explain string

2022-09-05 Thread GitBox
morrySnow opened a new pull request, #12358: URL: https://github.com/apache/doris/pull/12358 # Proposed changes In earlier PR #11842, we add the ability of projection on each ExecNode. But, we cannot get the projection expr list in explain. This is inconvenience to debug. This P

[GitHub] [doris] eldenmoon commented on a diff in pull request #12293: [Enhancement](array-type) record offsets info to speed up array colum…

2022-09-05 Thread GitBox
eldenmoon commented on code in PR #12293: URL: https://github.com/apache/doris/pull/12293#discussion_r962873840 ## be/src/common/config.h: ## @@ -849,6 +849,8 @@ CONF_Bool(enable_simdjson_reader, "false"); CONF_Int32(doris_remote_scanner_thread_pool_thread_num, "16"); // numbe

[GitHub] [doris] eldenmoon commented on a diff in pull request #12293: [Enhancement](array-type) record offsets info to speed up array colum…

2022-09-05 Thread GitBox
eldenmoon commented on code in PR #12293: URL: https://github.com/apache/doris/pull/12293#discussion_r962868057 ## be/src/common/config.h: ## @@ -849,6 +849,8 @@ CONF_Bool(enable_simdjson_reader, "false"); CONF_Int32(doris_remote_scanner_thread_pool_thread_num, "16"); // numbe

[GitHub] [doris] zxealous opened a new pull request, #12357: [Bug] (remote) Fix BE crash because of call the future's get method twice

2022-09-05 Thread GitBox
zxealous opened a new pull request, #12357: URL: https://github.com/apache/doris/pull/12357 # Proposed changes Issue Number: close #12355 ## Problem summary call the future's get method once and save it. ## Checklist(Required) 1. Does it affect the original

[doris] branch master updated (0deee72a63 -> b8e38b9167)

2022-09-05 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 0deee72a63 About the modification of broker load specifying hdfs user name parameter (#12330) add b8e38b9167 [B

[GitHub] [doris] yiguolei merged pull request #12350: [Bug](load) block call clear_column_data may have ref not equal 1

2022-09-05 Thread GitBox
yiguolei merged PR #12350: URL: https://github.com/apache/doris/pull/12350 -- 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] wangshuo128 opened a new pull request, #12356: [Refactor](Nereids) Simplify get input and output slots for plan/expression.

2022-09-05 Thread GitBox
wangshuo128 opened a new pull request, #12356: URL: https://github.com/apache/doris/pull/12356 # Proposed changes ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Yes - [ ] No

[GitHub] [doris] zxealous opened a new issue, #12355: [Bug] BE crash because of the get method of future is called multiple times

2022-09-05 Thread GitBox
zxealous opened a new issue, #12355: URL: https://github.com/apache/doris/issues/12355 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Version master

[GitHub] [doris] cambyzju commented on a diff in pull request #12293: [Enhancement](array-type) record offsets info to speed up array colum…

2022-09-05 Thread GitBox
cambyzju commented on code in PR #12293: URL: https://github.com/apache/doris/pull/12293#discussion_r962842996 ## be/src/common/config.h: ## @@ -849,6 +849,8 @@ CONF_Bool(enable_simdjson_reader, "false"); CONF_Int32(doris_remote_scanner_thread_pool_thread_num, "16"); // number

[GitHub] [doris] 2994265077 commented on issue #12215: [Bug] Routine load Invalid number format

2022-09-05 Thread GitBox
2994265077 commented on issue #12215: URL: https://github.com/apache/doris/issues/12215#issuecomment-1236921764 My data is validated. This problem actually occurs before the routine load task is created, so its not a data problem. I solved the problem by drop a materialized view of my table

[GitHub] [doris] morrySnow commented on a diff in pull request #11976: [feature](Nereids)support join algorithm in Nereids that already implemented in original planner

2022-09-05 Thread GitBox
morrySnow commented on code in PR #11976: URL: https://github.com/apache/doris/pull/11976#discussion_r962838334 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalDistribute.java: ## @@ -35,30 +36,40 @@ /** * Enforcer plan. */ -public class Ph

  1   2   >