James Fang has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3338 )
Change subject: [NO ISSUE] Create an abstraction for the ForwardOperatorDescriptor ...................................................................... Patch Set 4: (5 comments) https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/AbstractForwardPOperator.java File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/AbstractForwardPOperator.java: https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/AbstractForwardPOperator.java@49 PS4, Line 49: RangeMap generator > could we change this now that it's generic? Done https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/AbstractForwardPOperator.java@125 PS4, Line 125: rangemapSource > could we change this to sideDataSource? (and other similar places that ment Done https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/SortForwardPOperator.java File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/SortForwardPOperator.java: https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/SortForwardPOperator.java@39 PS4, Line 39: getRangeMapKey > we should probably change this now that it's not necessarily a range map. a Done https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/base/AbstractForwardOperatorDescriptor.java File hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/base/AbstractForwardOperatorDescriptor.java: https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/base/AbstractForwardOperatorDescriptor.java@32 PS4, Line 32: keyInContext > could we rename this to sideDataKey? Done https://asterix-gerrit.ics.uci.edu/#/c/3338/4/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/base/AbstractForwardOperatorDescriptor.java@46 PS4, Line 46: setActivities() > could we change this to two methods: Done -- To view, visit https://asterix-gerrit.ics.uci.edu/3338 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc3db4b386e69a98c2a1c40dadc96eb3e1a5d4fa Gerrit-Change-Number: 3338 Gerrit-PatchSet: 4 Gerrit-Owner: James Fang <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose (1000171) Gerrit-Reviewer: James Fang <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Comment-Date: Fri, 19 Apr 2019 16:58:45 +0000 Gerrit-HasComments: Yes
