Myasuka commented on code in PR #20103:
URL: https://github.com/apache/flink/pull/20103#discussion_r920098039
##########
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/AccessExecutionGraph.java:
##########
@@ -176,6 +176,13 @@ public interface AccessExecutionGraph extends
JobStatusProvider {
*/
Optional<String> getCheckpointStorageName();
+ /**
+ * Returns whether the state changelog is enabled for this ExecutionGraph.
+ *
+ * @return true, if state changelog enabled, false otherwise.
+ */
+ boolean isStateChangelogEnabled();
Review Comment:
I think `isChangelogStateBackendEnabled` looks better.
##########
flink-runtime-web/web-dashboard/src/app/pages/job/checkpoints/job-checkpoints.component.html:
##########
@@ -551,15 +551,15 @@
</td>
</tr>
<tr>
- <td>Changelog state-backend</td>
- <td *ngIf="checkPointConfig['state_backend'] ===
'ChangelogStateBackend'">Enabled</td>
- <td *ngIf="checkPointConfig['state_backend'] !==
'ChangelogStateBackend'">Disabled</td>
+ <td>State Changelog</td>
Review Comment:
```suggestion
<td>Changelog State</td>
```
##########
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/checkpoints/CheckpointConfigInfo.java:
##########
@@ -65,6 +65,8 @@ public class CheckpointConfigInfo implements ResponseBody {
public static final String FIELD_NAME_CHECKPOINTS_AFTER_TASKS_FINISH =
"checkpoints_after_tasks_finish";
+ public static final String FIELD_NAME_STATE_CHANGELOG = "state_changelog";
Review Comment:
I think `changelog_state_enabled` looks better.
##########
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ArchivedExecutionGraph.java:
##########
@@ -391,6 +400,7 @@ public static ArchivedExecutionGraph
createSparseArchivedExecutionGraph(
checkpointingSettings == null ? null :
CheckpointStatsSnapshot.empty(),
checkpointingSettings == null ? null : "Unknown",
checkpointingSettings == null ? null : "Unknown",
+ false,
Review Comment:
This would always be `false`?
--
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]
