ibzib commented on pull request #15174:
URL: https://github.com/apache/beam/pull/15174#issuecomment-902327635


   > Yes, I tried but it causes a compilation error because of the return type 
of `.asSingletonView()`. I tried setting ` .withoutDefaults()` just for 
Unbounded PCollections, and it is working now. 
[7473306](https://github.com/apache/beam/commit/7473306d414322f284fb3b29f16e280108f9e2d5)
   
   I think what we really want is to check whether or not the PCollection is in 
the global window, which is orthogonal to whether or not it's bounded. (In the 
test data, the bounded input is globally windowed while the unbounded input is 
not, but that's not always true generally.)
   
   I'll try to do a full review of this PR tomorrow, but I might not get to it 
till next week.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to