Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-03-01 Thread via GitHub
englefly closed pull request #30895: [feature](nereids) set wait_time for runtime filter URL: https://github.com/apache/doris/pull/30895 -- 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-21 Thread via GitHub
doris-robot commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1956129814 TPC-H: Total hot run time: 41690 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-21 Thread via GitHub
github-actions[bot] commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1956082722 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-20 Thread via GitHub
englefly commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1956070786 run buildall -- 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-20 Thread via GitHub
englefly commented on code in PR #30895: URL: https://github.com/apache/doris/pull/30895#discussion_r1497044048 ## fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/RuntimeFilterPrunerForExternalTable.java: ## @@ -52,7 +52,8 @@ public Plan processRoot(Plan plan, C

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-20 Thread via GitHub
github-actions[bot] commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1956039499 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-20 Thread via GitHub
morrySnow commented on code in PR #30895: URL: https://github.com/apache/doris/pull/30895#discussion_r1496933429 ## fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/RuntimeFilterPrunerForExternalTable.java: ## @@ -52,7 +52,8 @@ public Plan processRoot(Plan plan,

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-19 Thread via GitHub
doris-robot commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1952220985 TeamCity be ut coverage result: Function Coverage: 35.77% (8558/23924) Line Coverage: 27.71% (69389/250367) Region Coverage: 26.84% (36010/134174) Branch Coverage: 23.6

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-19 Thread via GitHub
englefly commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1952058031 run buildall -- 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-06 Thread via GitHub
englefly commented on code in PR #30895: URL: https://github.com/apache/doris/pull/30895#discussion_r1479623233 ## be/src/exprs/runtime_filter.cpp: ## @@ -1174,6 +1174,9 @@ Status IRuntimeFilter::init_with_desc(const TRuntimeFilterDesc* desc, const TQue _has_remote_target

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-06 Thread via GitHub
morrySnow commented on code in PR #30895: URL: https://github.com/apache/doris/pull/30895#discussion_r1479418711 ## be/src/exprs/runtime_filter.cpp: ## @@ -1174,6 +1174,9 @@ Status IRuntimeFilter::init_with_desc(const TRuntimeFilterDesc* desc, const TQue _has_remote_target

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-05 Thread via GitHub
github-actions[bot] commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1928873729 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

Re: [PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-05 Thread via GitHub
doris-robot commented on PR #30895: URL: https://github.com/apache/doris/pull/30895#issuecomment-1928864800 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+y

[PR] [feature](nereids) set wait_time for runtime filter [doris]

2024-02-05 Thread via GitHub
englefly opened a new pull request, #30895: URL: https://github.com/apache/doris/pull/30895 ## Proposed changes Issue Number: close #xxx ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto