You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/03/18 00:41:34 UTC

[GitHub] [beam] youngoli commented on pull request #17047: [BEAM-3304] documentation for PaneInfo in BPG

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-unsubscribe@beam.apache.org

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