[GitHub] [doris] Gabriel39 opened a new pull request, #12123: [Improvement](window funnel) restrict timestamp to datetime type in window funnel

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

[doris-website] branch master updated: fix

2022-08-27 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-website.git The following commit(s) were added to refs/heads/master by this push: new 0e83b9de17e fix 0e83b9de17e is desc

[GitHub] [doris] github-actions[bot] commented on pull request #12123: [Improvement](window funnel) restrict timestamp to datetime type in window funnel

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12123: URL: https://github.com/apache/doris/pull/12123#issuecomment-1229153157 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 #12123: [Improvement](window funnel) restrict timestamp to datetime type in window funnel

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12123: URL: https://github.com/apache/doris/pull/12123#issuecomment-1229153165 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 #12120: [api-change](http) change kill query http api by using query id

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12120: URL: https://github.com/apache/doris/pull/12120#issuecomment-1229153329 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 #12120: [api-change](http) change kill query http api by using query id

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12120: URL: https://github.com/apache/doris/pull/12120#issuecomment-1229153334 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] dataroaring commented on a diff in pull request #12117: [fix](storage) storage layer should aggregate on replace column

2022-08-27 Thread GitBox
dataroaring commented on code in PR #12117: URL: https://github.com/apache/doris/pull/12117#discussion_r956558085 ## be/src/exec/olap_scanner.cpp: ## @@ -152,14 +152,15 @@ Status OlapScanner::_init_tablet_reader_params( ->rowset() ->

[GitHub] [doris] dataalive closed issue #11888: [Bug] 数据列内容不一致

2022-08-27 Thread GitBox
dataalive closed issue #11888: [Bug] 数据列内容不一致 URL: https://github.com/apache/doris/issues/11888 -- 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-

[GitHub] [doris] dataalive commented on issue #11888: [Bug] 数据列内容不一致

2022-08-27 Thread GitBox
dataalive commented on issue #11888: URL: https://github.com/apache/doris/issues/11888#issuecomment-1229165123 有临时解决方案及后续版本已经fix,该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 above to go to the spe

[doris-website] branch master updated: fix

2022-08-27 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-website.git The following commit(s) were added to refs/heads/master by this push: new 07955ea831d fix 07955ea831d is desc

[GitHub] [doris] dataroaring opened a new pull request, #12124: [fix](checksum) checksum should respect DELETE_SIGN column

2022-08-27 Thread GitBox
dataroaring opened a new pull request, #12124: URL: https://github.com/apache/doris/pull/12124 If checksum does not respect DELETE_SIGN column, then is 2 replicas' compaction status is not same, then they may generate different checksums probabaly. BTW: We change checksum to vect

[GitHub] [doris] dataroaring merged pull request #12122: [regression-test](unique) Add tpch sf0.1 test under unique table with zstd

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

[GitHub] [doris] dataroaring commented on a diff in pull request #12117: [fix](storage) storage layer should aggregate on replace column

2022-08-27 Thread GitBox
dataroaring commented on code in PR #12117: URL: https://github.com/apache/doris/pull/12117#discussion_r956583630 ## be/src/exec/olap_scanner.cpp: ## @@ -152,14 +152,15 @@ Status OlapScanner::_init_tablet_reader_params( ->rowset() ->

[GitHub] [doris] morningman opened a new pull request, #12125: [dev-1.1.2][api-change](http) change kill query http api by using query id

2022-08-27 Thread GitBox
morningman opened a new pull request, #12125: URL: https://github.com/apache/doris/pull/12125 # Proposed changes cherry pick from #12120 ## Problem summary Now user can cancel query id by http by following steps: 1. Get query id by trace id 2. cancel query by query

[GitHub] [doris] morningman opened a new pull request, #12126: [dev-1.1.2][feature](http) get create table stmt for a given query #11979

2022-08-27 Thread GitBox
morningman opened a new pull request, #12126: URL: https://github.com/apache/doris/pull/12126 # Proposed changes cherry-pick from #11979 ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Y

