[GitHub] [doris] github-actions[bot] commented on a diff in pull request #15317: [enhancement](histogram) optimise aggregate function histogram

2022-12-22 Thread GitBox
github-actions[bot] commented on code in PR #15317: URL: https://github.com/apache/doris/pull/15317#discussion_r1056120029 ## be/test/vec/aggregate_functions/agg_histogram_test.cpp: ## @@ -80,8 +116,8 @@ class VAggHistogramTest : public testing::Test { std::unique_ptr m

[GitHub] [doris] weizhengte opened a new pull request, #15317: [enhancement](histogram) optimise aggregate function histogram

2022-12-22 Thread GitBox
weizhengte opened a new pull request, #15317: URL: https://github.com/apache/doris/pull/15317 # Proposed changes This pr mainly to optimize the histogram aggregation function. Including the following: 1. Support input parameters `sample_rate` and `max_bucket_num` 2. Add UT and r

[GitHub] [doris] github-actions[bot] commented on pull request #15087: [feature-wip](nereids) Made decimal in nereids more complete

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15087: URL: https://github.com/apache/doris/pull/15087#issuecomment-1363706054 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 #15087: [feature-wip](nereids) Made decimal in nereids more complete

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15087: URL: https://github.com/apache/doris/pull/15087#issuecomment-1363706015 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 #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15312: URL: https://github.com/apache/doris/pull/15312#issuecomment-1363703988 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 #15296: [fix](bitmapfilter) fix core dump caused by bitmap filter

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15296: URL: https://github.com/apache/doris/pull/15296#issuecomment-1363703305 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 #15296: [fix](bitmapfilter) fix core dump caused by bitmap filter

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15296: URL: https://github.com/apache/doris/pull/15296#issuecomment-1363703325 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] weizhengte closed pull request #15309: Draft

2022-12-22 Thread GitBox
weizhengte closed pull request #15309: Draft URL: https://github.com/apache/doris/pull/15309 -- 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-uns

[GitHub] [doris] Gabriel39 opened a new pull request, #15316: [bug](decimalv3) Fix wrong decimal scale for arithmetic expr

2022-12-22 Thread GitBox
Gabriel39 opened a new pull request, #15316: URL: https://github.com/apache/doris/pull/15316 # 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 #15298: [bugfix](from_unixtime) fix timezone not work for from_unixtime

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15298: URL: https://github.com/apache/doris/pull/15298#issuecomment-1363697628 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 #15298: [bugfix](from_unixtime) fix timezone not work for from_unixtime

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15298: URL: https://github.com/apache/doris/pull/15298#issuecomment-1363697595 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] morrySnow commented on pull request #14044: [Enhancement](Nereids)optimize groupExpressionMatch

2022-12-22 Thread GitBox
morrySnow commented on PR #14044: URL: https://github.com/apache/doris/pull/14044#issuecomment-1363694634 many conflict and need to rethink how to do 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

[GitHub] [doris] morrySnow closed pull request #14044: [Enhancement](Nereids)optimize groupExpressionMatch

2022-12-22 Thread GitBox
morrySnow closed pull request #14044: [Enhancement](Nereids)optimize groupExpressionMatch URL: https://github.com/apache/doris/pull/14044 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [doris] morrySnow merged pull request #14807: [Enhancement](Nereids)change expression to conjuncts in filter

2022-12-22 Thread GitBox
morrySnow merged PR #14807: URL: https://github.com/apache/doris/pull/14807 -- 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 #15306: [pipeline](fix) Fix bugs to pass all regression cases

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15306: URL: https://github.com/apache/doris/pull/15306#issuecomment-1363691868 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 #14807: [Enhancement](Nereids)change expression to conjuncts in filter

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #14807: URL: https://github.com/apache/doris/pull/14807#issuecomment-1363690316 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 #14807: [Enhancement](Nereids)change expression to conjuncts in filter

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #14807: URL: https://github.com/apache/doris/pull/14807#issuecomment-1363690286 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 #15262: [deps](fe)upgrade deps version

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15262: URL: https://github.com/apache/doris/pull/15262#issuecomment-1363688623 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 #15274: [fix](multi-catalog) fix and optimize iceberg v2 reader

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15274: URL: https://github.com/apache/doris/pull/15274#issuecomment-1363685412 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] zy-kkk commented on issue #15285: join table Be OOM

