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

Chesnay Schepler commented on FLINK-21036:
------------------------------------------

hmm...for plain docker that certainly works, but for kubernetes I suppose the 
issue is that'd you have to modify the container args parameter, with which you 
end up mixing what process you run (e.g., taskmanager) and what config options 
you want to set.
I don't know enough about the kubernetes configuration, but isn't there 
something where you can share bits across different files?


> Consider removing automatic configuration fo number of slots from docker
> ------------------------------------------------------------------------
>
>                 Key: FLINK-21036
>                 URL: https://issues.apache.org/jira/browse/FLINK-21036
>             Project: Flink
>          Issue Type: Improvement
>          Components: Deployment / Scripts
>            Reporter: Chesnay Schepler
>            Assignee: Chesnay Schepler
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.13.0
>
>
> The {{docker-entrypoint.sh}} supports setting the number of task slots via 
> the {{TASK_MANAGER_NUMBER_OF_TASK_SLOTS}} environment variable, which 
> defaults to the number of cpu cores via {{$(grep -c ^processor 
> /proc/cpuinfo)}}.
> The environment variable itself is redundant nowadays since we introduced 
> {{FLINK_PROPERTIES}}, and is no longer documented.
> Defaulting to the number of CPU cores can be considered convenience, but it 
> seems odd to have this specific to docker while the distribution defaults to 
> {{1}}.
> The bigger issue in my mind though is that this creates a configuration 
> mismatch between the Job- and TaskManager processes; the ResourceManager 
> specifically needs to know how many slots a worker has to make decisions 
> about redundancy and allocating resources.



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

Reply via email to