robertwb commented on PR #30587: URL: https://github.com/apache/beam/pull/30587#issuecomment-1995760773
> tested `test_windowing_before_sql` locally and passed > > For curiosity, what caused the window being "unknown" for that specific test? Due to the environment nesting it didn't realize that "interval window" type was understood by all SDKs, so fell back to the generic/unknown wrapping. > And after this change, it seems the window is still "unknown" but the timestamp value is accessible, or I understood it incorrectly? Yep, that's exactly it. -- 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]
