Github user indrc commented on the issue:
https://github.com/apache/flink/pull/6302
@StephanEwen Thanks for your comments and feedback, my replies below
1. after talking to @StefanRRichter we decided to bring in this dependency
to use a standard library instead; initially the pr had an impl of random
generator function. Are you suggesting to revert to the old?
(And due to a squash, i lost the review comments; sorry about losing the
context)
2. the feature gets enabled through a new config key; the specific
constructor helps initializing the new feature. My thought was this way, it
doesn't get exposed to default codepath. Do you see a problem with this
approach?
3. sounds good, i will remove that warning.
---
