Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4255
you're right - wasn't looking thoroughly enough (and only recently learned
about `ExpectedException`).
Additionally, I found out that this ignored exception was actually required
to be thrown for some of these tests while others didn't even throw it. So with
the newest changes (also rebased on top of the current `master`), the tests
actually better reflect the intended behaviour.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
