Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/1105
+1
---
Github user vrozov commented on the issue:
https://github.com/apache/drill/pull/1105
@arina-ielchiieva Please review.
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/1105
Squashed commits. @arina-ielchiieva please let me know if you have any
comments.
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/1105
@vrozov @arina-ielchiieva Applied review comments, please let me know if
there are anymore comments.
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/1105
@vrozov @arina-ielchiieva Handled multiple receiver finished messages
correctly. This PR is ready for another round of review.
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/1105
Nope never mind I think I spoke too soon. I just realized we may get
multiple receivingFragmentFinished requests, one for each downstream receiver.
Back to the drawing board.
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/1105
@arina-ielchiieva @vrozov In light of Vlad's comments I have reworked the
synchronization model yet again. This change now removes all synchronization
from PartitionSenderRootExec and enforces the gu