>From Ali Alsuliman <[email protected]>: Ali Alsuliman has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17240 )
Change subject: [NO ISSUE][COMP] Reset singleDatasetPreds when init'ing JoinEnum ...................................................................... [NO ISSUE][COMP] Reset singleDatasetPreds when init'ing JoinEnum - user model changes: no - storage format changes: no - interface changes: no Details: Change-Id: I287553169add5e0e9fc49388d4838078144cd7b1 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17240 Integration-Tests: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Reviewed-by: Ali Alsuliman <[email protected]> Reviewed-by: Wail Alkowaileet <[email protected]> --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java 1 file changed, 21 insertions(+), 1 deletion(-) Approvals: Wail Alkowaileet: Looks good to me, approved Ali Alsuliman: Looks good to me, but someone else must approve Jenkins: Verified; Verified Objections: Anon. E. Moose #1000171: Violations found diff --git a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java index 306bad9..b1a6fb6 100644 --- a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java +++ b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java @@ -100,7 +100,6 @@ protected ICostMethods costMethods; public JoinEnum() { - singleDatasetPreds = new ArrayList<>(); } public void initEnum(AbstractLogicalOperator op, boolean cboMode, boolean cboTestMode, int numberOfFromTerms, @@ -108,6 +107,7 @@ Map<EmptyTupleSourceOperator, ILogicalOperator> joinLeafInputsHashMap, Map<DataSourceScanOperator, EmptyTupleSourceOperator> dataSourceEmptyTupleHashMap, List<ILogicalOperator> internalEdges, List<ILogicalOperator> joinOps, IOptimizationContext context) { + this.singleDatasetPreds = new ArrayList<>(); this.joinConditions = new ArrayList<>(); this.internalEdges = new ArrayList<>(); this.allPlans = new ArrayList<>(); -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17240 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I287553169add5e0e9fc49388d4838078144cd7b1 Gerrit-Change-Number: 17240 Gerrit-PatchSet: 2 Gerrit-Owner: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Wail Alkowaileet <[email protected]> Gerrit-MessageType: merged