[GitHub] [doris] github-actions[bot] commented on pull request #12102: [fix](array-type) fix the wrong data when use stream load to import '\N'

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12102: URL: https://github.com/apache/doris/pull/12102#issuecomment-1229196674 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 #12102: [fix](array-type) fix the wrong data when use stream load to import '\N'

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12102: URL: https://github.com/apache/doris/pull/12102#issuecomment-1229196680 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 commented on a diff in pull request #12084: [enhancement](test) add restore action and s3 helper methond

2022-08-27 Thread GitBox
morningman commented on code in PR #12084: URL: https://github.com/apache/doris/pull/12084#discussion_r956585260 ## regression-test/framework/src/main/groovy/org/apache/doris/regression/action/RestoreAction.groovy: ## @@ -0,0 +1,213 @@ +// Licensed to the Apache Software Foundat

[GitHub] [doris] jackwener opened a new pull request, #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
jackwener opened a new pull request, #12127: URL: https://github.com/apache/doris/pull/12127 # Proposed changes Issue Number: close #xxx ## Problem summary - enhance the equals UT - add EqualTo of - minor ## Checklist(Required) 1. Does it affect the or

[GitHub] [doris] morningman commented on a diff in pull request #12124: [fix](checksum) checksum should respect DELETE_SIGN column

2022-08-27 Thread GitBox
morningman commented on code in PR #12124: URL: https://github.com/apache/doris/pull/12124#discussion_r956585665 ## be/src/olap/task/engine_checksum_task.cpp: ## @@ -54,71 +55,58 @@ Status EngineChecksumTask::_compute_checksum() { return Status::OLAPInternalError(OLAP_E

[GitHub] [doris] github-actions[bot] commented on pull request #12125: [dev-1.1.2][api-change](http) change kill query http api by using query id

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12125: URL: https://github.com/apache/doris/pull/12125#issuecomment-1229202791 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] dataalive commented on issue #11126: [Bug] be down

2022-08-27 Thread GitBox
dataalive commented on issue #11126: URL: https://github.com/apache/doris/issues/11126#issuecomment-1229206587 问题已经close,代码到1.1.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 above to go to the specific co

[GitHub] [doris] dataalive closed issue #11126: [Bug] be down

2022-08-27 Thread GitBox
dataalive closed issue #11126: [Bug] be down URL: https://github.com/apache/doris/issues/11126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-u

[GitHub] [doris] morningman commented on a diff in pull request #12107: [fix](view) Fix view not showing specific lengths for varchar type

2022-08-27 Thread GitBox
morningman commented on code in PR #12107: URL: https://github.com/apache/doris/pull/12107#discussion_r956592831 ## fe/fe-core/src/main/java/org/apache/doris/analysis/BaseViewStmt.java: ## @@ -87,18 +87,27 @@ protected void createColumnAndViewDefs(Analyzer analyzer) throws Anal

[GitHub] [doris] github-actions[bot] commented on pull request #11964: [feature](nerieds)Support partition prune

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #11964: URL: https://github.com/apache/doris/pull/11964#issuecomment-1229217106 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 #11964: [feature](nerieds)Support partition prune

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #11964: URL: https://github.com/apache/doris/pull/11964#issuecomment-1229217115 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 a diff in pull request #11964: [feature](nerieds)Support partition prune

2022-08-27 Thread GitBox
morrySnow commented on code in PR #11964: URL: https://github.com/apache/doris/pull/11964#discussion_r956596571 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalOlapScan.java: ## @@ -58,27 +64,17 @@ public LogicalOlapScan(OlapTable table, List qua

[GitHub] [doris] morrySnow commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
morrySnow commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956597319 ## fe/fe-core/src/test/java/org/apache/doris/nereids/trees/plans/PlanEqualsTest.java: ## @@ -48,193 +49,259 @@ import org.junit.jupiter.api.Assertions; import org.ju

[GitHub] [doris] HappenLee opened a new pull request, #12128: [Feature](scan) add num_free_block_in_scan in session variables

2022-08-27 Thread GitBox
HappenLee opened a new pull request, #12128: URL: https://github.com/apache/doris/pull/12128 # Proposed changes Only support in dev-1.1.2,to reduce mem use by free blocks in scan node ## Problem summary Describe your changes. ## Checklist(Required) 1. Does i

