twalthr commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1155050451
@MartijnVisser there might be misunderstanding here. This "JSON plan" is not the SQL "CompiledPlan" but a JSON representation of the StreamGraph. Nevertheless, this is public API so we need to be careful that we don't break down stream systems. I'm pretty sure that platform teams might have build UI or other tooling around this JSON. Hopefully, their logic is flexible enough to accept an additional field. But nevertheless we should not change or remove fields without a good reason. -- 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]
