[ 
https://issues.apache.org/jira/browse/FLINK-17515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17099647#comment-17099647
 ] 

Kostas Kloudas commented on FLINK-17515:
----------------------------------------

Thanks for the comment [~felixzheng] .

The code that I am refactoring makes certain assumptions that may not be valid 
for Kubernetes. For example it registers files as local resource 
(yarn-specific) and it makes use of the Hadoop Filesystem abstraction. The 
purpose of this PR is simply to start cleaning up a bit the code in the 
{{YarnClusterDescriptor}} (although it is a small step). It does not introduce 
any new code or functionality.

If we want to make these things more abstract then I think we should come up 
with a design.

> Move YARN staging functionality to a separate class
> ---------------------------------------------------
>
>                 Key: FLINK-17515
>                 URL: https://issues.apache.org/jira/browse/FLINK-17515
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Client / Job Submission, Deployment / YARN
>    Affects Versions: 1.11.0
>            Reporter: Kostas Kloudas
>            Assignee: Kostas Kloudas
>            Priority: Major
>              Labels: pull-request-available
>
> Currently the file uploading functionality when deploying on Yarn is spread 
> across the {{YarnClusterDescriptor}} and the {{org.apache.flink.yarn.Utils}}. 
> This issue aims at consolidating all this functionality into a single class, 
> the {{YarnApplicationFileUploader}}. 
> This issue does not add/remove/change any of the existing functionality.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to