youngoli commented on pull request #17047: URL: https://github.com/apache/beam/pull/17047#issuecomment-1071906229
This looks mostly good to me, except I think the changes to Beam itself should go in a separate PR and the website changes should be submitted once the feature is available in a current release (so we should wait a little if this is coming in 2.38). Ritesh, can you create a new PR with just the changes in pkg/beam and test/integration? That one should be a quick review & merge from me. -- 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
