GitHub user zentol opened a pull request:
https://github.com/apache/flink/pull/1044
[FLINK-2556] Refactor/Fix pre-flight Key validation
Removed redundant key validation in DistinctOperator
Keys constructors now make sure the type of every key is an instance of
AtomicType/CompositeType, and that type.isKeyType() is true.
Additionally, the ExpressionKeys int[] constructor explicitly rejects Tuple0
Changes one test that actually tried something that shouldn't work in the
first place.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zentol/flink isKeyType_check
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1044.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 #1044
----
commit 7a57b6ef2ecdc7adaf770f8585cf8f974c684705
Author: zentol <s.mo...@web.de>
Date: 2015-08-23T13:57:34Z
[FLINK-2556] Refactor/Fix pre-flight Key validation
----
---
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.
---
