[GitHub] flink pull request: [FLINK-2103] Expose partitionBy to user

2015-05-28 Thread aljoscha
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

2015-06-02 Thread mbalassi
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

2015-06-02 Thread aljoscha
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

2015-06-02 Thread mbalassi
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

2015-06-02 Thread mbalassi
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

2015-06-03 Thread asfgit
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.
---