[
https://issues.apache.org/jira/browse/FLINK-9823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16542971#comment-16542971
]
ASF GitHub Bot commented on FLINK-9823:
---------------------------------------
Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/6320#discussion_r202305883
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/entrypoint/ClusterConfiguration.java
---
@@ -18,27 +18,43 @@
package org.apache.flink.runtime.entrypoint;
-import org.apache.flink.util.Preconditions;
+import javax.annotation.Nonnull;
+
+import java.util.Properties;
/**
* Configuration class which contains the parsed command line arguments for
* the {@link ClusterEntrypoint}.
*/
public class ClusterConfiguration {
+
+ @Nonnull
private final String configDir;
- private final int restPort;
+ @Nonnull
+ private final Properties dynamicProperties;
+
+ @Nonnull
+ private final String[] args;
- public ClusterConfiguration(String configDir, int restPort) {
- this.configDir = Preconditions.checkNotNull(configDir);
- this.restPort = restPort;
+ public ClusterConfiguration(@Nonnull String configDir, @Nonnull
Properties dynamicProperties, @Nonnull String[] args) {
--- End diff --
`@Nonnull` is not used consistently, e.g., `FlinkParseException` is not
annotated. I think it's easier to assume everything is non-null by default, and
the rest should be annotated with `@Nullable`.
> Add Kubernetes deployment files for standalone job cluster
> ----------------------------------------------------------
>
> Key: FLINK-9823
> URL: https://issues.apache.org/jira/browse/FLINK-9823
> Project: Flink
> Issue Type: New Feature
> Components: Kubernetes
> Affects Versions: 1.6.0
> Reporter: Till Rohrmann
> Assignee: Till Rohrmann
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.6.0
>
>
> Similar to FLINK-9822, it would be helpful for the user to have example
> Kubernetes deployment files to start a standalone job cluster.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
