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/08/24 17:25:58 UTC

[GitHub] [beam] damccorm opened a new issue, #22844: [Task]: Consolidate exec.Invoke[...] parameters

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

   ### What needs to happen?
   
   Over time, there has been a slow expansion of the number of parameters passed into `exec.Invoke` - https://github.com/apache/beam/blob/80ced0fe2161c58c955b2c030b0035e117bcb103/sdks/go/pkg/beam/core/runtime/exec/fn.go#L166. Each time we add a new param, we are technically making a breaking change even though it is internals which are not supposed to be relied on (which can actually break people, it does internally at Google at least).
   
   We should make one last breaking change to consolidate most of the options into a single object, which can be freely added to as needed.
   
   ### Issue Priority
   
   Priority: 3
   
   ### Issue Component
   
   Component: sdk-go


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