gvijay452 opened a new pull request, #26672: URL: https://github.com/apache/flink/pull/26672
<!-- *Thank you very much for contributing to Apache Flink - we are happy that you want to help us improve Flink. To help the community review your contribution in the best possible way, please go through the checklist below, which will get the contribution into a shape in which it can be best reviewed.* *Please understand that we do not do this to make contributions to Flink a hassle. In order to uphold a high standard of quality for code contributions, while at the same time managing a large number of contributions, we need contributors to prepare the contributions well, and give reviewers enough contextual information for the review. Please also understand that contributions that do not follow this guide will take longer to review and thus typically be picked up with lower priority by the community.* What is the purpose of the change This change allows users to dynamically pass kerberos keytab and principal name when submitting a flink job. Ex: flink run -m yarn-cluster -yD security.kerberos.login.keytab=/path/to/kerb.keytab -yD security.kerberos.login.principal=kerbprincipalname /path/to/my.jar Brief change log Continuation to this PR: https://github.com/apache/flink/pull/8166 by [jiasheng55](https://github.com/jiasheng55) Make applyCommandLineOptionsToConfiguration(CommandLine commandLine) a method of interface CustomCommandLine and change it to applyCommandLineOptionsToConfiguration(Configuration configuration, CommandLine commandLine), as suggested by @aljoscha Make org.apache.flink.client.cli.CliFrontend#parseParameters return parsed CommandLine Apply command line options to configuration before installing security modules Verifying this change This change is already covered by existing tests, such as org.apache.flink.client.cli.CliFrontend...* Does this pull request potentially affect one of the following parts: Dependencies (does it add or upgrade a dependency): (yes / no) - no The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no) - no The serializers: (yes / no / don't know) - no The runtime per-record code paths (performance sensitive): (yes / no / don't know) - no Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know) - Yes, deployment The S3 file system connector: (yes / no / don't know) - no Documentation Does this pull request introduce a new feature? (yes / no) - no If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) - n/a -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
