gustavodemorais commented on code in PR #26899:
URL: https://github.com/apache/flink/pull/26899#discussion_r2272942912
##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/plan/nodes/exec/stream/ProcessTableFunctionSemanticTests.java:
##########
@@ -26,6 +27,11 @@
/** Semantic tests for {@link StreamExecProcessTableFunction}. */
public class ProcessTableFunctionSemanticTests extends SemanticTestBase {
+ // TODO [FLINK-38233]: Remove this override when PTF support in
+ // StreamNonDeterministicUpdatePlanVisitor is added.
+ @Override
+ protected void applyDefaultEnvironmentOptions(TableConfig config) {}
Review Comment:
- Updated ProcessTableFunctionSemanticTests to call super and then ignore
NDU analyzer
- We have two suites that run PTF Test Programs:
`ProcessTableFunctionSemanticTests`, `QueryOperationSqlSemanticTest` and
`QueryOperationSqlSerializationTest`
- I disabled it for the entire `ProcessTableFunctionSemanticTests` suite.
- For `QueryOperationSqlSemanticTest` and
`QueryOperationSqlSerializationTest`, I only disabled it directly for the two
tests that use PTFs instead of disabling for all the 15+ tests in the suites.
--
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]
