[GitHub] [incubator-doris] xiepengcheng01 commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
xiepengcheng01 commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867678373 ## be/src/vec/exec/vbroker_scan_node.h: ## @@ -31,7 +31,8 @@ namespace vectorized { class VBrokerScanNode final : public BrokerScanNode { public: V

[GitHub] [incubator-doris] BiteTheDDDDt commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
BiteThet commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867678019 ## be/src/vec/exec/vjson_scanner.cpp: ## @@ -0,0 +1,517 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license ag

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9402: [Feature] support utf8mb4 for mysql external table

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9402: URL: https://github.com/apache/incubator-doris/pull/9402#issuecomment-1120698314 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9402: [Feature] support utf8mb4 for mysql external table

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9402: URL: https://github.com/apache/incubator-doris/pull/9402#issuecomment-1120698338 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 us

[GitHub] [incubator-doris] morrySnow commented on pull request #9460: [style]format fe code with some check rules

2022-05-08 Thread GitBox
morrySnow commented on PR #9460: URL: https://github.com/apache/incubator-doris/pull/9460#issuecomment-1120698015 all fe check style error is not related to the rules this patch care about -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-doris] BiteTheDDDDt commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
BiteThet commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867671833 ## be/src/vec/exec/vjson_scanner.cpp: ## @@ -0,0 +1,517 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license ag

[GitHub] [incubator-doris] xlwh commented on pull request #9456: [Refactor] Replace '#ifndef' with '#pragma once'

2022-05-08 Thread GitBox
xlwh commented on PR #9456: URL: https://github.com/apache/incubator-doris/pull/9456#issuecomment-1120696408 > Is there any way to automatically detect header guard style? Sorry, I don't have a good way to automatically check. Do you have any good suggest? thanks! -- This is an au

