[GitHub] [flink] MartijnVisser commented on pull request #17873: [FLINK-25009][CLI] Output slotSharingGroup as part of JsonGraph

2023-06-06 Thread via GitHub
MartijnVisser commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1579093655 @KarmaGYZ Are you still planning to review this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flink] MartijnVisser commented on pull request #17873: [FLINK-25009][CLI] Output slotSharingGroup as part of JsonGraph

2022-10-20 Thread GitBox
MartijnVisser commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1285943401 Asking @godfreyhe to have a look at the PR -- 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

[GitHub] [flink] MartijnVisser commented on pull request #17873: [FLINK-25009][CLI] Output slotSharingGroup as part of JsonGraph

2022-08-01 Thread GitBox
MartijnVisser commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1201232277 No feedback from my end anymore :) -- 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 th

[GitHub] [flink] MartijnVisser commented on pull request #17873: [FLINK-25009][CLI] Output slotSharingGroup as part of JsonGraph

2022-06-14 Thread GitBox
MartijnVisser commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1155056713 Thanks for clarifying @twalthr :) -- 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 t

[GitHub] [flink] MartijnVisser commented on pull request #17873: [FLINK-25009][CLI] Output slotSharingGroup as part of JsonGraph

2022-06-07 Thread GitBox
MartijnVisser commented on PR #17873: URL: https://github.com/apache/flink/pull/17873#issuecomment-1148993708 Breaking the JSON plan is something that can't be done, because it's required for the SQL upgrade story -- This is an automated message from the Apache Git Service. To respond to