You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "vinoyang (JIRA)" <ji...@apache.org> on 2019/05/16 08:23:00 UTC
[jira] [Commented] (FLINK-12526) Remove STATE_UPDATER in
ExecutionGraph
[ https://issues.apache.org/jira/browse/FLINK-12526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16841100#comment-16841100 ]
vinoyang commented on FLINK-12526:
----------------------------------
Hi [~till.rohrmann] There are two refactor mode:
# just change the variable assignment style (use {{state}} variable to replace {{STATE_UPDATER}})
# do more further things (refactor the two {{transitionState}} methods change both parameters and return value type because there is no CAS action)
Which one do you prefer? cc [~richtesn]
> Remove STATE_UPDATER in ExecutionGraph
> --------------------------------------
>
> Key: FLINK-12526
> URL: https://issues.apache.org/jira/browse/FLINK-12526
> Project: Flink
> Issue Type: Sub-task
> Components: Runtime / Coordination
> Reporter: vinoyang
> Assignee: vinoyang
> Priority: Major
>
> Since {{ExecutionGraph}} can only be accessed from a single thread(FLINK-11417). We can remove the {{AtomicReferenceFieldUpdater<ExecutionGraph, JobStatus> STATE_UPDATER}} from {{ExecutionGraph}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)