[GitHub] [doris] zy-kkk opened a new pull request, #16557: [typo](docs)Change the lowercase letters of the disk type example to uppercase

2023-02-09 Thread via GitHub
zy-kkk opened a new pull request, #16557: URL: https://github.com/apache/doris/pull/16557 …uppercase # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior:

[GitHub] [doris] xiaokang opened a new pull request, #16558: [Improvement](writer) make DeltaWriter close idempotent to be more robust

2023-02-09 Thread via GitHub
xiaokang opened a new pull request, #16558: URL: https://github.com/apache/doris/pull/16558 # Proposed changes Issue Number: releated to #16453 ## Problem summary return `Status::OK()` instead of `Status::Error()` for close() in `DeltaWriter` if it's already closed.

[GitHub] [doris] github-actions[bot] commented on pull request #16558: [Improvement](writer) make DeltaWriter close idempotent to be more robust

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16558: URL: https://github.com/apache/doris/pull/16558#issuecomment-1423786569 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 #16551: [refactor](Nereids): mv AggregateStrategies to implementation rules

2023-02-09 Thread via GitHub
hello-stephen commented on PR #16551: URL: https://github.com/apache/doris/pull/16551#issuecomment-1423791088 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.43 seconds stream load tsv: 483 seconds loaded 74807831229 Bytes, about 147 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16558: [Improvement](writer) make DeltaWriter close idempotent to be more robust

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16558: URL: https://github.com/apache/doris/pull/16558#issuecomment-1423793007 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 #16558: [Improvement](writer) make DeltaWriter close idempotent to be more robust

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16558: URL: https://github.com/apache/doris/pull/16558#issuecomment-1423793068 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 #16551: [refactor](Nereids): mv AggregateStrategies to implementation rules

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16551: URL: https://github.com/apache/doris/pull/16551#issuecomment-1423796129 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 #16551: [refactor](Nereids): mv AggregateStrategies to implementation rules

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16551: URL: https://github.com/apache/doris/pull/16551#issuecomment-1423796183 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] hello-stephen commented on pull request #16553: [BugFix](cooldown) push correct cooldownttl to be

2023-02-09 Thread via GitHub
hello-stephen commented on PR #16553: URL: https://github.com/apache/doris/pull/16553#issuecomment-1423802052 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.58 seconds stream load tsv: 470 seconds loaded 74807831229 Bytes, about 151 MB/s

[GitHub] [doris] yinzhijian opened a new pull request, #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
yinzhijian opened a new pull request, #16559: URL: https://github.com/apache/doris/pull/16559 # Proposed changes Issue Number: close #16203 ## Problem summary same as https://github.com/apache/doris/pull/16231 but resolve the bug in a more elegant manner ## Chec

[GitHub] [doris] Gabriel39 opened a new issue, #16560: [Enhancement] Support microseconds in function `unix_timestamp`

2023-02-09 Thread via GitHub
Gabriel39 opened a new issue, #16560: URL: https://github.com/apache/doris/issues/16560 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Description Now Doris already

[GitHub] [doris] github-actions[bot] commented on pull request #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16559: URL: https://github.com/apache/doris/pull/16559#issuecomment-1423815428 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 issue, #16561: [Enhancement] Support function `microseconds_add` and `microseconds_sub`

2023-02-09 Thread via GitHub
Gabriel39 opened a new issue, #16561: URL: https://github.com/apache/doris/issues/16561 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Description Now Doris already

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16559: URL: https://github.com/apache/doris/pull/16559#discussion_r1101130974 ## be/src/runtime/result_buffer_mgr.h: ## @@ -47,8 +47,7 @@ class ResultBufferMgr { // the returned sender do not need release // sender is not us

[GitHub] [doris] spaces-X opened a new pull request, #16562: [Feature](vec)(quantile_state): support quantile state in vectorized engine

2023-02-09 Thread via GitHub
spaces-X opened a new pull request, #16562: URL: https://github.com/apache/doris/pull/16562 # Proposed changes Issue Number: close #15868 ## Problem summary 1. support related quantile_state function in vectorized engine 2. add up some regression test cases for the quant

[GitHub] [doris] morningman merged pull request #16485: [improvement](filecache) use dynamic segment size to cache remote file block

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