[GitHub] [doris] adonis0147 commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
adonis0147 commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956598644 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] adonis0147 opened a new pull request, #12129: [feature-wip](nereids) Support having clause

2022-08-27 Thread GitBox
adonis0147 opened a new pull request, #12129: URL: https://github.com/apache/doris/pull/12129 # Proposed changes Implement the having clause for Nereids Planner. **NOTE:** This pr aims at making Nereids Planner generate the correct logical plan and physical plan only. Th

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

2022-08-27 Thread GitBox
morrySnow commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956598271 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -255,12 +255,12 @@ public PlanFragment visitPhysicalOlapScan(P

[GitHub] [doris] jackwener commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
jackwener commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956601249 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] adonis0147 commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
adonis0147 commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956601868 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] adonis0147 commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
adonis0147 commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956601868 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] adonis0147 commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
adonis0147 commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956601868 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] adonis0147 commented on a diff in pull request #12127: [fix](Nereids): enhance equals UT

2022-08-27 Thread GitBox
adonis0147 commented on code in PR #12127: URL: https://github.com/apache/doris/pull/12127#discussion_r956601868 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/EqualTo.java: ## @@ -33,6 +33,10 @@ public EqualTo(Expression left, Expression right) {

[GitHub] [doris] morrySnow opened a new pull request, #12130: [fix](Nereids)join output order need same with child plan node output when translate

