Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-6313

Some words was spelled wrong and incorrect LOG.error without print

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.3.0
    • Fix Version/s: 1.3.0
    • Component/s: Queryable State
    • Labels:
      None

      Description

      I find some words are spelled wrong and log.error without print information.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user zhangminglei opened a pull request:

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

          FLINK-6313 [flink-runtime] Fix Some words was spelled wrong and inc…

          …orrect LOG.error without print.

          Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
          If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the [How To Contribute guide](http://flink.apache.org/how-to-contribute.html).
          In addition to going through the list, please provide a meaningful description of your changes.

          • [ ] General
          • The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
          • The pull request addresses only one issue
          • Each commit in the PR has a meaningful commit message (including the JIRA id)
          • [ ] Documentation
          • Documentation has been added for new functionality
          • Old documentation affected by the pull request has been updated
          • JavaDoc for public methods has been added
          • [ ] Tests & Build
          • Functionality added by the pull request is covered by tests
          • `mvn clean verify` has been executed successfully locally or a Travis build has passed

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

          $ git pull https://github.com/zhangminglei/flink 6313

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

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


          commit a7c07ce5acac38c544045ff40c189d6973c91a6e
          Author: zhangminglei <zml13856086071@163.com>
          Date: 2017-04-17T07:49:54Z

          FLINK-6313 [flink-runtime] Fix Some words was spelled wrong and incorrect LOG.error without print.


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user zhangminglei opened a pull request: https://github.com/apache/flink/pull/3728 FLINK-6313 [flink-runtime] Fix Some words was spelled wrong and inc… …orrect LOG.error without print. Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration. If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the [How To Contribute guide] ( http://flink.apache.org/how-to-contribute.html ). In addition to going through the list, please provide a meaningful description of your changes. [ ] General The pull request references the related JIRA issue (" [FLINK-XXX] Jira title text") The pull request addresses only one issue Each commit in the PR has a meaningful commit message (including the JIRA id) [ ] Documentation Documentation has been added for new functionality Old documentation affected by the pull request has been updated JavaDoc for public methods has been added [ ] Tests & Build Functionality added by the pull request is covered by tests `mvn clean verify` has been executed successfully locally or a Travis build has passed You can merge this pull request into a Git repository by running: $ git pull https://github.com/zhangminglei/flink 6313 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3728.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 #3728 commit a7c07ce5acac38c544045ff40c189d6973c91a6e Author: zhangminglei <zml13856086071@163.com> Date: 2017-04-17T07:49:54Z FLINK-6313 [flink-runtime] Fix Some words was spelled wrong and incorrect LOG.error without print.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zhangminglei commented on the issue:

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

          @tzulitai Hi, Could you please review those code ? Thanks and appreciate ~

          Show
          githubbot ASF GitHub Bot added a comment - Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/3728 @tzulitai Hi, Could you please review those code ? Thanks and appreciate ~
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

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

          merging.

          Show
          githubbot ASF GitHub Bot added a comment - Github user zentol commented on the issue: https://github.com/apache/flink/pull/3728 merging.
          Hide
          Zentol Chesnay Schepler added a comment -

          1.3: 33abc25f0f7f15433a81147cbaf815d87808486f

          Show
          Zentol Chesnay Schepler added a comment - 1.3: 33abc25f0f7f15433a81147cbaf815d87808486f
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

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

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3728

            People

            • Assignee:
              mingleizhang mingleizhang
              Reporter:
              mingleizhang mingleizhang
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development