Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-170162685
on your second point- I agree that Enum would be better for human readable
variable setting, but if one was doing a grid parameter search it would be
easier to search over a range of integers than having to establish an array of
specifically named variables. Not significantly harder, but still slightly more
tedious.
Further, since I'm just learning scala- limited research/understanding
leads me to believe 1) scala doesn't have an enum that works just like Java,
and 2) changing the cases to more informative string names would achieve the
desired goal?
In the mean time working on documentation and reverting the indentation
(which I thought I had already done), and thanks for bearing with me as I am
still learning.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
