Github user mxm commented on a diff in the pull request:
https://github.com/apache/flink/pull/2275#discussion_r73159225
--- Diff:
flink-core/src/main/java/org/apache/flink/configuration/ConfigConstants.java ---
@@ -1016,6 +1016,23 @@
/** The environment variable name which contains the location of the
lib folder */
public static final String ENV_FLINK_LIB_DIR = "FLINK_LIB_DIR";
+ // -------------------------------- Security
-------------------------------
+
+ /**
+ * The config parameter defining security credentials required
+ * for securing Flink cluster.
+ */
+
+ /** Keytab file key name to be used in flink configuration file */
+ public static final String SECURITY_KEYTAB_KEY = "security.keytab";
+
+ /** Kerberos security principal key name to be used in flink
configuration file */
+ public static final String SECURITY_PRINCIPAL_KEY =
"security.principal";
+
+ /** Keytab file name populated in YARN container */
+ public static final String KEYTAB_FILE_NAME = "krb5.keytab";
--- End diff --
My point is that this class is declared `@Public` and we will have to stick
with this config key name. It could also reside in the `flink-yarn` module
because it doesn't have to be exposed to the user.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
