You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by mjsax <gi...@git.apache.org> on 2015/02/24 19:43:26 UTC

[GitHub] flink pull request: fixed package statement

GitHub user mjsax opened a pull request:

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

    fixed package statement

    package statement was not updated after class movement
    also added a missing import

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

    $ git pull https://github.com/mjsax/flink bug_fix_package_statement

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

    https://github.com/apache/flink/pull/437.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 #437
    
----
commit 1470b0f4af86d457d841afc037bd9583cdb9d1c1
Author: mjsax <mj...@informatik.hu-berlin.de>
Date:   2015-02-24T18:39:27Z

    fixed package statement

----


---
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] flink pull request: fixed package statement

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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] flink pull request: fixed package statement

Posted by hsaputra <gi...@git.apache.org>.
Github user hsaputra commented on the pull request:

    https://github.com/apache/flink/pull/437#issuecomment-75835795
  
    Once build finished if successful will commit this if no one else has comment.
    
    CC @rmetzger, @aljoscha 


---
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] flink pull request: fixed package statement

Posted by hsaputra <gi...@git.apache.org>.
Github user hsaputra commented on the pull request:

    https://github.com/apache/flink/pull/437#issuecomment-75822423
  
    +1 LGTM
    
    How can Java compiler miss this I am confused =(
    My IntelliJ IDEA has a red wiggly line indicating mismatch package name.


---
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] flink pull request: fixed package statement

Posted by rmetzger <gi...@git.apache.org>.
Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/437#issuecomment-75836121
  
    Sounds good. Thank you Henry.


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