2022-12-22 Thread GitBox
zy-kkk commented on issue #15285: URL: https://github.com/apache/doris/issues/15285#issuecomment-1363682283 可以升级1.2测试下,1.2对join内存做了很大优化,可以添加我的微信`zyk815574403`,我们后续跟进一下这个问题 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [doris] github-actions[bot] commented on pull request #15274: [fix](multi-catalog) fix and optimize iceberg v2 reader

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15274: URL: https://github.com/apache/doris/pull/15274#issuecomment-1363678190 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 #15219: [Improvement](JSONB) improve performance JSONB initial json parsing using simdjson

2022-12-22 Thread GitBox
hello-stephen commented on PR #15219: URL: https://github.com/apache/doris/pull/15219#issuecomment-1363665595 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.66 seconds load time: 674 seconds storage size: 17123137353 Bytes https://doris-co

[doris] branch master updated (cb295de981 -> fe562bc3e7)

2022-12-22 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 cb295de981 [Bug](decimalv3) Fix wrong precision of DECIMALV3 (#15302) add fe562bc3e7 [Bug](Agg) fix crash when e

[GitHub] [doris] yiguolei closed issue #15256: [Bug] unsupported agg fn init would cause coredump

2022-12-22 Thread GitBox
yiguolei closed issue #15256: [Bug] unsupported agg fn init would cause coredump URL: https://github.com/apache/doris/issues/15256 -- 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] yiguolei merged pull request #15257: [Bug](Agg) fix crash when encountering not supported agg function like last_value(bitmap)

2022-12-22 Thread GitBox
yiguolei merged PR #15257: URL: https://github.com/apache/doris/pull/15257 -- 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] hello-stephen commented on pull request #15308: [Improvement](meta) support rename hive and jdbc engine external table

2022-12-22 Thread GitBox
hello-stephen commented on PR #15308: URL: https://github.com/apache/doris/pull/15308#issuecomment-1363655366 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.11 seconds load time: 641 seconds storage size: 17123143033 Bytes https://doris-co

[GitHub] [doris] hf200012 closed pull request #15315: [chore](fix)update workflow thrift version

2022-12-22 Thread GitBox
hf200012 closed pull request #15315: [chore](fix)update workflow thrift version URL: https://github.com/apache/doris/pull/15315 -- 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] github-actions[bot] commented on pull request #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15312: URL: https://github.com/apache/doris/pull/15312#issuecomment-1363653772 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

[doris] branch master updated (b085ff49f0 -> cb295de981)

2022-12-22 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 b085ff49f0 [refactor](non-vec) delete non-vec data sink (#15283) add cb295de981 [Bug](decimalv3) Fix wrong preci

[GitHub] [doris] yiguolei merged pull request #15302: [Bug](decimalv3) Fix wrong precision of DECIMALV3

2022-12-22 Thread GitBox
yiguolei merged PR #15302: URL: https://github.com/apache/doris/pull/15302 -- 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] yiguolei merged pull request #15283: [refactor](non-vec) delete non-vec data sink

2022-12-22 Thread GitBox
yiguolei merged PR #15283: URL: https://github.com/apache/doris/pull/15283 -- 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 (38530100d8 -> b085ff49f0)

2022-12-22 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 38530100d8 [fix](localgc) check gc only cache directory (#15238) add b085ff49f0 [refactor](non-vec) delete non-v

[GitHub] [doris] hf200012 opened a new pull request, #15315: [chore](fix)update thrift version

2022-12-22 Thread GitBox
hf200012 opened a new pull request, #15315: URL: https://github.com/apache/doris/pull/15315 # 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 #15306: [pipeline](fix) Fix bugs to pass all regression cases

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15306: URL: https://github.com/apache/doris/pull/15306#issuecomment-1363650468 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 #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15312: URL: https://github.com/apache/doris/pull/15312#issuecomment-1363648698 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 #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on code in PR #15312: URL: https://github.com/apache/doris/pull/15312#discussion_r1056061188 ## be/test/vec/function/function_test_util.h: ## @@ -233,7 +234,8 @@ fn_ctx_return.type = doris_udf::FunctionContext::TYPE_BOOLEAN; } else if

[GitHub] [doris] github-actions[bot] commented on pull request #15277: [fix](compile) fix compile error caused by `mysql_scan_node.cpp` not being found when enabling `WITH_MYSQL`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15277: URL: https://github.com/apache/doris/pull/15277#issuecomment-1363632647 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 #15143: [fix](having-clause) having clause do not works correct with same alias name

2022-12-22 Thread GitBox
cambyzju commented on code in PR #15143: URL: https://github.com/apache/doris/pull/15143#discussion_r1056045507 ## fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java: ## @@ -989,18 +989,36 @@ private void analyzeAggregation(Analyzer analyzer) throws AnalysisExce

[GitHub] [doris] github-actions[bot] commented on pull request #15313: [refactor](non-vec)remove schema change related non-vec code

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15313: URL: https://github.com/apache/doris/pull/15313#issuecomment-1363613812 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [doris] starocean999 opened a new pull request, #15314: [fix](union)the union node should not pass through children in some case

2022-12-22 Thread GitBox
starocean999 opened a new pull request, #15314: URL: https://github.com/apache/doris/pull/15314 # Proposed changes Issue Number: close #xxx ## Problem summary pick from master https://github.com/apache/doris/pull/15286 ## Checklist(Required) 1. Does it affec

[GitHub] [doris] github-actions[bot] commented on pull request #15306: [pipeline](fix) Fix bugs to pass all regression cases

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15306: URL: https://github.com/apache/doris/pull/15306#issuecomment-1363613589 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 #15306: [pipeline](fix) Fix bugs to pass all regression cases

2022-12-22 Thread GitBox
hello-stephen commented on PR #15306: URL: https://github.com/apache/doris/pull/15306#issuecomment-1363613194 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.93 seconds load time: 667 seconds storage size: 17123619600 Bytes https://doris-co

[GitHub] [doris] yiguolei opened a new pull request, #15313: [refactor](non-vec)remove schema change related non-vec code

2022-12-22 Thread GitBox
yiguolei opened a new pull request, #15313: URL: https://github.com/apache/doris/pull/15313 # 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 #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15312: URL: https://github.com/apache/doris/pull/15312#issuecomment-1363611462 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 #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15312: URL: https://github.com/apache/doris/pull/15312#issuecomment-1363610425 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] Gabriel39 opened a new pull request, #15312: [Bug](timediff) Fix wrong result for function `timediff`

2022-12-22 Thread GitBox
Gabriel39 opened a new pull request, #15312: URL: https://github.com/apache/doris/pull/15312 # Proposed changes before: ![image](https://user-images.githubusercontent.com/37700562/209271986-da1a316c-6019-4c52-8b48-34ea8e8fcbb3.png) after: ![image](https://user-images.git

[GitHub] [doris] hello-stephen commented on pull request #15304: [fix](jdbc-catalog) fix close connection when use jdbc.db which table…

2022-12-22 Thread GitBox
hello-stephen commented on PR #15304: URL: https://github.com/apache/doris/pull/15304#issuecomment-1363603450 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.62 seconds load time: 620 seconds storage size: 17122945035 Bytes https://doris-co

[GitHub] [doris] hello-stephen commented on pull request #15303: [vectorized](jdbc) support array type in jdbc external table

2022-12-22 Thread GitBox
hello-stephen commented on PR #15303: URL: https://github.com/apache/doris/pull/15303#issuecomment-1363602784 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.23 seconds load time: 629 seconds storage size: 17123605735 Bytes https://doris-co

[GitHub] [doris] Kikyou1997 opened a new pull request, #15311: [feature-wip](nereids) Implement using join

2022-12-22 Thread GitBox
Kikyou1997 opened a new pull request, #15311: URL: https://github.com/apache/doris/pull/15311 # 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 #15143: [fix](having-clause) having clause do not works correct with same alias name

2022-12-22 Thread GitBox
morrySnow commented on code in PR #15143: URL: https://github.com/apache/doris/pull/15143#discussion_r1056025197 ## fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java: ## @@ -989,18 +989,36 @@ private void analyzeAggregation(Analyzer analyzer) throws AnalysisExc

[GitHub] [doris] hello-stephen commented on pull request #15302: [Bug](decimalv3) Fix wrong precision of DECIMALV3

2022-12-22 Thread GitBox
hello-stephen commented on PR #15302: URL: https://github.com/apache/doris/pull/15302#issuecomment-1363588318 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 36.25 seconds load time: 692 seconds storage size: 17123606957 Bytes https://doris-co

[GitHub] [doris] JackDrogon commented on a diff in pull request #15250: [feature] Add auto bucket implement

2022-12-22 Thread GitBox
JackDrogon commented on code in PR #15250: URL: https://github.com/apache/doris/pull/15250#discussion_r1056019591 ## fe/fe-core/src/main/java/org/apache/doris/common/util/AutoBucketUtils.java: ## @@ -0,0 +1,97 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] JackDrogon commented on a diff in pull request #15250: [feature] Add auto bucket implement

2022-12-22 Thread GitBox
JackDrogon commented on code in PR #15250: URL: https://github.com/apache/doris/pull/15250#discussion_r1056017197 ## fe/fe-core/src/main/java/org/apache/doris/common/util/PropertyAnalyzer.java: ## @@ -93,6 +93,9 @@ public class PropertyAnalyzer { public static final Strin

[GitHub] [doris] JackDrogon commented on a diff in pull request #15250: [feature] Add auto bucket implement

2022-12-22 Thread GitBox
JackDrogon commented on code in PR #15250: URL: https://github.com/apache/doris/pull/15250#discussion_r1056016539 ## fe/fe-core/src/main/java/org/apache/doris/common/util/AutoBucketUtils.java: ## @@ -0,0 +1,97 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] JackDrogon commented on a diff in pull request #15250: [feature] Add auto bucket implement

2022-12-22 Thread GitBox
JackDrogon commented on code in PR #15250: URL: https://github.com/apache/doris/pull/15250#discussion_r1056016421 ## fe/fe-core/src/main/java/org/apache/doris/common/util/AutoBucketUtils.java: ## @@ -0,0 +1,97 @@ +// Licensed to the Apache Software Foundation (ASF) under one +//

[GitHub] [doris] JackDrogon commented on a diff in pull request #15250: [feature] Add auto bucket implement

2022-12-22 Thread GitBox
JackDrogon commented on code in PR #15250: URL: https://github.com/apache/doris/pull/15250#discussion_r1056015723 ## fe/fe-core/src/main/java/org/apache/doris/clone/DynamicPartitionScheduler.java: ## @@ -140,6 +142,69 @@ private Map createDefaultRuntimeInfo() { return d

[GitHub] [doris] github-actions[bot] commented on pull request #15277: [fix](compile) fix compile error caused by `mysql_scan_node.cpp` not being found when enabling `WITH_MYSQL`

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15277: URL: https://github.com/apache/doris/pull/15277#issuecomment-1363578239 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 commented on a diff in pull request #15107: [refactor](fe) remove non vectorized code in fe

2022-12-22 Thread GitBox
morningman commented on code in PR #15107: URL: https://github.com/apache/doris/pull/15107#discussion_r1056013062 ## fe/fe-core/src/main/java/org/apache/doris/qe/Coordinator.java: ## @@ -318,9 +318,10 @@ private void setFromUserProperty(Analyzer analyzer) { private void i

[GitHub] [doris] github-actions[bot] commented on pull request #15277: [fix](compile) fix compile error caused by `mysql_scan_node.cpp` not being found

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15277: URL: https://github.com/apache/doris/pull/15277#issuecomment-1363576414 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 #15298: [bugfix](from_unixtime) fix timezone not work for from_unixtime

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15298: URL: https://github.com/apache/doris/pull/15298#issuecomment-1363575482 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 #15309: [enhancement](histogram) optimise aggregate function histogram

2022-12-22 Thread GitBox
github-actions[bot] commented on code in PR #15309: URL: https://github.com/apache/doris/pull/15309#discussion_r1056005592 ## be/src/vec/utils/histogram_helpers.hpp: ## @@ -0,0 +1,299 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor licen

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #15298: [bugfix](from_unixtime) fix timezone not work for from_unixtime

2022-12-22 Thread GitBox
github-actions[bot] commented on code in PR #15298: URL: https://github.com/apache/doris/pull/15298#discussion_r1056004413 ## be/test/exprs/timestamp_functions_test.cpp: ## @@ -41,19 +40,11 @@ class TimestampFunctionsTest : public testing::Test { TimestampFunctionsTest() {}

[GitHub] [doris] qzsee opened a new issue, #15310: [Fix](Nerids) fix dup key for pull predicate from project children #15292

2022-12-22 Thread GitBox
qzsee opened a new issue, #15310: URL: https://github.com/apache/doris/issues/15310 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Version master ### What's W

[GitHub] [doris] github-actions[bot] commented on pull request #15218: [feature](planner) remove restrict of offset without order by

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15218: URL: https://github.com/apache/doris/pull/15218#issuecomment-1363563087 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 #15218: [feature](planner) remove restrict of offset without order by

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15218: URL: https://github.com/apache/doris/pull/15218#issuecomment-1363563071 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[doris] branch branch-1.2-lts updated: [feature](planner) mark join to support subquery in disjunction (#14579) (#15291)

2022-12-22 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a commit to branch branch-1.2-lts in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/branch-1.2-lts by this push: new fd51474649 [feature](planner)

[GitHub] [doris] morningman merged pull request #15291: [feature](planner) mark join to support subquery in disjunction (#14579)

2022-12-22 Thread GitBox
morningman merged PR #15291: URL: https://github.com/apache/doris/pull/15291 -- 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] github-actions[bot] commented on pull request #15249: [pipeline](test) test regression

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15249: URL: https://github.com/apache/doris/pull/15249#issuecomment-1363560882 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 #15309: [enhancement](histogram) optimise aggregate function histogram

2022-12-22 Thread GitBox
github-actions[bot] commented on code in PR #15309: URL: https://github.com/apache/doris/pull/15309#discussion_r1055999472 ## be/src/vec/utils/histogram_helpers.hpp: ## @@ -0,0 +1,299 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor licen

[GitHub] [doris] github-actions[bot] commented on pull request #15249: [pipeline](test) test regression

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15249: URL: https://github.com/apache/doris/pull/15249#issuecomment-1363559392 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] weizhengte opened a new pull request, #15309: [enhancement](histogram) optimise aggregate function histogram

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

[doris] branch master updated: [fix](localgc) check gc only cache directory (#15238)

2022-12-22 Thread pengxiangyu
This is an automated email from the ASF dual-hosted git repository. pengxiangyu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 38530100d8 [fix](localgc) check gc only cach

[GitHub] [doris] pengxiangyu closed issue #15237: [Bug] non-cache directory into local gc

2022-12-22 Thread GitBox
pengxiangyu closed issue #15237: [Bug] non-cache directory into local gc URL: https://github.com/apache/doris/issues/15237 -- 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 un

[GitHub] [doris] pengxiangyu merged pull request #15238: [fix](localgc) check gc only cache directory

2022-12-22 Thread GitBox
pengxiangyu merged PR #15238: URL: https://github.com/apache/doris/pull/15238 -- 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](union)the union node should not pass through children in some case (#15286)

2022-12-22 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 82fbfab77f [fix](union)the union node should no

[GitHub] [doris] yiguolei merged pull request #15286: [fix](union)the union node should not pass through children in some case

2022-12-22 Thread GitBox
yiguolei merged PR #15286: URL: https://github.com/apache/doris/pull/15286 -- 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 (6b3721af23 -> 020c47f528)

2022-12-22 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 6b3721af23 [Bug](function) fix core dump on reverse() when big string input add 020c47f528 [load](config) updat

[GitHub] [doris] yiguolei merged pull request #15280: [load](config) update max timeout

2022-12-22 Thread GitBox
yiguolei merged PR #15280: URL: https://github.com/apache/doris/pull/15280 -- 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 (09a22813e4 -> 6b3721af23)

2022-12-22 Thread panxiaolei
This is an automated email from the ASF dual-hosted git repository. panxiaolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 09a22813e4 [feature](Nereids) support syntax SELECT DISTINCT (#15197) add 6b3721af23 [Bug](function) fix core

[GitHub] [doris] BiteTheDDDDt merged pull request #15215: [Bug](function) fix core dump on reverse() when big string input

2022-12-22 Thread GitBox
BiteThet merged PR #15215: URL: https://github.com/apache/doris/pull/15215 -- 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

[GitHub] [doris] Gabriel39 commented on issue #15307: [Bug] Use correct decimal scale for function `round`

2022-12-22 Thread GitBox
Gabriel39 commented on issue #15307: URL: https://github.com/apache/doris/issues/15307#issuecomment-1363537023 > I willing to continue working on this issue Great! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[doris] branch branch-1.1-lts updated: FIX: ctas string as first column (#15233)

2022-12-22 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a commit to branch branch-1.1-lts in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/branch-1.1-lts by this push: new e25899eb2e FIX: ctas string as

[GitHub] [doris] yiguolei merged pull request #15233: [Cherry-pick](CTAS) Fix ctas first column is string error

2022-12-22 Thread GitBox
yiguolei merged PR #15233: URL: https://github.com/apache/doris/pull/15233 -- 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 #15251: [Fix](Nereids)fix repeat node nullable error bugs

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15251: URL: https://github.com/apache/doris/pull/15251#issuecomment-1363498881 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] gojumprope commented on issue #15166: [Bug] Kafka routine load datetime is not following time zone (UTC)

2022-12-22 Thread GitBox
gojumprope commented on issue #15166: URL: https://github.com/apache/doris/issues/15166#issuecomment-1363489133 Is this the same issue with https://github.com/apache/doris/issues/15284 ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [doris] TangSiyang2001 commented on issue #13982: [Feature] group_uniq_array function

2022-12-22 Thread GitBox
TangSiyang2001 commented on issue #13982: URL: https://github.com/apache/doris/issues/13982#issuecomment-1363108994 When doing this job,I found that this function is quite similar with `collect_set`, except for the 'max_size ' limit.The sibling function `group_array` seems also in this situ

[GitHub] [doris] hello-stephen commented on pull request #15274: [fix](multi-catalog) fix and optimize iceberg v2 reader

2022-12-22 Thread GitBox
hello-stephen commented on PR #15274: URL: https://github.com/apache/doris/pull/15274#issuecomment-1363070304 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.72 seconds load time: 631 seconds storage size: 17123707946 Bytes https://doris-co

[GitHub] [doris] morrySnow commented on pull request #15292: [Fix](Nerids) fix dup key for pull predicate from project children

2022-12-22 Thread GitBox
morrySnow commented on PR #15292: URL: https://github.com/apache/doris/pull/15292#issuecomment-1363021500 please add some commit msg to explain this bug and add UT or regression test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [doris] github-actions[bot] commented on pull request #15290: [fix](Nereids): convert empty inner join to cross join

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15290: URL: https://github.com/apache/doris/pull/15290#issuecomment-1363019135 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 #15290: [fix](Nereids): convert empty inner join to cross join

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15290: URL: https://github.com/apache/doris/pull/15290#issuecomment-1363019055 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] morrySnow commented on a diff in pull request #15275: [Feature](Nereids) Support hll and count for materialized index.

2022-12-22 Thread GitBox
morrySnow commented on code in PR #15275: URL: https://github.com/apache/doris/pull/15275#discussion_r1055615863 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Ndv.java: ## @@ -0,0 +1,117 @@ +// Licensed to the Apache Software Foundation (AS

[GitHub] [doris] Yulei-Yang opened a new pull request, #15308: [Improvement][meta] support rename hive and jdbc engine external table

2022-12-22 Thread GitBox
Yulei-Yang opened a new pull request, #15308: URL: https://github.com/apache/doris/pull/15308 Currently, if you rename a hive / jdbc engine table, you will be refused: mysql> use db1; alter table ctp2 rename tp2_orc; ERROR 1105 (HY000): errCode = 2, detailMessage = Do not support alter

[doris] branch master updated: [feature](Nereids) support syntax SELECT DISTINCT (#15197)

2022-12-22 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 09a22813e4 [feature](Nereids) support syntax S

[GitHub] [doris] morrySnow merged pull request #15197: [feature](nereids) support syntax SELECT DISTINCT

2022-12-22 Thread GitBox
morrySnow merged PR #15197: URL: https://github.com/apache/doris/pull/15197 -- 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 #15197: [feature](nereids) support syntax SELECT DISTINCT

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15197: URL: https://github.com/apache/doris/pull/15197#issuecomment-1362984579 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 #15197: [feature](nereids) support syntax SELECT DISTINCT

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #15197: URL: https://github.com/apache/doris/pull/15197#issuecomment-1362984537 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] hello-stephen commented on pull request #15299: [fix](iceberg-v2) fix fe iceberg split, add regression case

2022-12-22 Thread GitBox
hello-stephen commented on PR #15299: URL: https://github.com/apache/doris/pull/15299#issuecomment-1362976967 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.11 seconds load time: 631 seconds storage size: 17123219584 Bytes https://doris-co

[GitHub] [doris] yiguolei merged pull request #15278: [refactor](non-vec) Remove non vec code from be

2022-12-22 Thread GitBox
yiguolei merged PR #15278: URL: https://github.com/apache/doris/pull/15278 -- 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] hello-stephen commented on pull request #15298: [bugfix](from_unixtime) fix timezone not work for from_unixtime

2022-12-22 Thread GitBox
hello-stephen commented on PR #15298: URL: https://github.com/apache/doris/pull/15298#issuecomment-1362968457 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.9 seconds load time: 630 seconds storage size: 17123148199 Bytes https://doris-com

[GitHub] [doris] morrySnow commented on a diff in pull request #15252: [feature](Nereids): support digital_masking function

2022-12-22 Thread GitBox
morrySnow commented on code in PR #15252: URL: https://github.com/apache/doris/pull/15252#discussion_r1055571451 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/scalar/DigitalMasking.java: ## @@ -0,0 +1,70 @@ +// Licensed to the Apache Software F

  1   2   3   >