snuyanzin commented on code in PR #27381:
URL: https://github.com/apache/flink/pull/27381#discussion_r2662292616
##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/operations/SqlDdlToOperationConverterTest.java:
##########
@@ -1359,15 +1359,15 @@ void testFailedToAlterTableDropConstraint() throws
Exception {
prepareTable("tb1", 0);
assertThatThrownBy(() -> parse("alter table tb1 drop primary key"))
.isInstanceOf(ValidationException.class)
- .hasMessageContaining("The base table does not define any
primary key.");
+ .hasMessageContaining("The current table does not define any
primary key.");
assertThatThrownBy(() -> parse("alter table tb1 drop constraint ct"))
.isInstanceOf(ValidationException.class)
- .hasMessageContaining("The base table does not define any
primary key.");
+ .hasMessageContaining("The current table does not define any
primary key.");
prepareTable("tb2", 1);
assertThatThrownBy(() -> parse("alter table tb2 drop constraint ct2"))
.isInstanceOf(ValidationException.class)
.hasMessageContaining(
- "The base table does not define a primary key
constraint named 'ct2'. Available constraint name: ['ct1'].");
+ "The current table does not define a primary key
constraint named 'ct2'. Available constraint name: ['ct1'].");
Review Comment:
added
ideally it should be parameterized and refactored, however this PR is
already large, so probably in a separate one
--
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]
