raminqaf commented on code in PR #27901:
URL: https://github.com/apache/flink/pull/27901#discussion_r3105263534
##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/plan/nodes/exec/stream/FromChangelogTestPrograms.java:
##########
@@ -93,32 +93,6 @@ public class FromChangelogTestPrograms {
+ "op_mapping => MAP['c, r', 'INSERT',
'ub', 'UPDATE_BEFORE', 'ua', 'UPDATE_AFTER', 'd', 'DELETE'])")
.build();
- public static final TableTestProgram UNMAPPED_CODES_DROPPED =
Review Comment:
Test makes sense when we have the `invalid_op_handling SKIP`
##########
flink-table/flink-table-common/src/test/java/org/apache/flink/table/types/inference/strategies/FromChangelogInputTypeStrategyTest.java:
##########
@@ -61,20 +61,6 @@ protected Stream<TestSpec> testData() {
"d", "DELETE"))
.expectArgumentTypes(TABLE_TYPE, DESCRIPTOR_TYPE,
MAP_TYPE),
- // Valid: retract-style mapping with UPDATE_BEFORE
- TestSpec.forStrategy("Valid with UPDATE_BEFORE",
FROM_CHANGELOG_INPUT_TYPE_STRATEGY)
Review Comment:
Test was duplicate
--
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]
