You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by "zhijiang (JIRA)" <ji...@apache.org> on 2019/06/18 08:17:00 UTC

[jira] [Updated] (FLINK-12882) Remove ExecutionAttemptID field from ShuffleEnvironment#createResultPartitionWriters

     [ https://issues.apache.org/jira/browse/FLINK-12882?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

zhijiang updated FLINK-12882:
-----------------------------
    Description: 
The {{ExecutionAttemptID}} is only used for constructing {{ResultPartitionID}} during creating {{ResultPartitionWriters in ShuffleEnvironment}}.

Actually the {{ResultPartitionID}} could be got directly from {{ResultPartitionDeploymentDescriptor}} via {{ShuffleDescriptor#getResultPartitionID}} then we could avoid passing this field in the interface to make it simple.

  was:
The {{ExecutionAttemptID}} is only used for constructing {{ResultPartitionID}} during creating {{ResultPartitionWriter}}s in {{ShuffleEnvironment}}.

Actually the {{ResultPartitionID}} could be got directly from {{ResultPartitionDeploymentDescriptor}} via {{ShuffleDescriptor#getResultPartitionID}} then we could avoid passing this field in the interface to make it simple.


> Remove ExecutionAttemptID field from ShuffleEnvironment#createResultPartitionWriters
> ------------------------------------------------------------------------------------
>
>                 Key: FLINK-12882
>                 URL: https://issues.apache.org/jira/browse/FLINK-12882
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Runtime / Network
>            Reporter: zhijiang
>            Assignee: zhijiang
>            Priority: Minor
>
> The {{ExecutionAttemptID}} is only used for constructing {{ResultPartitionID}} during creating {{ResultPartitionWriters in ShuffleEnvironment}}.
> Actually the {{ResultPartitionID}} could be got directly from {{ResultPartitionDeploymentDescriptor}} via {{ShuffleDescriptor#getResultPartitionID}} then we could avoid passing this field in the interface to make it simple.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)