lindong28 opened a new pull request #35: URL: https://github.com/apache/flink-ml/pull/35
## What is the purpose of the change This PR updates WithParams::set(...) to throw Exception if the given param is not defined on this instance. This is because we don't have use-case where users will want to set a parameter not defined on the given Stage instance. ## Brief change log This PR mades the following changes: - Updates WithParams::set(...) to throw Exception if the given param is not defined on this instance. ## Verifying this change The change is tested by `StageTest::testSetUndefinedParam`. ## 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 changed class annotated with `@Public(Evolving)`: (yes) ## Documentation - Does this pull request introduce a new feature? (no) - If yes, how is the feature 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
