[GitHub] [doris] yiguolei opened a new pull request, #12180: Revert "[enhancement](memtracker) Improve performance of tracking real physical memory of PODArray"

2022-08-29 Thread GitBox
yiguolei opened a new pull request, #12180: URL: https://github.com/apache/doris/pull/12180 Reverts apache/doris#12168 This PR will cause compile failed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[doris] branch revert-12168-20220829_fix_podarray_resetpeak created (now e7231d49e5)

2022-08-29 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch revert-12168-20220829_fix_podarray_resetpeak in repository https://gitbox.apache.org/repos/asf/doris.git at e7231d49e5 Revert "[enhancement](memtracker) Improve performance of tracking r

[doris] 01/01: Revert "[enhancement](memtracker) Improve performance of tracking real physical memory of PODArray #12168"

2022-08-29 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a commit to branch revert-12168-20220829_fix_podarray_resetpeak in repository https://gitbox.apache.org/repos/asf/doris.git commit e7231d49e5a335a5ba5b838e6ea3349cf83db103 Author: yiguolei <676222...@qq.com> Autho

[GitHub] [doris] github-actions[bot] commented on pull request #12179: [bugfix](simdjson) disable simdjson by default, wait for fix the bug

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12179: URL: https://github.com/apache/doris/pull/12179#issuecomment-1231227300 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 #12179: [bugfix](simdjson) disable simdjson by default, wait for fix the bug

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12179: URL: https://github.com/apache/doris/pull/12179#issuecomment-1231227320 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] morningman closed pull request #12177: [fix](compile) fix compile bug after merging #12042 and #12161

2022-08-29 Thread GitBox
morningman closed pull request #12177: [fix](compile) fix compile bug after merging #12042 and #12161 URL: https://github.com/apache/doris/pull/12177 -- 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

[GitHub] [doris] morningman commented on pull request #12177: [fix](compile) fix compile bug after merging #12042 and #12161

2022-08-29 Thread GitBox
morningman commented on PR #12177: URL: https://github.com/apache/doris/pull/12177#issuecomment-1231225331 Fixed in #12179 , close this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[GitHub] [doris] yangzhg opened a new pull request, #12179: [bugfix](simdjson) disable simdjson by default, wait for fix the bug

2022-08-29 Thread GitBox
yangzhg opened a new pull request, #12179: URL: https://github.com/apache/doris/pull/12179 # Proposed changes disable simdjson by default, wait for fix the bug ## Problem summary disable simdjson by default, wait for fix the bug ## Checklist(Required) 1. D

[GitHub] [doris] Userwhite opened a new pull request, #12178: [Improvement](storage) reuse schema and rowblockv2 on single scanner_…

2022-08-29 Thread GitBox
Userwhite opened a new pull request, #12178: URL: https://github.com/apache/doris/pull/12178 cherry-pick (#11392) * support reuse rowblockv2 on single thread # Proposed changes Issue Number: close #11392 ## Problem summary Describe your changes. ## Ch

[GitHub] [doris] github-actions[bot] commented on pull request #12177: [fix](compile) fix compile bug after merging #12042 and #12161

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12177: URL: https://github.com/apache/doris/pull/12177#issuecomment-1231175072 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 #12177: [fix](compile) fix compile bug after merging #12042 and #12161

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12177: URL: https://github.com/apache/doris/pull/12177#issuecomment-1231175049 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] YangShaw commented on pull request #12046: [feature](nereids)support one expression-rewrite rule: inPredicateToEqualTo

2022-08-29 Thread GitBox
YangShaw commented on PR #12046: URL: https://github.com/apache/doris/pull/12046#issuecomment-1231174833 Still have little bug about recursive, like: ``` a in (1, 2, (a in (1)))``` This kind of cases will be resolved in next pr. -- This is an automated message from the Apache Git Ser

[GitHub] [doris] YIFENG12 commented on issue #11706: Good First Issue

2022-08-29 Thread GitBox
YIFENG12 commented on issue #11706: URL: https://github.com/apache/doris/issues/11706#issuecomment-1231173375 [WeOpen Star] I would like to help DOCS & BLOGS TRANSLATION Doris 索引机制解析 https://mp.weixin.qq.com/s/KdCdXb9Z3MdUZ5S0RV726Q -- This is an automated message from the Apache Git Serv

