jnh5y commented on code in PR #25186:
URL: https://github.com/apache/flink/pull/25186#discussion_r1711536525
##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/api/CompiledPlanITCase.java:
##########
@@ -80,17 +80,25 @@ protected void setup() throws Exception {
@Test
void testCompilePlanSql() throws IOException {
CompiledPlan compiledPlan =
- tableEnv.compilePlanSql("INSERT INTO MySink SELECT * FROM
MyTable");
+ tableEnv.compilePlanSql(
+ "INSERT INTO MySink SELECT * FROM MyTable"
+ // OPTIONS hints here do not play any
significant role
+ // we just have to be sure that these options
are present in compile
Review Comment:
nit: ^compile^compiled
##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/api/internal/StatementSetImplTest.java:
##########
@@ -65,15 +65,19 @@ void testGetJsonPlan() throws IOException {
tableEnv.executeSql(sinkTableDdl);
StatementSet stmtSet = tableEnv.createStatementSet();
- stmtSet.addInsertSql("INSERT INTO MySink SELECT * FROM MyTable");
+ stmtSet.addInsertSql(
+ "INSERT INTO MySink SELECT * FROM MyTable "
+ // OPTIONS hints here do not play any significant role
+ // we just have to be sure that these options are
present in compile plan
Review Comment:
Ditto
--
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]
