klion26 commented on issue #10344: [FLINK-14264][StateBackend][Rest] Expose 
state backend in checkpoint rest api
URL: https://github.com/apache/flink/pull/10344#issuecomment-560210743
 
 
   @zentol thanks for your review.
   First, I want to set the `checkpoint backend name` in 
`StreamingJobGraphGenerator`, so we do not need to update the `checkpoint 
backend name` after, but we may get a `null` when calling 
[streamGraph.getStateBackend()](https://github.com/apache/flink/blob/8c6cc4505a4c27daadb00cd94df8a7e955eb8d52/flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java#L905),
 so I changed to udpate the `checkpoint backend name` as currently in 
`ExecutionGraphBuilder` which can get the truely `checkpoint backend name`, 
this `checkpoint backend` will also be passed into `CheckpointCoordinator` to 
create the `checkpoint storage`.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to