GitHub user ChengXiangLi opened a pull request:
https://github.com/apache/flink/pull/1047
[FLINK-2564] Failing Test: RandomSamplerTest
As discussed in
[here](https://github.com/apache/flink/pull/949#issuecomment-133990256), it's a
balance between accurate verification and failure-positive result. As this
failure and KS test failure happens more often than expected, we should expand
the verification boundary which would reduce the fail-positive case to an
acceptable level.
With this PR, the KS test and fraction verification never fail after tens
of thousands executions in my local environment. @tillrohrmann , @chiwanpark .
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ChengXiangLi/flink FLINK-2564
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1047.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1047
----
commit ab5c81d708c31591df160b323244df7fe4445dc6
Author: chengxiang li <[email protected]>
Date: 2015-08-24T09:49:44Z
[FLINK-2564] [test] expand the verification boundary for random sampler
test.
----
---
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.
---
