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/06/05 00:19:23 UTC

[GitHub] [beam] damccorm opened a new issue, #21556: Fix Iterable todo In CoGroupByKey

damccorm opened a new issue, #21556:
URL: https://github.com/apache/beam/issues/21556

    
   
   Change iterable_input_value_types to be either type.Iterable or rename the variable to specify it is supposed to be a list.
   
   https://github.com/apache/beam/blob/6e339718c63f14fa3cc6249bec43a1b0e3a32af0/sdks/python/apache_beam/transforms/util.py#L198
   
   Imported from Jira [BEAM-14084](https://issues.apache.org/jira/browse/BEAM-14084). Original Jira may contain additional context.
   Reported by: Ryan.Thompson.


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

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

[GitHub] [beam] ryanthompson591 commented on issue #21556: Fix Iterable todo In CoGroupByKey

Posted by GitBox <gi...@apache.org>.
ryanthompson591 commented on issue #21556:
URL: https://github.com/apache/beam/issues/21556#issuecomment-1245536034

   .take-issue


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

[GitHub] [beam] ryanthompson591 commented on issue #21556: Fix Iterable todo In CoGroupByKey

Posted by GitBox <gi...@apache.org>.
ryanthompson591 commented on issue #21556:
URL: https://github.com/apache/beam/issues/21556#issuecomment-1245540076

   Fixed in PR 
   
   This will be a breaking change to any pipelines that expect the output of CoGroupByKey to be a list instead of an iterable.
   
   I've  [documentation](https://docs.google.com/document/d/1RIzm8-g-0CyVsPb6yasjwokJQFoKHG4NjRUcKHKINu0) on how to fix pipelines for this breaking change.


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

[GitHub] [beam] tvalentyn closed issue #21556: Fix Iterable todo In CoGroupByKey

Posted by GitBox <gi...@apache.org>.
tvalentyn closed issue #21556: Fix Iterable todo In CoGroupByKey
URL: https://github.com/apache/beam/issues/21556


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

[GitHub] [beam] ryanthompson591 commented on issue #21556: Fix Iterable todo In CoGroupByKey

Posted by GitBox <gi...@apache.org>.
ryanthompson591 commented on issue #21556:
URL: https://github.com/apache/beam/issues/21556#issuecomment-1282840123

   .add-label done-and-done


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