abdullah alamoudi has posted comments on this change.

Change subject: Allow Project Runtime to Pass Through an Empty Frame
......................................................................


Patch Set 2:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/634/2//COMMIT_MSG
Commit Message:

Line 9: Before this change, project runtime expects at least a single record. 
now it can also process an empty frame.
> Could we keep the lines in the comment to 77 chars?
Done.
The change for the register node work is already in master. hence it is not 
mentioned here.


https://asterix-gerrit.ics.uci.edu/#/c/634/2/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/StreamProjectRuntimeFactory.java
File 
algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/StreamProjectRuntimeFactory.java:

Line 78:                         }
> Couldn't we just do
the additional value is that if flushFramesRapidly is set, we forward the 
output at the end. if not, we will just append the tuple and return the the 
previous operator.


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/634
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I87dc6eb83a748f7f91610e7d11ebaec9be914e29
Gerrit-PatchSet: 2
Gerrit-Project: hyracks
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: Yes

Reply via email to