TheNeuralBit commented on pull request #12067: URL: https://github.com/apache/beam/pull/12067#issuecomment-651426642
Ok I think I'm finally done churning on this. Sorry for all the changes. What I have now stores the component ids in a `ComponentIdContext` object stored on the Pipeline. The same context is used in both `ExternalTransform.expand` and in `Pipeline.to_runner_api`, so component ids will be consistent across both of those functions, but everything we store in memory will be cleaned up with the Pipeline. PTAL ---------------------------------------------------------------- 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]
