[FLINK-13965] Keep hasDeprecatedKeys and deprecatedKeys methods in ConfigOption and mark it with @Deprecated annotation

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Done
• Affects Version/s: 1.9.0
• Fix Version/s: 1.8.3, 1.9.1, 1.10.0
• Labels: pull-request-available
• Assignee: vinoyang
• Reporter: vinoyang
• Created: 05/Sep/19 03:41
• Updated: 02/Oct/19 17:48
• Resolved: 18/Sep/19 12:34

Description

In our program based on Flink 1.7.2, we used method ConfigOption#hasDeprecatedKeys. But, this method was renamed to hasFallbackKeys in FLINK-10436. So after we bump our flink version to 1.9.0, we meet compile error.

 
It seems we replaced the deprecated key with an entity FallbackKey. However, I still see the method withDeprecatedKeys. Since we keep the method withDeprecatedKeys, why not keep the method hasDeprecatedKeys. Although, this public API did not marked as Public annotation. IMHO, ConfigOption is hosted in flink-core module, many users also use it, we should maintain the compatibility as far as possible.

Issue Links

• links to: GitHub Pull Request #9691