[GitHub] [doris] zxealous commented on a diff in pull request #12165: [optimize](remote) Optimize cache reader use a pre-created buffer when downloading the cache

2022-08-29 Thread GitBox
zxealous commented on code in PR #12165: URL: https://github.com/apache/doris/pull/12165#discussion_r958029282 ## be/src/io/cache/sub_file_cache.cpp: ## @@ -153,27 +153,41 @@ Status SubFileCache::_generate_cache_reader(size_t offset, size_t req_size) { LOG(

[GitHub] [doris] morningman opened a new pull request, #12177: [fix](compile) fix compile bug after merging #12042 and #12161

2022-08-29 Thread GitBox
morningman opened a new pull request, #12177: URL: https://github.com/apache/doris/pull/12177 # Proposed changes Issue Number: close #xxx ## Problem summary Fix merge conflict ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Yes

[GitHub] [doris] morningman opened a new pull request, #12176: [feature] add ADMIN COPY TABLET stmt for local debug

2022-08-29 Thread GitBox
morningman opened a new pull request, #12176: URL: https://github.com/apache/doris/pull/12176 # Proposed changes Issue Number: close #xxx ## Problem summary Add a new stmt `ADMIN COPY TABLET` for easy copy a tablet to local env to reproduce problem. See document for m

[GitHub] [doris] HappenLee commented on a diff in pull request #12146: [Opt](load) Speed up the vectorized load

2022-08-29 Thread GitBox
HappenLee commented on code in PR #12146: URL: https://github.com/apache/doris/pull/12146#discussion_r958004376 ## be/src/vec/functions/function_helpers.cpp: ## @@ -26,6 +26,42 @@ namespace doris::vectorized { +Block create_block_with_nested_columns_only_args(const Block& b

[GitHub] [doris] github-actions[bot] commented on pull request #12175: [fix](array-type) array_sort function with empty input

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12175: URL: https://github.com/apache/doris/pull/12175#issuecomment-1231138176 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 #12175: [fix](array-type) array_sort function with empty input

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12175: URL: https://github.com/apache/doris/pull/12175#issuecomment-1231138152 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] hf200012 merged pull request #12109: [docs](hive)add hadoop.username config item

2022-08-29 Thread GitBox
hf200012 merged PR #12109: URL: https://github.com/apache/doris/pull/12109 -- 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: [docs](hive)add hadoop.username config item (#12109)

2022-08-29 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.git The following commit(s) were added to refs/heads/master by this push: new eea756fd5b [docs](hive)add hadoop.username

[doris] branch master updated (dcad318fcc -> 7f295fc2fb)

