Github user attilapiros commented on the issue:
https://github.com/apache/spark/pull/19599
As I see inside the StringParam the options even can be mapped to enum
values. This way ignoring cases at the matching part would be no problem at all
(and it would be very efficient). Going further this could be a type safe
construct by making StringParam a generic where the generic parameter is the
enum type used as the target of the mapping: like
StringParam[NaiveBayesModelType].
What is your opinion?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
