hackergin commented on code in PR #24765: URL: https://github.com/apache/flink/pull/24765#discussion_r1597406989
########## flink-table/flink-sql-gateway/src/test/java/org/apache/flink/table/gateway/service/MaterializedTableStatementITCase.java: ########## @@ -197,6 +220,29 @@ void testCreateMaterializedTableInContinuousMode() throws Exception { .isEqualTo(CatalogMaterializedTable.RefreshStatus.ACTIVATED); assertThat(actualMaterializedTable.getRefreshHandlerDescription()).isNotEmpty(); assertThat(actualMaterializedTable.getSerializedRefreshHandler()).isNotEmpty(); + + ContinuousRefreshHandler continuousRefreshHandler = + ContinuousRefreshHandlerSerializer.INSTANCE.deserialize( + actualMaterializedTable.getSerializedRefreshHandler(), + getClass().getClassLoader()); + // check the background job is running + String describeJobDDL = + String.format("DESCRIBE JOB '%s'", continuousRefreshHandler.getJobId()); + OperationHandle describeJobHandle = + service.executeStatement(sessionHandle, describeJobDDL, -1, new Configuration()); + awaitOperationTermination(service, sessionHandle, describeJobHandle); + List<RowData> jobResults = fetchAllResults(sessionHandle, describeJobHandle); + assertThat(jobResults.get(0).getString(2).toString()).isEqualTo("RUNNING"); + + // get checkpoint config from the materialized table + CheckpointConfigInfo checkpointConfigInfo = + getCheckpointConfigInfo(clusterClient, continuousRefreshHandler.getJobId()); Review Comment: Currently, I have not found a good way to directly obtain the response of type String. A better way might be to add a get method to CheckpointConfigInfo class. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org