You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by twalthr <gi...@git.apache.org> on 2014/11/25 11:53:45 UTC
[GitHub] incubator-flink pull request: Small changes and tests for the newl...
GitHub user twalthr opened a pull request:
https://github.com/apache/incubator-flink/pull/230
Small changes and tests for the newly introduced EnumType
Small changes and tests for the newly introduced EnumType. I will also adapt the TypeInfoParser, if #203 is merged.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/twalthr/incubator-flink EnumTypeImprovement
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-flink/pull/230.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 #230
----
commit a375d514c1d67e6af1962c15719ad1beb168b9d0
Author: twalthr <in...@twalthr.com>
Date: 2014-11-25T10:51:10Z
Small changes and tests for the newly introduced EnumType
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-flink pull request: Small changes and tests for the newl...
Posted by twalthr <gi...@git.apache.org>.
Github user twalthr commented on the pull request:
https://github.com/apache/incubator-flink/pull/230#issuecomment-64386309
yes, its indepent of the other PR
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-flink pull request: Small changes and tests for the newl...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-flink/pull/230
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-flink pull request: Small changes and tests for the newl...
Posted by uce <gi...@git.apache.org>.
Github user uce commented on the pull request:
https://github.com/apache/incubator-flink/pull/230#issuecomment-64431488
Looks good to me. Will merge today.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-flink pull request: Small changes and tests for the newl...
Posted by rmetzger <gi...@git.apache.org>.
Github user rmetzger commented on the pull request:
https://github.com/apache/incubator-flink/pull/230#issuecomment-64384985
Looks good to merge.
Can we merge it before the other pull request?
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---