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

    https://github.com/apache/flink/pull/5593#discussion_r171231898
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ---
    @@ -1317,15 +1340,18 @@ public static ApplicationSubmissionContextReflector 
getInstance() {
                private static final String APPLICATION_TAGS_METHOD_NAME = 
"setApplicationTags";
                private static final String ATTEMPT_FAILURES_METHOD_NAME = 
"setAttemptFailuresValidityInterval";
                private static final String KEEP_CONTAINERS_METHOD_NAME = 
"setKeepContainersAcrossApplicationAttempts";
    +           private static final String NODE_LABEL_EXPRESSION_NAME = 
"setNodeLabelExpression";
     
                private final Method applicationTagsMethod;
                private final Method attemptFailuresValidityIntervalMethod;
                private final Method keepContainersMethod;
    +           private final Method nodeLabelExpressionMethod;
    --- End diff --
    
    Please mark the field `@Nullable`.


---

Reply via email to