youngoli commented on pull request #13255: URL: https://github.com/apache/beam/pull/13255#issuecomment-723354730
I worked around the scope problem by essentially removing any composite transforms from the proto representation of the external transform, before continuing on with the existing logic. I spent a while trying to more properly fix it so it would work with the composite transform in there, but it seems like there are many places in the existing logic where composites weren't accounted for, and at the end of the day I don't think there's anything gained at the moment by changing the logic to work with the composite in there. @lostluck let me know what you think. Any issues we might hit with this workaround in the future? ---------------------------------------------------------------- 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: [email protected]
