csviri opened a new pull request, #1012: URL: https://github.com/apache/flink-kubernetes-operator/pull/1012
## What is the purpose of the change This change introduces `config` field for `FlinkDeployment` and `FlinkSessionJob` as an alternative to `flinkConfiguration`. While `flinkConfiguration` accepts string key-value pairs, the new field accepts arbitrary yaml input, as defined in [flink configuration](https://nightlies.apache.org/flink/flink-docs-master/docs/deployment/config/#flink-configuration-file) documentation. While this change allows adding values to both `flinkConfiguration` and `config`, if there is a value with same key the one from `config` will take precedence. However, it is the users resposibility to make sure that there are no overlapping keys, thus keys with deprecated names and with actual key names, in that case behavior is not deterministic. TODO: Flip ## Brief change log - adds `config` to CRD - backfills `flinkConfiguration` from `config` ## Verifying this change TODO describe tests ## Does this pull request potentially affect one of the following parts: - Dependencies (does it add or upgrade a dependency): no - The public API, i.e., is any changes to the `CustomResourceDescriptors`: yes - Core observer or reconciler logic that is regularly executed: yes ## Documentation - Does this pull request introduce a new feature? yes - If yes, how is the feature documented? docs and JavaDocs (TODO) -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
