abdullah alamoudi has posted comments on this change. Change subject: Support Sending Messages Alongside Frame Data ......................................................................
Patch Set 6: (5 comments) https://asterix-gerrit.ics.uci.edu/#/c/604/6/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/WriteResultOperator.java File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/WriteResultOperator.java: Line 23: > Revert file Done https://asterix-gerrit.ics.uci.edu/#/c/604/6/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/HashPartitionExchangePOperator.java File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/HashPartitionExchangePOperator.java: Line 93: int i = 0; > Revert this file Done https://asterix-gerrit.ics.uci.edu/#/c/604/6/hyracks/hyracks-dataflow-common/src/main/java/org/apache/hyracks/dataflow/common/io/MessagingFrameTupleAppender.java File hyracks/hyracks-dataflow-common/src/main/java/org/apache/hyracks/dataflow/common/io/MessagingFrameTupleAppender.java: Line 33: private IHyracksTaskContext ctx; > final Done Line 55: if (getTupleCount() > 0) { > use tupleCount variable Done https://asterix-gerrit.ics.uci.edu/#/c/604/6/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/connectors/PartitionDataWriter.java File hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/connectors/PartitionDataWriter.java: Line 64: protected FrameTupleAppender createTupleAppender(IHyracksTaskContext ctx2) { > ctx2 -> ctx Done -- To view, visit https://asterix-gerrit.ics.uci.edu/604 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I56ae8124052c13a52ca42965b8d00e18ecf35a28 Gerrit-PatchSet: 6 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Steven Jacobs <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: Yes
