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

    https://github.com/apache/flink/pull/2703#discussion_r87673973
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/configuration/ConfigConstants.java ---
    @@ -489,6 +489,29 @@
         */
        public static final String MESOS_RESOURCEMANAGER_TASKS_CPUS = 
"mesos.resourcemanager.tasks.cpus";
     
    +   /**
    +    * The base container image to use for task managers (started via the 
unified containerizer).
    --- End diff --
    
    Please rephrase the comment: "The container image to use for task 
managers."    In other words, it isn't a 'base' image in this context, and 
numerous containerizers are now supported.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to