[GitHub] flink pull request: [FLINK-1816] Fixes wrongly introduced exceptio...

2015-04-02 Thread tillrohrmann
GitHub user tillrohrmann opened a pull request:

https://github.com/apache/flink/pull/562

[FLINK-1816] Fixes wrongly introduced exception checks in 
DegreesWithExceptionITCase

I wrongly introduced exception type checks in the 
```DegreesWithExceptionITCase``` which are corrected with this PR. See #475 for 
more details.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tillrohrmann/flink 
fixDegreesWithExceptionITCase

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/562.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 #562


commit 10d20bff289ea5f27e4c04b49a4dcb349fbf6ce6
Author: Till Rohrmann trohrm...@apache.org
Date:   2015-04-02T09:28:38Z

[FLINK-1816] [gelly tests] Fixes wrongly introduced exception checks in 
DegreesWithExceptionITCase




---
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-1816] Fixes wrongly introduced exceptio...

2015-04-02 Thread vasia
Github user vasia commented on the pull request:

https://github.com/apache/flink/pull/562#issuecomment-88851563
  
wow that was quick :)) Thanks a lot @tillrohrmann!


---
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-1816] Fixes wrongly introduced exceptio...

2015-04-02 Thread uce
Github user uce commented on the pull request:

https://github.com/apache/flink/pull/562#issuecomment-88850486
  
Good to merge. This was probably just a mistake during a merge. ;)


---
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-1816] Fixes wrongly introduced exceptio...

2015-04-02 Thread tillrohrmann
Github user tillrohrmann commented on the pull request:

https://github.com/apache/flink/pull/562#issuecomment-88919364
  
I'll merge it now.


---
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-1816] Fixes wrongly introduced exceptio...

2015-04-02 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/flink/pull/562


---
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.
---