2022-08-27 Thread GitBox
morrySnow opened a new pull request, #12130: URL: https://github.com/apache/doris/pull/12130 ## Problem summary In BE, There is an implicit convention that HashJoinNode's left child's output Slot must before right child's output slot in intermediateTuple. However, after we do commu

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956609347 ## fe/fe-core/src/main/java/org/apache/doris/analysis/SlotDescriptor.java: ## @@ -338,6 +338,7 @@ public String getExplainString(String prefix) { builder.ap

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956609518 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -125,10 +126,12 @@ public PlanFragment translatePlan(PhysicalP

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956609518 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -125,10 +126,12 @@ public PlanFragment translatePlan(PhysicalP

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956609754 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -255,12 +255,12 @@ public PlanFragment visitPhysicalOlapScan(

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956610017 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -434,28 +432,65 @@ public PlanFragment visitPhysicalProject(P

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956610406 ## fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java: ## @@ -142,6 +139,10 @@ public abstract class PlanNode extends TreeNode implements PlanStats {

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956610431 ## fe/fe-core/src/main/java/org/apache/doris/planner/HashJoinNode.java: ## @@ -1080,13 +1080,18 @@ protected void toThrift(TPlanNode msg) { } if (vS

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956610549 ## fe/fe-core/src/main/java/org/apache/doris/planner/HashJoinNode.java: ## @@ -1080,13 +1080,18 @@ protected void toThrift(TPlanNode msg) { } if (vS

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

2022-08-27 Thread GitBox
morrySnow commented on code in PR #12129: URL: https://github.com/apache/doris/pull/12129#discussion_r956609185 ## fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java: ## @@ -949,4 +958,18 @@ public Expression visitExist(ExistContext context) {

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611039 ## fe/fe-core/src/main/java/org/apache/doris/planner/ExchangeNode.java: ## @@ -83,6 +84,11 @@ public ExchangeNode(PlanNodeId id, PlanNode inputNode, boolean copyConj

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611126 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PlanTranslatorContext.java: ## @@ -110,23 +113,25 @@ public SlotDescriptor createSlotDesc(TupleDe

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611146 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PlanTranslatorContext.java: ## @@ -54,6 +55,8 @@ public class PlanTranslatorContext { priv

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611207 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/Expression.java: ## @@ -146,4 +153,34 @@ public boolean equals(Object o) { public int hash

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611440 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/AbstractPhysicalPlan.java: ## @@ -64,6 +64,11 @@ public List getOutput() { return l

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611813 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -434,28 +432,65 @@ public PlanFragment visitPhysicalProject(P

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956611917 ## fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java: ## @@ -434,28 +432,65 @@ public PlanFragment visitPhysicalProject(P

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956612049 ## fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/batch/RewriteJob.java: ## @@ -51,6 +53,10 @@ public RewriteJob(CascadesContext cascadesContext) {

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956615096 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/visitor/RewriteAliasToChildExpr.java: ## @@ -0,0 +1,37 @@ +// Licensed to the Apache Software F

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956612049 ## fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/batch/RewriteJob.java: ## @@ -51,6 +53,10 @@ public RewriteJob(CascadesContext cascadesContext) {

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

2022-08-27 Thread GitBox
Kikyou1997 commented on code in PR #11842: URL: https://github.com/apache/doris/pull/11842#discussion_r956612049 ## fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/batch/RewriteJob.java: ## @@ -51,6 +53,10 @@ public RewriteJob(CascadesContext cascadesContext) {

[GitHub] [doris] github-actions[bot] commented on pull request #8243: [improvment] Ignore the case of column name, when do the stream load in json format

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #8243: URL: https://github.com/apache/doris/pull/8243#issuecomment-1229343666 We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.

[GitHub] [doris] englefly commented on a diff in pull request #12106: [refactor](BE) return error status when vslot_ref contains invalid slot_id

2022-08-27 Thread GitBox
englefly commented on code in PR #12106: URL: https://github.com/apache/doris/pull/12106#discussion_r956653808 ## be/src/vec/exprs/vslot_ref.cpp: ## @@ -51,16 +51,16 @@ Status VSlotRef::prepare(doris::RuntimeState* state, const doris::RowDescriptor& return Status::Inte

[GitHub] [doris-website] pangzhili opened a new pull request, #71: Modify the Duplicate model document error in the data-module directory

2022-08-27 Thread GitBox
pangzhili opened a new pull request, #71: URL: https://github.com/apache/doris-website/pull/71 When creating the table of the Duplicate model, specify the SortKet of error_code as Yes, but when creating the table later, the DUPLICATE KEY does not contain the error_code field, only the times

[GitHub] [doris] freemandealer commented on a diff in pull request #11939: [fix](memtracker) Remove useless memory exceed check

2022-08-27 Thread GitBox
freemandealer commented on code in PR #11939: URL: https://github.com/apache/doris/pull/11939#discussion_r956656266 ## be/src/exec/tablet_sink.cpp: ## @@ -302,8 +302,7 @@ Status NodeChannel::add_row(Tuple* input_tuple, int64_t tablet_id) { // _cancelled may be set by rpc c

[GitHub] [doris] pangzhili opened a new pull request, #12131: update data-model document problem

2022-08-27 Thread GitBox
pangzhili opened a new pull request, #12131: URL: https://github.com/apache/doris/pull/12131 When creating the table of the Duplicate model, specify the SortKet of error_code as Yes, but when creating the table later, the DUPLICATE KEY does not contain the error_code field, only the timesta

[GitHub] [doris] kpfly commented on a diff in pull request #11939: [fix](memtracker) Remove useless memory exceed check

2022-08-27 Thread GitBox
kpfly commented on code in PR #11939: URL: https://github.com/apache/doris/pull/11939#discussion_r956662011 ## be/src/exec/tablet_sink.cpp: ## @@ -302,8 +302,7 @@ Status NodeChannel::add_row(Tuple* input_tuple, int64_t tablet_id) { // _cancelled may be set by rpc callback,

[GitHub] [doris] jackwener opened a new pull request, #12132: [improve](Nereids): unify all plan toString().

2022-08-27 Thread GitBox
jackwener opened a new pull request, #12132: URL: https://github.com/apache/doris/pull/12132 # Proposed changes Issue Number: close #xxx ## Problem summary unify all plan toString(). ## Checklist(Required) 1. Does it affect the original behavior: - [

[GitHub] [doris] github-actions[bot] commented on pull request #12119: [Improvement](load) accelerate streamload and compaction

2022-08-27 Thread GitBox
github-actions[bot] commented on PR #12119: URL: https://github.com/apache/doris/pull/12119#issuecomment-1229393703 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 #12119: [Improvement](load) accelerate streamload and compaction

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