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

Reply via email to