>From Preetham Poluparthi <[email protected]>: Attention is currently required from: Ali Alsuliman, Preetham Poluparthi, [email protected].
Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins, Peeyush Gupta, [email protected], I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19604?usp=email to look at the new patch set (#26). The following approvals got outdated and were removed: Integration-Tests-1 by Jenkins, Verified-1 by Jenkins Change subject: [ASTERIXDB-3606][COMP]: Refactor plan representation: split PlanNode, use DatasetRegistry, replace index-based tracking ...................................................................... [ASTERIXDB-3606][COMP]: Refactor plan representation: split PlanNode, use DatasetRegistry, replace index-based tracking - user model changes: no - storage format changes: no - interface changes: no Details: - Replaces some integer array index variables with Java object references to enhance readability and reduce indexing errors. - Introduces an abstract PlanNode class with concrete subclasses ScanPlanNode and JoinPlanNode, instead of using a single generic PlanNode for all node types. - Implements a centralized DatasetRegistry to manage datasets, eliminating the need to track subsets using integers in various locations. - Standardizes all LeafInput instances as RealLeafInput, removing the need to separately track LogicalOperators as leaf inputs. Ext-ref: MB-66641 Change-Id: I5c5a670fb66b5d525af31a3e8eca1e0c429e7adc --- A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/AbstractLeafInput.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/AbstractPlanNode.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/DatasetRegistry.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/DummyPlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinCondition.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinPlanNode.java D asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanRegistry.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/RealLeafInput.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/ScanPlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/Stats.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/UnnestLeafInput.java 15 files changed, 1,090 insertions(+), 798 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/04/19604/26 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19604?usp=email To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I5c5a670fb66b5d525af31a3e8eca1e0c429e7adc Gerrit-Change-Number: 19604 Gerrit-PatchSet: 26 Gerrit-Owner: Preetham Poluparthi <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Peeyush Gupta <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-CC: Murtadha Hubail <[email protected]> Gerrit-Attention: Ali Alsuliman <[email protected]> Gerrit-Attention: Preetham Poluparthi <[email protected]> Gerrit-Attention: [email protected]