[doris] branch master updated (9090c5e4e5 -> 539fd684e9)

2023-02-09 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 9090c5e4e5 [fix](docs) Fix memory & rowset count metrics (#16550) add 539fd684e9 [improvement](filecache) use

[GitHub] [doris] zhangstar333 opened a new pull request, #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
zhangstar333 opened a new pull request, #16563: URL: https://github.com/apache/doris/pull/16563 # Proposed changes `when set this.enableFoldConstantByBe = true; and doing prepare work will return error for some reason, eg: can't find function in BE. so will return directly with

[GitHub] [doris] github-actions[bot] commented on pull request #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1423829849 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 #16562: [Feature](vec)(quantile_state): support quantile state in vectorized engine

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16562: URL: https://github.com/apache/doris/pull/16562#discussion_r1101140301 ## be/src/vec/data_types/data_type_quantilestate.h: ## @@ -0,0 +1,85 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contribut

[GitHub] [doris] github-actions[bot] commented on pull request #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16559: URL: https://github.com/apache/doris/pull/16559#issuecomment-1423832239 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 commented on issue #15307: [Bug] Use correct decimal scale for function `round`

2023-02-09 Thread via GitHub
Gabriel39 commented on issue #15307: URL: https://github.com/apache/doris/issues/15307#issuecomment-1423834674 > I have encountered the same problem with version 1.2.1, but version 1.2.0 is normal. (在1.2.1版本中遇到相同问题,在1.2.0中没问题) > > I found that it may be related to the odd and even num

[GitHub] [doris] github-actions[bot] commented on pull request #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1423835071 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] zhannngchen opened a new pull request, #16564: [feature](merge-on-write) add CHECK in compaction to detect duplicate key

2023-02-09 Thread via GitHub
zhannngchen opened a new pull request, #16564: URL: https://github.com/apache/doris/pull/16564 # Proposed changes Issue Number: close #xxx ## Problem summary MoW will mark all duplicate primary key as deleted, so we can add a CHECK while compaction, if MoW's delete bitma

[GitHub] [doris] github-actions[bot] commented on pull request #16343: [Enhancement](Stmt) Set insert_into timeout session variable separately

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16343: URL: https://github.com/apache/doris/pull/16343#issuecomment-1423838054 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 commented on issue #15307: [Bug] Use correct decimal scale for function `round`

2023-02-09 Thread via GitHub
Gabriel39 commented on issue #15307: URL: https://github.com/apache/doris/issues/15307#issuecomment-1423840026 Hi @Prakhar047 , we need this improvement in 2.0 and since time is limited, I decided to assign this issue to @maochongxin again. Sorry for that. If you are interested in Doris and

[GitHub] [doris] github-actions[bot] commented on pull request #16343: [Enhancement](Stmt) Set insert_into timeout session variable separately

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16343: URL: https://github.com/apache/doris/pull/16343#issuecomment-1423841541 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] Raven888888 commented on issue #16538: [Bug] JDBC OutOfMemoryError: Java heap space

2023-02-09 Thread via GitHub
Raven88 commented on issue #16538: URL: https://github.com/apache/doris/issues/16538#issuecomment-1423841756 Bump, same issue here -- 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] github-actions[bot] commented on pull request #16564: [feature](merge-on-write) add CHECK in compaction to detect duplicate key

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16564: URL: https://github.com/apache/doris/pull/16564#issuecomment-1423841974 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] morrySnow opened a new pull request, #16565: [fix](planner)(Nereids) add date and datev2 signature to greatest and least function

2023-02-09 Thread via GitHub
morrySnow opened a new pull request, #16565: URL: https://github.com/apache/doris/pull/16565 # 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 #16378: [Feature](Materialized-View) support multiple slot on one column in materialized view

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16378: URL: https://github.com/apache/doris/pull/16378#issuecomment-1423852329 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] lsy3993 opened a new pull request, #16566: [typo](docs)delete char and varchar in java udf when create

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

[GitHub] [doris] github-actions[bot] commented on pull request #16378: [Feature](Materialized-View) support multiple slot on one column in materialized view

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16378: URL: https://github.com/apache/doris/pull/16378#issuecomment-1423858646 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] YangTan09 opened a new issue, #16567: [Bug] 物化视图子查询不生效

2023-02-09 Thread via GitHub
YangTan09 opened a new issue, #16567: URL: https://github.com/apache/doris/issues/16567 ### 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 doris 1.1.3 ###

[GitHub] [doris] Lchangliang opened a new pull request, #16568: [Improve](regression) Add timeout check in rollup regression

2023-02-09 Thread via GitHub
Lchangliang opened a new pull request, #16568: URL: https://github.com/apache/doris/pull/16568 # Proposed changes Issue Number: close #xxx ## Problem summary When the PRs run CI, the BE crash. The case will be always running. It wastes the resource. So modify it and add

[GitHub] [doris] github-actions[bot] commented on pull request #16565: [fix](planner)(Nereids) add date and datev2 signature to greatest and least function

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16565: URL: https://github.com/apache/doris/pull/16565#issuecomment-1423872681 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] Tanya-W opened a new pull request, #16569: [Optimization](index) Optimization for no need to read raw data for index column that only in where clause

2023-02-09 Thread via GitHub
Tanya-W opened a new pull request, #16569: URL: https://github.com/apache/doris/pull/16569 # Proposed changes Issue Number: close #xxx ## Problem summary At the storage layer, the raw data of the index column will still be read after apply index(bitmap_index or inverted_inde

[GitHub] [doris] github-actions[bot] commented on pull request #16565: [fix](planner)(Nereids) add date and datev2 signature to greatest and least function

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16565: URL: https://github.com/apache/doris/pull/16565#issuecomment-1423872747 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 #16073: [feature](Load)Remove user/password in properties for mysql load to avoid double auth.

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16073: URL: https://github.com/apache/doris/pull/16073#issuecomment-1423876800 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 #16569: [Optimization](index) Optimization for no need to read raw data for index column that only in where clause

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16569: URL: https://github.com/apache/doris/pull/16569#discussion_r1101185873 ## be/src/olap/rowset/segment_v2/segment_iterator.h: ## @@ -51,6 +51,32 @@ struct ColumnPredicateInfo { ColumnPredicateInfo() = default; + +std:

[GitHub] [doris] BiteTheDDDDt opened a new pull request, #16570: [Enchancement](function) display elements number on check_chars_length

2023-02-09 Thread via GitHub
BiteThet opened a new pull request, #16570: URL: https://github.com/apache/doris/pull/16570 # Proposed changes display elements number on check_chars_length ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original b

[GitHub] [doris] yongjinhou opened a new pull request, #16571: [Enchancement](HttpServer) Add http interface authentication

2023-02-09 Thread via GitHub
yongjinhou opened a new pull request, #16571: URL: https://github.com/apache/doris/pull/16571 # Proposed changes Issue Number: close #15658 ## Problem summary Add http interface authentication. ## Checklist(Required) 1. Does it affect the original behavior:

[GitHub] [doris] github-actions[bot] commented on pull request #16570: [Enchancement](function) display elements number on check_chars_length

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16570: URL: https://github.com/apache/doris/pull/16570#issuecomment-1423888729 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 #16562: [Feature](vec)(quantile_state): support quantile state in vectorized engine

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16562: URL: https://github.com/apache/doris/pull/16562#discussion_r1101192840 ## be/test/vec/core/column_complex_test.cpp: ## @@ -83,7 +84,42 @@ class ColumnBitmapTest : public testing::Test { DataTypeBitMap _bitmap_type; }; -

[GitHub] [doris] morrySnow opened a new issue, #16572: [Enhancement](Nereids) type coercion on subquery when bind

2023-02-09 Thread via GitHub
morrySnow opened a new issue, #16572: URL: https://github.com/apache/doris/issues/16572 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Description currently, we do t

[GitHub] [doris] github-actions[bot] commented on pull request #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1423902687 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] wsjz opened a new pull request, #16573: (fix)[dlf]Add default credentials list

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

[GitHub] [doris] github-actions[bot] commented on pull request #16451: [Chore](build) make compile option work on C objects && some refactor of cmakelists

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16451: URL: https://github.com/apache/doris/pull/16451#issuecomment-1423903616 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 #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1423902775 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] AshinGau opened a new pull request, #16574: [improvement](filecache) Use consistent hash to assign the same scan range into the same backend among different queries

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

[GitHub] [doris] dataroaring commented on a diff in pull request #16343: [Enhancement](Stmt) Set insert_into timeout session variable separately

2023-02-09 Thread via GitHub
dataroaring commented on code in PR #16343: URL: https://github.com/apache/doris/pull/16343#discussion_r1101211587 ## be/src/runtime/runtime_state.cpp: ## @@ -21,7 +21,9 @@ #include "runtime/runtime_state.h" #include +#include +#include Review Comment: uselss includ

[GitHub] [doris] Tanya-W opened a new pull request, #16575: [fix](regression test) fix test_array_index.groovy without 'order by' lead to result mismatch

2023-02-09 Thread via GitHub
Tanya-W opened a new pull request, #16575: URL: https://github.com/apache/doris/pull/16575 # Proposed changes Issue Number: close #xxx ## Problem summary fix test_array_index.groovy without 'order by' lead to result mismatch ## Checklist(Required) 1. Does it

[GitHub] [doris] morningman commented on a diff in pull request #16073: [feature](Load)Remove user/password in properties for mysql load to avoid double auth.

2023-02-09 Thread via GitHub
morningman commented on code in PR #16073: URL: https://github.com/apache/doris/pull/16073#discussion_r1101018493 ## fe/fe-core/src/main/java/org/apache/doris/load/loadv2/MysqlLoadManager.java: ## @@ -43,55 +59,297 @@ import org.apache.http.util.EntityUtils; import org.apache.

[GitHub] [doris] github-actions[bot] commented on pull request #16451: [Chore](build) make compile option work on C objects && some refactor of cmakelists

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16451: URL: https://github.com/apache/doris/pull/16451#issuecomment-1423915438 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 #16451: [Chore](build) make compile option work on C objects && some refactor of cmakelists

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16451: URL: https://github.com/apache/doris/pull/16451#issuecomment-1423915506 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] d378971067 opened a new issue, #16576: [Bug] cross join带有order by 跟limit的情况下查询结果总的条数不对

2023-02-09 Thread via GitHub
d378971067 opened a new issue, #16576: URL: https://github.com/apache/doris/issues/16576 ### 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 1.2.1 ### What

[GitHub] [doris] morrySnow opened a new issue, #16577: [Bug](Nereids) Cannot hit materialized view as expected

2023-02-09 Thread via GitHub
morrySnow opened a new issue, #16577: URL: https://github.com/apache/doris/issues/16577 ### 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

[GitHub] [doris] ZhangGuoqiang666 commented on pull request #15216: [fix](jdbc catalog) fix bugs of jdbc catalog and table valued function

2023-02-09 Thread via GitHub
ZhangGuoqiang666 commented on PR #15216: URL: https://github.com/apache/doris/pull/15216#issuecomment-1423924488 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] [doris] xiaokang opened a new pull request, #16578: [Improvement](inverted index) inverted index query match bitmap cache

2023-02-09 Thread via GitHub
xiaokang opened a new pull request, #16578: URL: https://github.com/apache/doris/pull/16578 # Proposed changes Issue Number: close #xxx ## Problem summary Add cache for inverted index query match bitmap to accelerate common query keyword, especially for keyword matching

[GitHub] [doris] github-actions[bot] commented on pull request #16537: [fix](memory) Fix parquet load stack memory overflow

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

[GitHub] [doris] nextdreamblue opened a new pull request, #16579: [fix](subquery) fix bug of using constexpr and count as subquery's ou…

2023-02-09 Thread via GitHub
nextdreamblue opened a new pull request, #16579: URL: https://github.com/apache/doris/pull/16579 …tput Signed-off-by: nextdreamblue # Proposed changes Issue Number: close #16481 ## Problem summary Describe your changes. ## Checklist(Required)

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16578: [Improvement](inverted index) inverted index query match bitmap cache

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16578: URL: https://github.com/apache/doris/pull/16578#discussion_r1101238781 ## be/src/olap/rowset/segment_v2/inverted_index_cache.h: ## @@ -163,5 +165,105 @@ class InvertedIndexCacheHandle { DISALLOW_COPY_AND_ASSIGN(InvertedInd

[GitHub] [doris] hello-stephen commented on pull request #16555: [bugfix](information schema) sometimes fe throw thrift_rpc_error

2023-02-09 Thread via GitHub
hello-stephen commented on PR #16555: URL: https://github.com/apache/doris/pull/16555#issuecomment-1423977889 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.67 seconds stream load tsv: 476 seconds loaded 74807831229 Bytes, about 149 MB/s

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

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16172: URL: https://github.com/apache/doris/pull/16172#issuecomment-1423980162 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] BiteTheDDDDt commented on pull request #16579: [fix](subquery) fix bug of using constexpr and count as subquery's ou…

2023-02-09 Thread via GitHub
BiteThet commented on PR #16579: URL: https://github.com/apache/doris/pull/16579#issuecomment-1423989198 please add regression test case -- 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 sp

[GitHub] [doris] github-actions[bot] commented on pull request #16378: [Feature](Materialized-View) support multiple slot on one column in materialized view

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16378: URL: https://github.com/apache/doris/pull/16378#issuecomment-1423992336 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] yiguolei opened a new pull request, #16580: [refactor](remove unused code) remove load stream mgr

2023-02-09 Thread via GitHub
yiguolei opened a new pull request, #16580: URL: https://github.com/apache/doris/pull/16580 # Proposed changes 1. remove old stream load pipe 2. remove old stream load manager ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it aff

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #16580: [refactor](remove unused code) remove load stream mgr

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #16580: URL: https://github.com/apache/doris/pull/16580#discussion_r1101306551 ## be/test/vec/exec/vtablet_sink_test.cpp: ## @@ -344,7 +343,6 @@ k_add_batch_status = Status::OK(); _env = ExecEnv::GetInstance();

[GitHub] [doris] github-actions[bot] commented on pull request #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16559: URL: https://github.com/apache/doris/pull/16559#issuecomment-1424022828 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 #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16559: URL: https://github.com/apache/doris/pull/16559#issuecomment-1424022912 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 #16073: [feature](Load)Remove user/password in properties for mysql load to avoid double auth.

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16073: URL: https://github.com/apache/doris/pull/16073#issuecomment-1424024302 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 #16580: [refactor](remove unused code) remove load stream mgr

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16580: URL: https://github.com/apache/doris/pull/16580#issuecomment-1424028609 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 #16580: [refactor](remove unused code) remove load stream mgr

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16580: URL: https://github.com/apache/doris/pull/16580#issuecomment-1424028678 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 #16463: [Bug](pipeline) Fix bug in non-local exchange on pipeline engine

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16463: URL: https://github.com/apache/doris/pull/16463#issuecomment-1424030490 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 #16463: [Bug](pipeline) Fix bug in non-local exchange on pipeline engine

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16463: URL: https://github.com/apache/doris/pull/16463#issuecomment-1424030536 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] Gabriel39 merged pull request #16463: [Bug](pipeline) Fix bug in non-local exchange on pipeline engine

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

[doris] branch master updated: [Bug](pipeline) Fix bug in non-local exchange on pipeline engine (#16463)

2023-02-09 Thread gabriellee
This is an automated email from the ASF dual-hosted git repository. gabriellee pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new a038fdaec6 [Bug](pipeline) Fix bug in non-loc

[GitHub] [doris] FreeOnePlus opened a new pull request, #16581: [style](profile)Change Code-Checks Add Docker Dir

2023-02-09 Thread via GitHub
FreeOnePlus opened a new pull request, #16581: URL: https://github.com/apache/doris/pull/16581 # Proposed changes Change Code-Checks Add Docker Dir ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior:

[GitHub] [doris] github-actions[bot] commented on pull request #16581: [style](profile)Change Code-Checks Add Docker Dir

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16581: URL: https://github.com/apache/doris/pull/16581#issuecomment-1424045462 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 #16581: [style](profile)Change Code-Checks Add Docker Dir

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16581: URL: https://github.com/apache/doris/pull/16581#issuecomment-1424045509 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 #16258: [feature](cooldown)Add cooldown delete

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16258: URL: https://github.com/apache/doris/pull/16258#issuecomment-1424048675 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 commented on a diff in pull request #16514: [Improvement](topn) add limit threashold session variable and fuzzy for topn optimizations

2023-02-09 Thread via GitHub
Gabriel39 commented on code in PR #16514: URL: https://github.com/apache/doris/pull/16514#discussion_r1101345616 ## fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java: ## @@ -448,6 +448,7 @@ private void checkAndSetTopnOpt(PlanNode node) { SortNo

[GitHub] [doris] Gabriel39 commented on a diff in pull request #16514: [Improvement](topn) add limit threashold session variable and fuzzy for topn optimizations

2023-02-09 Thread via GitHub
Gabriel39 commented on code in PR #16514: URL: https://github.com/apache/doris/pull/16514#discussion_r1101345616 ## fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java: ## @@ -448,6 +448,7 @@ private void checkAndSetTopnOpt(PlanNode node) { SortNo

[GitHub] [doris] zy-kkk commented on issue #16576: [Bug] cross join带有order by 跟limit的情况下查询结果总的条数不对

2023-02-09 Thread via GitHub
zy-kkk commented on issue #16576: URL: https://github.com/apache/doris/issues/16576#issuecomment-1424058284 Your SQL has `group by`, and the data is deduplicated, so the `limit 5,10 `May only have five entries -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [doris] morningman merged pull request #16558: [Improvement](writer) make DeltaWriter close idempotent to be more robust

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

[doris] branch master updated (a038fdaec6 -> 130b3599bc)

2023-02-09 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from a038fdaec6 [Bug](pipeline) Fix bug in non-local exchange on pipeline engine (#16463) add 130b3599bc [Improvem

[GitHub] [doris] github-actions[bot] commented on pull request #16494: [fix](sequence-column) MergeIterator does not use the correct seq column for comparison

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16494: URL: https://github.com/apache/doris/pull/16494#issuecomment-1424063950 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] luzhijing merged pull request #16322: [Doc](stream_load) remove 2pc config doc

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

[doris] branch master updated: fix (#16322)

2023-02-09 Thread luzhijing
This is an automated email from the ASF dual-hosted git repository. luzhijing 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 77b7b84c34 fix (#16322) 77b7b84c34 is describe

[GitHub] [doris] hello-stephen commented on pull request #16563: [fix](fold_constant) add defer in fold constant to close and open fuzzy

2023-02-09 Thread via GitHub
hello-stephen commented on PR #16563: URL: https://github.com/apache/doris/pull/16563#issuecomment-1424068603 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.93 seconds stream load tsv: 484 seconds loaded 74807831229 Bytes, about 147 MB/s

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

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #15249: URL: https://github.com/apache/doris/pull/15249#issuecomment-1424076656 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 #16555: [bugfix](information schema) sometimes fe throw thrift_rpc_error

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16555: URL: https://github.com/apache/doris/pull/16555#issuecomment-1424077333 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 #16555: [bugfix](information schema) sometimes fe throw thrift_rpc_error

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

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

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16172: URL: https://github.com/apache/doris/pull/16172#issuecomment-1424078108 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 #16383: [Improvement](thrift) optimize thrift message

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16383: URL: https://github.com/apache/doris/pull/16383#issuecomment-1424090992 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 #16559: [improvement](vresultsink) BufferControlBlock may block all fragment threads

2023-02-09 Thread via GitHub
hello-stephen commented on PR #16559: URL: https://github.com/apache/doris/pull/16559#issuecomment-1424108540 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.51 seconds stream load tsv: 477 seconds loaded 74807831229 Bytes, about 149 MB/s

[GitHub] [doris] github-actions[bot] commented on pull request #16557: [typo](docs)Change the lowercase letters of the disk type example to uppercase

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16557: URL: https://github.com/apache/doris/pull/16557#issuecomment-1424119535 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 #16557: [typo](docs)Change the lowercase letters of the disk type example to uppercase

2023-02-09 Thread via GitHub
github-actions[bot] commented on PR #16557: URL: https://github.com/apache/doris/pull/16557#issuecomment-1424119589 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 a diff in pull request #15917: [improvement](scan) Support pushdown execute expr ctx

2023-02-09 Thread via GitHub
github-actions[bot] commented on code in PR #15917: URL: https://github.com/apache/doris/pull/15917#discussion_r1101401156 ## be/src/olap/rowset/segment_v2/segment_iterator.cpp: ## @@ -1452,6 +1536,119 @@ Status SegmentIterator::next_batch(vectorized::Block* block) { retur

  1   2   3   >