[GitHub] [doris] jackwener commented on a diff in pull request #12497: [improve](Nereids): add optionalAnd to refactor code.

2022-09-08 Thread GitBox
jackwener commented on code in PR #12497: URL: https://github.com/apache/doris/pull/12497#discussion_r966614018 ## fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/logical/PushPredicateThroughJoin.java: ## @@ -114,22 +113,12 @@ public Rule build() { private

[GitHub] [doris] FreeOnePlus opened a new pull request, #12502: [Feature-wip](docker)Initialize and submit code for rapid development of Docker

2022-09-08 Thread GitBox
FreeOnePlus opened a new pull request, #12502: URL: https://github.com/apache/doris/pull/12502 # Proposed changes Submitted shell script code for Dockerfile, Docker-Compose and related registers. ## Problem summary Describe your changes. ## Checklist(Required)

[GitHub] [doris] github-actions[bot] commented on pull request #12349: [Improvement](sort) improve partial sort algorithm

2022-09-08 Thread GitBox
github-actions[bot] commented on PR #12349: URL: https://github.com/apache/doris/pull/12349#issuecomment-1241510726 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 #12349: [Improvement](sort) improve partial sort algorithm

2022-09-08 Thread GitBox
github-actions[bot] commented on PR #12349: URL: https://github.com/apache/doris/pull/12349#issuecomment-1241510752 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] zhangstar333 commented on a diff in pull request #12492: [feature](outfile) support parquet writer

2022-09-08 Thread GitBox
zhangstar333 commented on code in PR #12492: URL: https://github.com/apache/doris/pull/12492#discussion_r966653604 ## be/src/vec/runtime/vparquet_writer.cpp: ## @@ -0,0 +1,782 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agree

[GitHub] [doris] Yankee24 commented on a diff in pull request #12235: [typo](docs) Update docker-doris in developer

2022-09-08 Thread GitBox
Yankee24 commented on code in PR #12235: URL: https://github.com/apache/doris/pull/12235#discussion_r97462 ## docs/zh-CN/community/developer-guide/docker-dev.md: ## @@ -29,84 +29,27 @@ under the License. ## 相关详细文档导航 -- [使用 Docker 开发镜像编译](/docs/install/source-install/com

[GitHub] [doris] hf200012 opened a new pull request, #12503: [chore](docs)Modify the audit log plugin compilation script

2022-09-08 Thread GitBox
hf200012 opened a new pull request, #12503: URL: https://github.com/apache/doris/pull/12503 # Proposed changes Modify the audit log plugin compilation script Compile the audit loader by default when compiling FE Compile and export to output/fe/plugins directory Issue Numbe

[GitHub] [doris] morrySnow commented on a diff in pull request #12475: [fix](nereids) there are some invalid slot in other join conjuncts (hash join)

2022-09-08 Thread GitBox
morrySnow commented on code in PR #12475: URL: https://github.com/apache/doris/pull/12475#discussion_r966675954 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -471,9 +471,18 @@ public PlanFragment visitPhysicalHashJoin(

[GitHub] [doris] github-actions[bot] commented on pull request #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
github-actions[bot] commented on PR #12490: URL: https://github.com/apache/doris/pull/12490#issuecomment-1241564035 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 #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
github-actions[bot] commented on PR #12490: URL: https://github.com/apache/doris/pull/12490#issuecomment-1241564077 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] morrySnow commented on pull request #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
morrySnow commented on PR #12490: URL: https://github.com/apache/doris/pull/12490#issuecomment-1241565983 > Just curious, does BE support limit output before calculating projections inside a BE plan node currently? Let's say we have a plan `limit -> project -> olap scan`. After applying the

[GitHub] [doris] wangshuo128 commented on pull request #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
wangshuo128 commented on PR #12490: URL: https://github.com/apache/doris/pull/12490#issuecomment-1241572120 > > Just curious, does BE support limit output before calculating projections inside a BE plan node currently? Let's say we have a plan `limit -> project -> olap scan`. After applying

[GitHub] [doris] morrySnow commented on a diff in pull request #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
morrySnow commented on code in PR #12490: URL: https://github.com/apache/doris/pull/12490#discussion_r966686447 ## fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/batch/RewriteJob.java: ## @@ -64,7 +66,8 @@ public RewriteJob(CascadesContext cascadesContext) {

[GitHub] [doris] wangshuo128 commented on pull request #12490: [feature](nereids) push down predicate through limit

2022-09-08 Thread GitBox
wangshuo128 commented on PR #12490: URL: https://github.com/apache/doris/pull/12490#issuecomment-1241573489 BTW, we'd better correct the PR title as well. -- 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

[GitHub] [doris] carlvinhust2012 opened a new pull request, #12504: [fix](array-type) support to export the array type to hdfs

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

<    1   2   3