Abacn commented on PR #38410:
URL: https://github.com/apache/beam/pull/38410#issuecomment-4437243308
> I don't really understand this PR - can you explain it? The
initializeContext/clearContexts structure is the one that has been there for a
very long time, so I'm not sure why you're changing it.
Yeah agree that this part sounds less relevant. This is just an AI-assistent
attempt and it doesn't look clear to me either. It's just for reference.
The part that moves `streamName.write("");` around looks a bit more
plausible. However in any case we'd need way (and time) to test it. In terms of
release schedule would it be preferred to rollback (#38479) then roll forward
to get a clean commit at release cut?
--
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]
