[FLINK-12881] Add more functionalities for ML Params and ParamInfo class

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Component/s: Library / Machine Learning
• Labels: pull-request-available
• Assignee: Xu Yang
• Reporter: Xu Yang
• Created: 18/Jun/19 07:44
• Updated: 30/Jun/19 08:37
• Resolved: 30/Jun/19 08:37

Description

1. change Params$get to support aliases

• support aliases
• if the Params contains the specific parameter and alias, but has more than one value or
• if the Params doesn't contains the specific parameter, while the ParamInfo is optional but has no default value, it will throw exception
• when isOptional is true, contain(ParamInfo) is true, it will return the value found in Params, whether the value is null or not. when isOptional is true, contain(ParamInfo) is false: hasDefaultValue is true, it will return defaultValue. hasDefaultValue is false, it will throw exception. developer should use contain to check that Params has ParamInfo or not.

2. add size, clear, isEmpty, contains, fromJson in Params
3. fix ParamInfo, PipelineStage to adapt new Params

• assert null in alias
• use Params$loadJson in PipelineStage

4. add test cases about aliases

Issue Links

• links to: GitHub Pull Request #8776