2022-08-29 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from dcad318fcc [Doc](flink-connector) update sample code url (#12149) add 7f295fc2fb [typo](doc)Modify data oper

[GitHub] [doris] hf200012 merged pull request #12137: [typo](doc)Modify data operation documentation

2022-08-29 Thread GitBox
hf200012 merged PR #12137: URL: https://github.com/apache/doris/pull/12137 -- 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: [Doc](flink-connector) update sample code url (#12149)

2022-08-29 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.git The following commit(s) were added to refs/heads/master by this push: new dcad318fcc [Doc](flink-connector) update sa

[GitHub] [doris] hf200012 merged pull request #12149: [Doc](flink-connector) update sample code url

2022-08-29 Thread GitBox
hf200012 merged PR #12149: URL: https://github.com/apache/doris/pull/12149 -- 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] morrySnow commented on a diff in pull request #12159: [feature](nereids) support distinct count for TPC-H

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12159: URL: https://github.com/apache/doris/pull/12159#discussion_r957976831 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/AggregateDisassemble.java: ## @@ -54,86 +56,192 @@ */ public class AggregateDisassemble extends O

[GitHub] [doris] Toms1999 commented on issue #12164: [Enhancement] Optimize cache reader use pre-created buffer

2022-08-29 Thread GitBox
Toms1999 commented on issue #12164: URL: https://github.com/apache/doris/issues/12164#issuecomment-1231124355 let me seesee -- 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] cambyzju opened a new pull request, #12175: [fix](array-type) array_sort function with empty input

2022-08-29 Thread GitBox
cambyzju opened a new pull request, #12175: URL: https://github.com/apache/doris/pull/12175 # Proposed changes Issue Number: close #xxx ## Problem summary array_sort with empty array as input, will got error. because compare use UInt64, `(uint64_t)0 < (uint64_t)0 - 1`

[GitHub] [doris] Gabriel39 opened a new pull request, #12174: [Improvement](load) accelerate tablet sink

2022-08-29 Thread GitBox
Gabriel39 opened a new pull request, #12174: URL: https://github.com/apache/doris/pull/12174 # Proposed changes Issue Number: close #xxx ## Problem summary 201s->184s ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Yes -

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957980690 ## fe/fe-core/src/main/java/org/apache/doris/nereids/properties/UnboundLogicalProperties.java: ## @@ -37,4 +37,14 @@ public UnboundLogicalProperties() { public Li

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957980690 ## fe/fe-core/src/main/java/org/apache/doris/nereids/properties/UnboundLogicalProperties.java: ## @@ -37,4 +37,14 @@ public UnboundLogicalProperties() { public Li

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957979130 ## fe/fe-core/src/main/java/org/apache/doris/nereids/properties/LogicalProperties.java: ## @@ -50,4 +50,21 @@ public List getOutput() { public LogicalProperties w

[GitHub] [doris] wangshuo128 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
wangshuo128 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957977717 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -66,37 +67,18 @@ public Map getGroupExpressions() { * Add plan to Memo. * TOD

[GitHub] [doris] Gabriel39 commented on a diff in pull request #12146: [Opt](load) Speed up the vectorized load

2022-08-29 Thread GitBox
Gabriel39 commented on code in PR #12146: URL: https://github.com/apache/doris/pull/12146#discussion_r957955723 ## be/src/vec/olap/vcollect_iterator.h: ## @@ -89,10 +89,10 @@ class VCollectIterator { const IteratorRowRef* current_row_ref() const { return &_ref; } -

[GitHub] [doris] pengxiangyu commented on a diff in pull request #12165: [optimize](remote) Optimize cache reader use a pre-created buffer when downloading the cache

2022-08-29 Thread GitBox
pengxiangyu commented on code in PR #12165: URL: https://github.com/apache/doris/pull/12165#discussion_r957964185 ## be/src/io/cache/sub_file_cache.cpp: ## @@ -153,27 +153,41 @@ Status SubFileCache::_generate_cache_reader(size_t offset, size_t req_size) { L

[GitHub] [doris] github-actions[bot] commented on pull request #12169: [fix](Nereids)Sum return DoubleType when child is DecimalType by mistake

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

[GitHub] [doris] jackwener commented on issue #11768: [Enhancement] Use `NIO` to replace the legacy IO.

2022-08-29 Thread GitBox
jackwener commented on issue #11768: URL: https://github.com/apache/doris/issues/11768#issuecomment-1231082910 Yes, if have any question, feel free @me DingGeGe ***@***.***> 于 2022年8月30日周二 10:55写道: > @jackwener Hi jack, can i help this issue? >

[GitHub] [doris] dinggege1024 commented on issue #11706: Good First Issue

2022-08-29 Thread GitBox
dinggege1024 commented on issue #11706: URL: https://github.com/apache/doris/issues/11706#issuecomment-1231082555 [WeOpen Star] I would like to help [Enhancement] Use NIO to replace the legacy IO. #11768 https://github.com/apache/doris/issues/11768 -- This is an automated message from

[GitHub] [doris] github-actions[bot] commented on pull request #12137: [typo](doc)Modify data operation documentation

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12137: URL: https://github.com/apache/doris/pull/12137#issuecomment-1231081969 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 master updated (8370115cf6 -> a16cf0e2c8)

2022-08-29 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 8370115cf6 [enhancement](memtracker) Improve performance of tracking real physical memory of PODArray #12168 ad

[GitHub] [doris] yiguolei merged pull request #12042: [feature-wip](scan) add profile for new olap scan node

2022-08-29 Thread GitBox
yiguolei merged PR #12042: URL: https://github.com/apache/doris/pull/12042 -- 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] dinggege1024 commented on issue #11768: [Enhancement] Use `NIO` to replace the legacy IO.

2022-08-29 Thread GitBox
dinggege1024 commented on issue #11768: URL: https://github.com/apache/doris/issues/11768#issuecomment-1231081435 @jackwener Hi jack, can i help this issue? -- 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] github-actions[bot] commented on pull request #12109: [docs](hive)add hadoop.username config item

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12109: URL: https://github.com/apache/doris/pull/12109#issuecomment-1231079668 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 #12169: [fix](Nereids)Sum return DoubleType when child is DecimalType by mistake

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12169: URL: https://github.com/apache/doris/pull/12169#discussion_r957953663 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/Sum.java: ## @@ -49,6 +50,9 @@ public DataType getDataType() { DataType dataT

[GitHub] [doris] englefly commented on a diff in pull request #12169: [fix](Nereids)Sum return DoubleType when child is DecimalType by mistake

2022-08-29 Thread GitBox
englefly commented on code in PR #12169: URL: https://github.com/apache/doris/pull/12169#discussion_r957949646 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/Sum.java: ## @@ -49,6 +50,9 @@ public DataType getDataType() { DataType dataTy

[GitHub] [doris] dataroaring commented on pull request #10971: [Bug](compaction) fix uniq key compaction bug

2022-08-29 Thread GitBox
dataroaring commented on PR #10971: URL: https://github.com/apache/doris/pull/10971#issuecomment-1231068837 https://github.com/apache/doris/pull/10946 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [doris] coderjiang commented on issue #11247: BE.alive is false when i begain sql

2022-08-29 Thread GitBox
coderjiang commented on issue #11247: URL: https://github.com/apache/doris/issues/11247#issuecomment-1231067504 遇到相同的问题了。 I0719 17:49:32.507297 53813 env.cpp:46] Env init successfully. *** Aborted at 1658224338 (unix time) try "date -d @1658224338" if you are using GNU date *** ***

[GitHub] [doris] englefly commented on a diff in pull request #12113: [fix](join)join reorder by mistake

2022-08-29 Thread GitBox
englefly commented on code in PR #12113: URL: https://github.com/apache/doris/pull/12113#discussion_r957945642 ## fe/fe-core/src/main/java/org/apache/doris/analysis/FromClause.java: ## @@ -44,6 +46,8 @@ public class FromClause implements ParseNode, Iterable { private boole

[doris] branch master updated (2f192019d3 -> 8370115cf6)

2022-08-29 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 2f192019d3 [bugfix](delete hanlder) delete predicate is merged and could not find schema cause core dump (#12161)

[GitHub] [doris] yiguolei merged pull request #12168: [enhancement](memtracker) Improve performance of tracking real physical memory of PODArray

2022-08-29 Thread GitBox
yiguolei merged PR #12168: URL: https://github.com/apache/doris/pull/12168 -- 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 closed issue #12167: [Bug] PODArray reset_peak() reduces performance

2022-08-29 Thread GitBox
yiguolei closed issue #12167: [Bug] PODArray reset_peak() reduces performance URL: https://github.com/apache/doris/issues/12167 -- 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] englefly commented on a diff in pull request #12113: [fix](join)join reorder by mistake

2022-08-29 Thread GitBox
englefly commented on code in PR #12113: URL: https://github.com/apache/doris/pull/12113#discussion_r957939882 ## fe/fe-core/src/main/java/org/apache/doris/analysis/FromClause.java: ## @@ -44,6 +46,8 @@ public class FromClause implements ParseNode, Iterable { private boole

[GitHub] [doris] xinyiZzz commented on pull request #12168: [enhancement](memtracker) Improve performance of tracking real physical memory of PODArray

2022-08-29 Thread GitBox
xinyiZzz commented on PR #12168: URL: https://github.com/apache/doris/pull/12168#issuecomment-1231044159 > Please provide the flame graph before this PR and after this PR to ensure that this PR really solve the problem. done, reset_peak() does not exist -- This is an automated mess

[doris] branch master updated: [bugfix](delete hanlder) delete predicate is merged and could not find schema cause core dump (#12161)

2022-08-29 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/doris.git The following commit(s) were added to refs/heads/master by this push: new 2f192019d3 [bugfix](delete hanlder) delete predi

[GitHub] [doris] yangzhg closed issue #12155: [Bug] delete handler init failed and core

2022-08-29 Thread GitBox
yangzhg closed issue #12155: [Bug] delete handler init failed and core URL: https://github.com/apache/doris/issues/12155 -- 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] [doris] yangzhg merged pull request #12161: [bugfix](delete hanlder) delete predicate is merged and could not find schema cause core dump

2022-08-29 Thread GitBox
yangzhg merged PR #12161: URL: https://github.com/apache/doris/pull/12161 -- 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

[doris] branch master updated (c885913efe -> 67e94d2aea)

2022-08-29 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 c885913efe Fixed incorrect description of column length limit in partition bucketing documentation. (#12154) ad

[GitHub] [doris] yiguolei merged pull request #12141: [Enhancement](compaction) add compaction use time count

2022-08-29 Thread GitBox
yiguolei merged PR #12141: URL: https://github.com/apache/doris/pull/12141 -- 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 #12161: [bugfix](delete hanlder) delete predicate is merged and could not find schema cause core dump

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12161: URL: https://github.com/apache/doris/pull/12161#issuecomment-1231032921 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 #12161: [bugfix](delete hanlder) delete predicate is merged and could not find schema cause core dump

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

[GitHub] [doris] yiguolei commented on pull request #12168: [enhancement](memtracker) Improve performance of tracking real physical memory of PODArray

2022-08-29 Thread GitBox
yiguolei commented on PR #12168: URL: https://github.com/apache/doris/pull/12168#issuecomment-1231030026 Please provide the flame graph before this PR and after this PR to ensure that this PR really solve the problem. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [doris] github-actions[bot] closed pull request #8090: [refactor](optimize)refactor RowRef/RowRefList to decrease hash table memory size

2022-08-29 Thread GitBox
github-actions[bot] closed pull request #8090: [refactor](optimize)refactor RowRef/RowRefList to decrease hash table memory size URL: https://github.com/apache/doris/pull/8090 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [doris] morrySnow opened a new pull request, #12169: [fix](Nereids)Sum return DoubleType when child is DecimalType by mistake

2022-08-29 Thread GitBox
morrySnow opened a new pull request, #12169: URL: https://github.com/apache/doris/pull/12169 ## Problem summary When Sum's child is Decimal, Return Double Type by mistake lead to result error. ## Checklist(Required) 1. Does it affect the original behavior: - [ ]

[GitHub] [doris] morrySnow commented on a diff in pull request #12159: [feature](nereids) support distinct count for TPC-H

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12159: URL: https://github.com/apache/doris/pull/12159#discussion_r957615245 ## regression-test/data/nereids_syntax_p0/function.out: ## @@ -11,6 +11,11 @@ -- !count -- 3 3 +-- !distinct_count_with_group_by -- +1 1 +1 1 +1

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

2022-08-29 Thread GitBox
morrySnow commented on PR #12129: URL: https://github.com/apache/doris/pull/12129#issuecomment-1230611623 please add some analyze unit test and regression test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

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

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r957574813 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindSlotReference.java: ## @@ -131,6 +132,16 @@ public List buildRules() { ),

[GitHub] [doris] morrySnow commented on a diff in pull request #11842: [feature](planner) Column prune for nereids

2022-08-29 Thread GitBox
morrySnow commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r957567277 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -367,47 +352,32 @@ public PlanFragment visitPhysicalHashJoin(P

[GitHub] [doris] morrySnow commented on a diff in pull request #12008: [feature](Nereids): enable CBO and fix statistics

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12008: URL: https://github.com/apache/doris/pull/12008#discussion_r957547395 ## fe/fe-core/src/main/java/org/apache/doris/nereids/cost/CostCalculator.java: ## @@ -72,13 +72,12 @@ public CostEstimate visitPhysicalOlapScan(PhysicalOlapScan physi

[GitHub] [doris] xinyiZzz opened a new pull request, #12168: [enhancement](memtracker) Improve performance of tracking real physical memory of PODArray

2022-08-29 Thread GitBox
xinyiZzz opened a new pull request, #12168: URL: https://github.com/apache/doris/pull/12168 # Proposed changes Issue Number: close #12167 ## Problem summary Passing `std::shared_ptr` as a function parameter should be avoided, frequent new and delete `std::shared_ptr` tem

[GitHub] [doris] xinyiZzz opened a new issue, #12167: [Bug] PODArray reset_peak() reduces performance

2022-08-29 Thread GitBox
xinyiZzz opened a new issue, #12167: URL: https://github.com/apache/doris/issues/12167 ### 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 580b8dd3ec45b

[doris] branch master updated: Fixed incorrect description of column length limit in partition bucketing documentation. (#12154)

2022-08-29 Thread jakevin
This is an automated email from the ASF dual-hosted git repository. jakevin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new c885913efe Fixed incorrect description of column

[GitHub] [doris] jackwener merged pull request #12154: [Enhancement](doc) Partition table document correction

2022-08-29 Thread GitBox
jackwener merged PR #12154: URL: https://github.com/apache/doris/pull/12154 -- 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] AshinGau opened a new pull request, #12166: [feature-wip](parquet-reader) read and generate array column

2022-08-29 Thread GitBox
AshinGau opened a new pull request, #12166: URL: https://github.com/apache/doris/pull/12166 # 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: [chore](git) Some paths should be tracked by git (#12072)

2022-08-29 Thread adonisling
This is an automated email from the ASF dual-hosted git repository. adonisling 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 c3db282d8e [chore](git) Some paths should be

[GitHub] [doris] adonis0147 merged pull request #12072: [chore](git) Some paths should be tracked by git

2022-08-29 Thread GitBox
adonis0147 merged PR #12072: URL: https://github.com/apache/doris/pull/12072 -- 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] zxealous opened a new pull request, #12165: [optimize](remote) Optimize cache reader use a pre-created buffer when downloading the cache

2022-08-29 Thread GitBox
zxealous opened a new pull request, #12165: URL: https://github.com/apache/doris/pull/12165 # Proposed changes Issue Number: close #12164 ## Problem summary Optimize cache reader use a pre-created buffer when downloading the cache. ## Checklist(Required) 1.

[GitHub] [doris] zxealous opened a new issue, #12164: [Enhancement] Optimize cache reader use pre-created buffer

2022-08-29 Thread GitBox
zxealous opened a new issue, #12164: URL: https://github.com/apache/doris/issues/12164 ### 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 Now, cach

[doris] branch master updated (580b8dd3ec -> 2715ff9e0f)

2022-08-29 Thread caiconghui
This is an automated email from the ASF dual-hosted git repository. caiconghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 580b8dd3ec [improve](Nereids)make some DataType's constructor's access level to private (#12143) add 2715ff9e

[GitHub] [doris] caiconghui merged pull request #12111: [Enhancement](select) Make select variables request handled in fe without be to avoid potential blocked problem when login

2022-08-29 Thread GitBox
caiconghui merged PR #12111: URL: https://github.com/apache/doris/pull/12111 -- 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] LemonLiTree commented on issue #12142: doriswriter csv换行,json字段错位

2022-08-29 Thread GitBox
LemonLiTree commented on issue #12142: URL: https://github.com/apache/doris/issues/12142#issuecomment-1230405074 > 同步odps数据到doris,writer:doriswriter,一开始同步的时候format用默认的csv,直接报错,odps表里的字段有\n换行符,把format改成json之后,字段错位 最开始用 stream load curl命令去导,csv遇到表字段里面有数据是换行的就换行了,导入失败 我觉得你这种情况可以考虑先清洗下数据

[GitHub] [doris] github-actions[bot] commented on pull request #12154: [Enhancement](doc) Partition table document correction

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12154: URL: https://github.com/apache/doris/pull/12154#issuecomment-1230398994 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 #12154: [Enhancement](doc) Partition table document correction

2022-08-29 Thread GitBox
github-actions[bot] commented on PR #12154: URL: https://github.com/apache/doris/pull/12154#issuecomment-1230398910 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 #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957408891 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[GitHub] [doris] jackwener commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
jackwener commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r956998102 ## fe/fe-core/src/main/java/org/apache/doris/nereids/properties/LogicalProperties.java: ## @@ -50,4 +50,21 @@ public List getOutput() { public LogicalProperties w

[GitHub] [doris] morrySnow opened a new pull request, #12163: [fix](Nereids)get NPE from finalize TimestampArithmeticExpr that generated by ExpressionTranslator

2022-08-29 Thread GitBox
morrySnow opened a new pull request, #12163: URL: https://github.com/apache/doris/pull/12163 # Proposed changes This PR: - refactor getDataType in TimestampArithmetic - set TimeUnit correctly when translate TimestampArithmetic to TimestampArithmeticExpr ## Checklist(Requi

[GitHub] [doris] morningman closed pull request #12114: [Draft] enable new scan framework

2022-08-29 Thread GitBox
morningman closed pull request #12114: [Draft] enable new scan framework URL: https://github.com/apache/doris/pull/12114 -- 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 unsu

[GitHub] [doris] morrySnow commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957387267 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[GitHub] [doris] morrySnow commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
morrySnow commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957385006 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/GroupExpression.java: ## @@ -181,13 +208,14 @@ public boolean equals(Object o) { return false;

[GitHub] [doris] jackwener commented on pull request #10543: 分区为左闭右开区间描述修改

2022-08-29 Thread GitBox
jackwener commented on PR #10543: URL: https://github.com/apache/doris/pull/10543#issuecomment-1230351194 Long time no response. close 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

[GitHub] [doris] jackwener closed pull request #10543: 分区为左闭右开区间描述修改

2022-08-29 Thread GitBox
jackwener closed pull request #10543: 分区为左闭右开区间描述修改 URL: https://github.com/apache/doris/pull/10543 -- 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: comm

[GitHub] [doris] cambyzju opened a new pull request, #12162: [Enhancement](metric-type) more readable error message for only metric type

2022-08-29 Thread GitBox
cambyzju opened a new pull request, #12162: URL: https://github.com/apache/doris/pull/12162 # Proposed changes Issue Number: close #xxx ## Problem summary - reproduce step 1: create two table with same schema. ``` CREATE TABLE `t1` ( `k1` int(11) NULL,

[GitHub] [doris] lihuigang commented on pull request #12095: [fix](planner) orthogonal_bitmap_union_count can hit rollup

2022-08-29 Thread GitBox
lihuigang commented on PR #12095: URL: https://github.com/apache/doris/pull/12095#issuecomment-1230326088 ok~ -- 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

[GitHub] [doris] lihuigang commented on pull request #12095: [fix](planner) orthogonal_bitmap_union_count can hit rollup

2022-08-29 Thread GitBox
lihuigang commented on PR #12095: URL: https://github.com/apache/doris/pull/12095#issuecomment-1230324188 ok~ -- 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

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957339706 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957336047 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957336047 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[GitHub] [doris] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957329745 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

[doris] branch master updated (09b8d32421 -> 580b8dd3ec)

2022-08-29 Thread morrysnow
This is an automated email from the ASF dual-hosted git repository. morrysnow pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 09b8d32421 [fix](memtracker) Fix mem limit exceed return wrong format (#12139) add 580b8dd3ec [improve](Nereid

[GitHub] [doris] morrySnow merged pull request #12143: [improve](Nereids): make some Type constructor private

2022-08-29 Thread GitBox
morrySnow merged PR #12143: URL: https://github.com/apache/doris/pull/12143 -- 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] 924060929 commented on a diff in pull request #12147: [refactor](Nereids) refactor memo.copyIn

2022-08-29 Thread GitBox
924060929 commented on code in PR #12147: URL: https://github.com/apache/doris/pull/12147#discussion_r957322578 ## fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java: ## @@ -277,12 +346,172 @@ public void addEnforcerPlan(GroupExpression groupExpression, Group grou

  1   2   >