[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
GitHub user aljoscha opened a pull request: https://github.com/apache/flink/pull/743 [FLINK-2103] Expose partitionBy to user You can merge this pull request into a Git repository by running: $ git pull https://github.com/aljoscha/flink user-partition-by Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/743.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 #743 commit b5ba40739008e988905e0b9d58d0a6e2cc37e73f Author: Aljoscha Krettek Date: 2015-05-28T08:40:56Z [FLINK-2103] Expose partitionBy to user --- 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. ---
[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/743#issuecomment-107853134 Merging. --- 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. ---
[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/743#issuecomment-107853775 We should first update the documentation. --- 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. ---
[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/743#issuecomment-107854661 Good point, will do that anyway because of the renaming discussed on the mailing list. --- 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. ---
[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/743#issuecomment-107875574 While merging realized, that it is missing from the windowed and connected datastreams and the from the scala API. Also adding those. --- 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. ---
[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/743 --- 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. ---