[incubator-doris-flink-connector] branch master updated: fix broke url and ut (#32)

2022-05-08 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/incubator-doris-flink-connector.git The following commit(s) were added to refs/heads/master by this push: new a752f93 fix bro

[GitHub] [incubator-doris-flink-connector] hf200012 merged pull request #32: Fix broke URL and UT

2022-05-08 Thread GitBox
hf200012 merged PR #32: URL: https://github.com/apache/incubator-doris-flink-connector/pull/32 -- 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-u

[GitHub] [incubator-doris] BiteTheDDDDt commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
BiteThet commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867669760 ## be/src/exec/base_scanner.h: ## @@ -55,7 +57,7 @@ class BaseScanner { const std::vector& pre_filter_texprs, ScannerCounter* counter);

[GitHub] [incubator-doris] BiteTheDDDDt commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
BiteThet commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867669516 ## be/src/exec/json_scanner.cpp: ## @@ -329,6 +334,19 @@ Status JsonReader::init(const std::string& jsonpath, const std::string& json_roo return Statu

[GitHub] [incubator-doris] wangshuo128 commented on a diff in pull request #9459: [stream-load-vec]: memtable flush only if necessary after aggregated

2022-05-08 Thread GitBox
wangshuo128 commented on code in PR #9459: URL: https://github.com/apache/incubator-doris/pull/9459#discussion_r867666484 ## be/src/olap/memtable.h: ## @@ -195,9 +199,11 @@ class MemTable { //for vectorized vectorized::MutableBlock _input_mutable_block; vectorized

[GitHub] [incubator-doris] tianhui5 opened a new issue, #9461: [Bug] (load) Broker load kerberos auth fail

2022-05-08 Thread GitBox
tianhui5 opened a new issue, #9461: URL: https://github.com/apache/incubator-doris/issues/9461 ### 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 0.13

[GitHub] [incubator-doris] stalary commented on pull request #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
stalary commented on PR #9454: URL: https://github.com/apache/incubator-doris/pull/9454#issuecomment-1120673718 done. @wangshuo128 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

[GitHub] [incubator-doris] BiteTheDDDDt commented on pull request #9456: [Refactor] Replace '#ifndef' with '#pragma once'

2022-05-08 Thread GitBox
BiteThet commented on PR #9456: URL: https://github.com/apache/incubator-doris/pull/9456#issuecomment-1120661122 Is there any way to automatically detect header guard style? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-doris] morrySnow opened a new pull request, #9460: [style]format fe code with some check rules

2022-05-08 Thread GitBox
morrySnow opened a new pull request, #9460: URL: https://github.com/apache/incubator-doris/pull/9460 # Proposed changes Issue Number: close #9403 set below rules' severity to error and format code according check info. a. Merge conflicts unresolved b. Avoid using correspo

[GitHub] [incubator-doris] deardeng closed pull request #9358: fix #9351 can't load parquet file with column name case sensitive with Doris column

2022-05-08 Thread GitBox
deardeng closed pull request #9358: fix #9351 can't load parquet file with column name case sensitive with Doris column URL: https://github.com/apache/incubator-doris/pull/9358 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-doris] EmmyMiao87 commented on issue #9346: [Enhancement] 物化视图增强

2022-05-08 Thread GitBox
EmmyMiao87 commented on issue #9346: URL: https://github.com/apache/incubator-doris/issues/9346#issuecomment-1120637301 > 也不支持计算列 之所以支持的计算列不够丰富是考虑到:后续会更新物化视图计计算引擎,所以没必要在旧的引擎上去丰富计算列的功能。 在 @BiteThet 将物化视图切换到向量化引擎后,就可以支持丰富的计算列了。 -- This is an automated message from the

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9427: fix bug for OLAP_SUCCESS with Status

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9427: URL: https://github.com/apache/incubator-doris/pull/9427#issuecomment-1120633975 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

[GitHub] [incubator-doris] spaces-X opened a new pull request, #9459: [stream-load-vec]: memtable flush only if necessary after aggregated

2022-05-08 Thread GitBox
spaces-X opened a new pull request, #9459: URL: https://github.com/apache/incubator-doris/pull/9459 # Proposed changes When the input data in the current memtable is full, flush will be performed, and there is no logic to judge whether it is full after aggregation. I try to su

[GitHub] [incubator-doris] BiteTheDDDDt commented on issue #9346: [Enhancement] 物化视图增强

2022-05-08 Thread GitBox
BiteThet commented on issue #9346: URL: https://github.com/apache/incubator-doris/issues/9346#issuecomment-1120611965 在之后的版本里,向量化引擎的物化视图可能会提供更多计算功能 -- 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] [incubator-doris] zhengshiJ closed pull request #9216: Remaining nodes implement

2022-05-08 Thread GitBox
zhengshiJ closed pull request #9216: Remaining nodes implement URL: https://github.com/apache/incubator-doris/pull/9216 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsub

[GitHub] [incubator-doris] zhengshiJ opened a new pull request, #9458: [feature](statistics) Statistics derivation.Step 2:OtherNode implemen…

2022-05-08 Thread GitBox
zhengshiJ opened a new pull request, #9458: URL: https://github.com/apache/incubator-doris/pull/9458 …t statistics derivation # Proposed changes Issue Number: close #xxx ## Problem Summary: Second step of statistics derivation: implementation of nodes other than s

[GitHub] [incubator-doris] deardeng commented on a diff in pull request #9358: fix #9351 can't load parquet file with column name case sensitive with Doris column

2022-05-08 Thread GitBox
deardeng commented on code in PR #9358: URL: https://github.com/apache/incubator-doris/pull/9358#discussion_r867606594 ## fe/fe-core/src/main/java/org/apache/doris/load/Load.java: ## @@ -1045,12 +1045,23 @@ private static void initColumns(Table tbl, List columnExprs,

[GitHub] [incubator-doris] deardeng commented on a diff in pull request #9358: fix #9351 can't load parquet file with column name case sensitive with Doris column

2022-05-08 Thread GitBox
deardeng commented on code in PR #9358: URL: https://github.com/apache/incubator-doris/pull/9358#discussion_r867606706 ## fe/fe-core/src/main/java/org/apache/doris/load/Load.java: ## @@ -1045,12 +1045,23 @@ private static void initColumns(Table tbl, List columnExprs,

[incubator-doris] branch master updated: [feature] (vec) instead of converting line to src tuple for stream load in vectorized. (#9314)

2022-05-08 Thread lihaopeng
This is an automated email from the ASF dual-hosted git repository. lihaopeng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new eec1dfde3a [feature] (vec) instead

[GitHub] [incubator-doris] HappenLee merged pull request #9314: [feature] (vec) instead of converting line to src tuple for stream load in vectorized.

2022-05-08 Thread GitBox
HappenLee merged PR #9314: URL: https://github.com/apache/incubator-doris/pull/9314 -- 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...@

[GitHub] [incubator-doris] HappenLee commented on a diff in pull request #9432: [Feature]Use multi hash tables to aggregate

2022-05-08 Thread GitBox
HappenLee commented on code in PR #9432: URL: https://github.com/apache/incubator-doris/pull/9432#discussion_r867598416 ## be/src/vec/common/hash_table/hash_table_proxy_for_multi_tables.h: ## @@ -0,0 +1,189 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or

[GitHub] [incubator-doris] chenlinzhong closed issue #9237: [Bug] DeltaWriter::mem_consumption() coredump

2022-05-08 Thread GitBox
chenlinzhong closed issue #9237: [Bug] DeltaWriter::mem_consumption() coredump URL: https://github.com/apache/incubator-doris/issues/9237 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[GitHub] [incubator-doris] wangshuo128 commented on a diff in pull request #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
wangshuo128 commented on code in PR #9454: URL: https://github.com/apache/incubator-doris/pull/9454#discussion_r867599417 ## docs/zh-CN/sql-manual/sql-reference/Data-Definition-Statements/Create/CREATE-TABLE-AS-SELECT.md: ## @@ -0,0 +1,74 @@ +--- +{ +"title": "CREATE-TABLE-A

[GitHub] [incubator-doris] wangshuo128 commented on a diff in pull request #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
wangshuo128 commented on code in PR #9454: URL: https://github.com/apache/incubator-doris/pull/9454#discussion_r867599271 ## docs/en/sql-manual/sql-reference/Data-Definition-Statements/Create/CREATE-TABLE-AS-SELECT.md: ## @@ -0,0 +1,74 @@ +--- +{ +"title": "CREATE-TABLE-AS-S

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9391: [Feature] show create materialized view

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9391: URL: https://github.com/apache/incubator-doris/pull/9391#issuecomment-1120586055 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

[GitHub] [incubator-doris] stalary commented on pull request #9391: [Feature] show create materialized view

2022-05-08 Thread GitBox
stalary commented on PR #9391: URL: https://github.com/apache/incubator-doris/pull/9391#issuecomment-1120584207 > LGTM @stalary please resolve confilicts. done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [incubator-doris] qidaye commented on pull request #9391: [Feature] show create materialized view

2022-05-08 Thread GitBox
qidaye commented on PR #9391: URL: https://github.com/apache/incubator-doris/pull/9391#issuecomment-1120576605 LGTM @stalary please resolve confilicts. -- 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 t

[GitHub] [incubator-doris] xy720 commented on issue #9302: [Feature] Support protobuf(canal) format for routine load

2022-05-08 Thread GitBox
xy720 commented on issue #9302: URL: https://github.com/apache/incubator-doris/issues/9302#issuecomment-1120568471 I am still working on it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the s

[GitHub] [incubator-doris] liuzhuang2017 commented on pull request #9455: [docs][typo] Fix some typoes in "update.md" content.

2022-05-08 Thread GitBox
liuzhuang2017 commented on PR #9455: URL: https://github.com/apache/incubator-doris/pull/9455#issuecomment-1120565606 @hf200012 ,Sorry to bother you,can you help me review this pr when you have a free time. Thank you very much. -- This is an automated message from the Apache Git Service.

[incubator-doris] branch master updated: [code style](fe) Include test sources (#9366)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new d1b85d51a0 [code style](fe) Include

[GitHub] [incubator-doris] morningman merged pull request #9366: [FE Code Style] Include test sources

2022-05-08 Thread GitBox
morningman merged PR #9366: URL: https://github.com/apache/incubator-doris/pull/9366 -- 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...

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9366: [FE Code Style] Include test sources

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9366: URL: https://github.com/apache/incubator-doris/pull/9366#issuecomment-1120544873 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

[GitHub] [incubator-doris] liaoxin01 commented on issue #9208: [Enhancement] (decimalv3) optimize decimal performance and precision

2022-05-08 Thread GitBox
liaoxin01 commented on issue #9208: URL: https://github.com/apache/incubator-doris/issues/9208#issuecomment-1120543707 **Performance test:** The storage layer data is still decimalv2, which is calculated using decimalv3. Create a table and import 100 million pieces of data. set

[incubator-doris] branch master updated: [fix](ut) fix DeltaWriter::close_wait parameter mismatch in delta_writer_test (#9457)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new ae01862ae4 [fix](ut) fix DeltaWrite

[GitHub] [incubator-doris] morningman merged pull request #9457: [fix](ut) fix DeltaWriter::close_wait parameter mismatch in delta_writer_test

2022-05-08 Thread GitBox
morningman merged PR #9457: URL: https://github.com/apache/incubator-doris/pull/9457 -- 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...

[incubator-doris] branch master updated: [fix] UT MathFunctionTest.round_test fix (#9447)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 7e86c1beab [fix] UT MathFunctionTes

[GitHub] [incubator-doris] morningman merged pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
morningman merged PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447 -- 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...

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120543192 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 us

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120543174 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

[GitHub] [incubator-doris] yangzhg closed issue #9420: [Bug] doris write s3 temp file my failed

2022-05-08 Thread GitBox
yangzhg closed issue #9420: [Bug] doris write s3 temp file my failed URL: https://github.com/apache/incubator-doris/issues/9420 -- 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.

[incubator-doris] branch master updated: [fix](s3) fix s3 Temp file may write failed because of has no space on disk (#9421)

2022-05-08 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 6834fb23ca [fix](s3) fix s3 Temp file

[GitHub] [incubator-doris] yangzhg merged pull request #9421: [fix](s3)(int-overflow) manage s3 temp file path and fix row batch offset int overflow bug

2022-05-08 Thread GitBox
yangzhg merged PR #9421: URL: https://github.com/apache/incubator-doris/pull/9421 -- 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...@do

[GitHub] [incubator-doris] smallhibiscus closed pull request #9434: [Refactor][doc] update spark-doris-connector doc

2022-05-08 Thread GitBox
smallhibiscus closed pull request #9434: [Refactor][doc] update spark-doris-connector doc URL: https://github.com/apache/incubator-doris/pull/9434 -- 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

[GitHub] [incubator-doris] emerkfu commented on issue #9302: [Feature] Support protobuf(canal) format for routine load

2022-05-08 Thread GitBox
emerkfu commented on issue #9302: URL: https://github.com/apache/incubator-doris/issues/9302#issuecomment-1120535670 I think flink-doris-connector also needs this feature @JNSimba @hf200012 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-doris] hf200012 merged pull request #9448: [doc] update data-model.md and data-partition.md

2022-05-08 Thread GitBox
hf200012 merged PR #9448: URL: https://github.com/apache/incubator-doris/pull/9448 -- 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...@d

[incubator-doris] branch master updated: [doc] update data-model.md and data-partition.md (#9448)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 5df5d39161 [doc] update data-mode

[incubator-doris] branch master updated: [doc] Update DECIMAL.md (#9451)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 35f0725387 [doc] Update DECIMAL.m

[GitHub] [incubator-doris] hf200012 merged pull request #9451: [doc] Update DECIMAL.md

2022-05-08 Thread GitBox
hf200012 merged PR #9451: URL: https://github.com/apache/incubator-doris/pull/9451 -- 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...@d

[incubator-doris] branch master updated: Update data-partition.md (#9450)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 327f61b796 Update data-partition.

[GitHub] [incubator-doris] hf200012 merged pull request #9450: Update data-partition.md

2022-05-08 Thread GitBox
hf200012 merged PR #9450: URL: https://github.com/apache/incubator-doris/pull/9450 -- 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...@d

[GitHub] [incubator-doris] hf200012 merged pull request #9442: [Doc] fix doc link suffix .html to .md

2022-05-08 Thread GitBox
hf200012 merged PR #9442: URL: https://github.com/apache/incubator-doris/pull/9442 -- 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...@d

[GitHub] [incubator-doris] hf200012 closed issue #9411: [Bug] CTAS create table by list partition error

2022-05-08 Thread GitBox
hf200012 closed issue #9411: [Bug] CTAS create table by list partition error URL: https://github.com/apache/incubator-doris/issues/9411 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

[GitHub] [incubator-doris] stalary commented on issue #9444: [Bug] show proc "/cluster_balance/history_tablets" report the error of "Malformed packet"

2022-05-08 Thread GitBox
stalary commented on issue #9444: URL: https://github.com/apache/incubator-doris/issues/9444#issuecomment-1120506607 I didn't reproduce it in master, do you have any steps to reproduce it -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-doris] BiteTheDDDDt commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
BiteThet commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120453801 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

[GitHub] [incubator-doris] cambyzju commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
cambyzju commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120452756 > Add a regression test for the case. thank you ~~ done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-doris] cambyzju commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
cambyzju commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120445031 > function_math_test.cpp yes. /home/disk3/zxy/baidu/bdg/doris/core/be/test/vec/function/function_math_test.cpp:382 failed because of failed to find function round

[GitHub] [incubator-doris] xinyiZzz opened a new pull request, #9457: [fix](ut) fix DeltaWriter::close_wait parameter mismatch in delta_writer_test

2022-05-08 Thread GitBox
xinyiZzz opened a new pull request, #9457: URL: https://github.com/apache/incubator-doris/pull/9457 # Proposed changes Issue Number: close #xxx ## Problem Summary: ``` /home/disk3/zxy/baidu/bdg/doris/core/be/test/olap/delta_writer_test.cpp:370:50: error: too few argu

[GitHub] [incubator-doris] xinyiZzz commented on pull request #9447: [fix] UT MathFunctionTest.round_test fix

2022-05-08 Thread GitBox
xinyiZzz commented on PR #9447: URL: https://github.com/apache/incubator-doris/pull/9447#issuecomment-1120438089 ``` ==47967==ERROR: AddressSanitizer: SEGV on unknown address 0x (pc 0x55d6edb1fc2f bp 0x7ffeb746cd50 sp 0x7ffeb74698d0 T0) ==47967==The signal is caused by a RE

[GitHub] [incubator-doris] yinzhijian commented on a diff in pull request #9433: [feature-wip](parquet-vec) Support parquet scanner in vectorized engine

2022-05-08 Thread GitBox
yinzhijian commented on code in PR #9433: URL: https://github.com/apache/incubator-doris/pull/9433#discussion_r867502807 ## fe/fe-core/src/main/java/org/apache/doris/load/Load.java: ## @@ -1044,26 +1047,52 @@ private static void initColumns(Table tbl, List columnExprs,

[GitHub] [incubator-doris] yinzhijian commented on a diff in pull request #9433: [feature-wip](parquet-vec) Support parquet scanner in vectorized engine

2022-05-08 Thread GitBox
yinzhijian commented on code in PR #9433: URL: https://github.com/apache/incubator-doris/pull/9433#discussion_r867502807 ## fe/fe-core/src/main/java/org/apache/doris/load/Load.java: ## @@ -1044,26 +1047,52 @@ private static void initColumns(Table tbl, List columnExprs,

[GitHub] [incubator-doris] xlwh opened a new pull request, #9456: [Refactor] Replace '#ifndef' with '#pragma once'

2022-05-08 Thread GitBox
xlwh opened a new pull request, #9456: URL: https://github.com/apache/incubator-doris/pull/9456 # Proposed changes Issue Number: close #xxx ## Problem Summary: Describe the overview of changes. ## Checklist(Required) 1. Does it affect the original behavior:

[GitHub] [incubator-doris] liuzhuang2017 opened a new pull request, #9455: [docs][typo] Fix some typoes in "update.md" content.

2022-05-08 Thread GitBox
liuzhuang2017 opened a new pull request, #9455: URL: https://github.com/apache/incubator-doris/pull/9455 # Proposed changes Fix some typoes in "update.md" content. ## Problem Summary: Fix some typoes in "update.md" content. ## Checklist(Required) 1. Does it

[GitHub] [incubator-doris] ShawshankLin commented on issue #9346: [Enhancement] 物化视图增强

2022-05-08 Thread GitBox
ShawshankLin commented on issue #9346: URL: https://github.com/apache/incubator-doris/issues/9346#issuecomment-1120422486 > I'll enhance it later we also need materialized views of computable columns -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-doris] ShawshankLin commented on issue #9346: [Enhancement] 物化视图增强

2022-05-08 Thread GitBox
ShawshankLin commented on issue #9346: URL: https://github.com/apache/incubator-doris/issues/9346#issuecomment-1120422429 > I'll enhance it later we also need materialized views of computable columns -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9450: Update data-partition.md

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9450: URL: https://github.com/apache/incubator-doris/pull/9450#issuecomment-1120419935 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9451: [doc] Update DECIMAL.md

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9451: URL: https://github.com/apache/incubator-doris/pull/9451#issuecomment-1120419852 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

[incubator-doris] branch master updated: [fix](schema_hash) Fix bug that introduced by removing schema_hash (#9449)

2022-05-08 Thread caiconghui
This is an automated email from the ASF dual-hosted git repository. caiconghui pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 580ce38a3f [fix](schema_hash) Fix b

[GitHub] [incubator-doris] caiconghui merged pull request #9449: [Fix](schema_hash) Fix bug that introduced by removing schema_hash

2022-05-08 Thread GitBox
caiconghui merged PR #9449: URL: https://github.com/apache/incubator-doris/pull/9449 -- 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...

[GitHub] [incubator-doris] caiconghui closed issue #9441: [Bug] All Export job exec error now

2022-05-08 Thread GitBox
caiconghui closed issue #9441: [Bug] All Export job exec error now URL: https://github.com/apache/incubator-doris/issues/9441 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. T

[GitHub] [incubator-doris] xinyiZzz commented on pull request #9350: remove explicit memtracker from scannode

2022-05-08 Thread GitBox
xinyiZzz commented on PR #9350: URL: https://github.com/apache/incubator-doris/pull/9350#issuecomment-1120410466 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 uns

[GitHub] [incubator-doris] dataroaring commented on pull request #9350: remove explicit memtracker from scannode

2022-05-08 Thread GitBox
dataroaring commented on PR #9350: URL: https://github.com/apache/incubator-doris/pull/9350#issuecomment-1120405697 > > > > > > _block_mem_tracker > > > > > > > > > > > > > > > So why remove `counterpart`, `counterpart` is just for `_block_mem_tracker`. > > > > > > > >

[GitHub] [incubator-doris] Gabriel39 commented on a diff in pull request #9440: [Vectorized][java-udf] add datetime&&largeint&&decimal type to java-udf

2022-05-08 Thread GitBox
Gabriel39 commented on code in PR #9440: URL: https://github.com/apache/incubator-doris/pull/9440#discussion_r867473767 ## fe/java-udf/src/main/java/org/apache/doris/udf/UdfExecutor.java: ## @@ -365,41 +445,81 @@ private void allocateInputObjects(long row) throws UdfRuntimeExce

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9454: URL: https://github.com/apache/incubator-doris/pull/9454#issuecomment-1120394133 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9454: URL: https://github.com/apache/incubator-doris/pull/9454#issuecomment-1120394135 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 us

[GitHub] [incubator-doris] stalary opened a new pull request, #9454: [Doc] 添加CTAS文档

2022-05-08 Thread GitBox
stalary opened a new pull request, #9454: URL: https://github.com/apache/incubator-doris/pull/9454 # Proposed changes Issue Number: close #xxx ## Problem Summary: Describe the overview of changes. ## Checklist(Required) 1. Does it affect the original behavio

[GitHub] [incubator-doris] stalary commented on pull request #9391: [Feature] show create materialized view

2022-05-08 Thread GitBox
stalary commented on PR #9391: URL: https://github.com/apache/incubator-doris/pull/9391#issuecomment-1120388058 done. @morrySnow -- 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 comm

[GitHub] [incubator-doris] carlvinhust2012 commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
carlvinhust2012 commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867463402 ## be/src/vec/exec/vjson_scanner.cpp: ## @@ -0,0 +1,545 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

[GitHub] [incubator-doris] carlvinhust2012 commented on a diff in pull request #9311: [feature]add vectorized vjson_scanner

2022-05-08 Thread GitBox
carlvinhust2012 commented on code in PR #9311: URL: https://github.com/apache/incubator-doris/pull/9311#discussion_r867463193 ## be/src/vec/exec/vjson_scanner.cpp: ## @@ -0,0 +1,545 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

[incubator-doris] branch master updated: [feature] (sql-digest) support sql digest (#8919)

2022-05-08 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/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new c633402ce3 [feature] (sql-digest) s

[GitHub] [incubator-doris] morningman merged pull request #8919: [Feature] (sql digest) support sql digest

2022-05-08 Thread GitBox
morningman merged PR #8919: URL: https://github.com/apache/incubator-doris/pull/8919 -- 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...

[GitHub] [incubator-doris] morningman closed issue #8918: [Feature] sql digest is badly needed in many scenarios

2022-05-08 Thread GitBox
morningman closed issue #8918: [Feature] sql digest is badly needed in many scenarios URL: https://github.com/apache/incubator-doris/issues/8918 -- 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 s

[GitHub] [incubator-doris] EmmyMiao87 merged pull request #9446: [Enhancement](Optimizer) Optimize nereids tree node structure

2022-05-08 Thread GitBox
EmmyMiao87 merged PR #9446: URL: https://github.com/apache/incubator-doris/pull/9446 -- 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...

[incubator-doris] branch master updated: [Enhancement](Optimizer) Optimize nereids tree node structure (#9446)

2022-05-08 Thread lingmiao
This is an automated email from the ASF dual-hosted git repository. lingmiao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git The following commit(s) were added to refs/heads/master by this push: new 52a2db18c0 [Enhancement](Optimizer) O

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9446: [Enhancement](Optimizer) Optimize nereids tree node structure

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9446: URL: https://github.com/apache/incubator-doris/pull/9446#issuecomment-1120377657 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9446: [Enhancement](Optimizer) Optimize nereids tree node structure

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9446: URL: https://github.com/apache/incubator-doris/pull/9446#issuecomment-1120377656 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

[GitHub] [incubator-doris] EmmyMiao87 commented on a diff in pull request #9446: [Enhancement](Optimizer) Optimize nereids tree node structure

2022-05-08 Thread GitBox
EmmyMiao87 commented on code in PR #9446: URL: https://github.com/apache/incubator-doris/pull/9446#discussion_r867338822 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/AbstractTreeNode.java: ## @@ -0,0 +1,86 @@ +// Licensed to the Apache Software Foundation (ASF) und

[GitHub] [incubator-doris] morrySnow commented on a diff in pull request #9391: [Feature] show create materialized view

2022-05-08 Thread GitBox
morrySnow commented on code in PR #9391: URL: https://github.com/apache/incubator-doris/pull/9391#discussion_r867451804 ## fe/fe-core/src/main/java/org/apache/doris/qe/ShowExecutor.java: ## @@ -2178,4 +2182,25 @@ private void handleAdminDiagnoseTablet() { resultSet = ne

[GitHub] [incubator-doris] morrySnow commented on pull request #8919: [Feature] (sql digest) support sql digest

2022-05-08 Thread GitBox
morrySnow commented on PR #8919: URL: https://github.com/apache/incubator-doris/pull/8919#issuecomment-1120369759 please fix fe code style error, thanks -- 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] [incubator-doris] github-actions[bot] commented on pull request #9446: [Enhancement](Optimizer) Optimize nereids tree node structure

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9446: URL: https://github.com/apache/incubator-doris/pull/9446#issuecomment-1120367917 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 us

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9450: Update data-partition.md

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9450: URL: https://github.com/apache/incubator-doris/pull/9450#issuecomment-1120367747 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 us

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9451: [doc] Update DECIMAL.md

2022-05-08 Thread GitBox
github-actions[bot] commented on PR #9451: URL: https://github.com/apache/incubator-doris/pull/9451#issuecomment-1120367642 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 us