twalthr commented on pull request #18479: URL: https://github.com/apache/flink/pull/18479#issuecomment-1023274448
As I said, having the id in Context is fine and also no additional constructor argument is good. I'm just wondering if we can somehow have the id in the JSON plan directly. Can't we add it as a field in the base but fill the field with information from the Context in the base constructor? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
