Github user sihuazhou commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5239#discussion_r170243957
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerServices.java
 ---
    @@ -141,7 +147,7 @@ public JobLeaderService getJobLeaderService() {
                return jobLeaderService;
        }
     
    -   public TaskExecutorLocalStateStoresManager getTaskStateManager() {
    +   public TaskExecutorLocalStateStoresManager getTaskManagerStateStore() {
                return taskStateManager;
    --- End diff --
    
    nit: Since we have changed `getTaskStateManager` to 
`getTaskManagerStateStore` maybe we should also change `taskStateManager` to 
`taskManagerStateStore`... this will be more consistent.


---

Reply via email to