Hi,

we have a small number of options in Flink whose key is a prefix of other keys.

This violates the YAML spec; when you view the options as a tree only leaf nodes may have properties.

While this is a minor issue from our side I think this can be quite annoying for users, since it means you can't read or validate a Flink config with standard yaml tools.

I'd like to add a suffix to those keys to resolve this particular problem, while still supporting the previous keys (as deprecated).

AFAICT there aren't any risks here,
except if users have a search&replace step for one of these options in the default config of the Flink distribution; however this seems unsafe in any case since the contents of the default config may change.

This would also bring us a step closer to our goal of using a compliant YAML parser.

Reply via email to