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 2020/07/16 19:29:38 UTC

[GitHub] [beam] aaltay commented on pull request #12277: [BEAM-7390] Add combineperkey code snippets

aaltay commented on pull request #12277:
URL: https://github.com/apache/beam/pull/12277#issuecomment-659622474


   - The correct staging link seems to be (http://apache-beam-website-pull-requests.storage.googleapis.com/12277/documentation/transforms/python/aggregation/combineperkey/index.html). I reviewed that it LGTM.
   - Please revert the changes to other files. It makes the history much more cleaner and focused.
   - A question to both you and @rosetn - There is a quite a bit of content duplication between this page and CombineGlobally page. Is there a more de-duplicated way of presenting this content?


----------------------------------------------------------------
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.

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