[GitHub] [flink] slinkydeveloper commented on pull request #18572: [FLINK-25895][table-planner] Splitted ExecNodeGraphJsonPlanGenerator

2022-02-03 Thread GitBox
slinkydeveloper commented on pull request #18572: URL: https://github.com/apache/flink/pull/18572#issuecomment-1029081840 I reverted the removal of `ExecNodeGraphValidator`, as it seems like because of legacy tables, `TemporalTableSourceSpec` can be initiated with empty

[GitHub] [flink] slinkydeveloper commented on pull request #18572: [FLINK-25895][table-planner] Splitted ExecNodeGraphJsonPlanGenerator

2022-01-31 Thread GitBox
slinkydeveloper commented on pull request #18572: URL: https://github.com/apache/flink/pull/18572#issuecomment-1025941687 @matriv given your two comments, does it makes sense to just go ahead and remove this validator? -- This is an automated message from the Apache Git Service. To