priyansndesai commented on code in PR #31420:
URL: https://github.com/apache/beam/pull/31420#discussion_r1617638363


##########
sdks/go/pkg/beam/core/runtime/exec/datasource.go:
##########
@@ -125,9 +130,16 @@ func (n *DataSource) process(ctx context.Context, data 
func(bcr *byteCountReader
        bcr := byteCountReader{reader: &r, count: &byteCount}
 
        for {
+               // The SDK is currently waiting for the Runner to send data for 
it to be
+               // processed. Hence, the boolean is marked as true.
+               n.waitingForRunnerToSendData.Store(true)
                var err error
                select {
                case e, ok := <-elms:
+                       // Upon receiving some item from the Runner, the SDK is 
busy. Hence, the
+                       // boolean is marked as false as it is not waiting for 
the Runner to send
+                       // it anything.

Review Comment:
   Done.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to