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

Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.1.3
    • Fix Version/s: 1.3.0
    • Component/s: DataStream API
    • Labels:

      Description

      StreamingFunctionUtils.java is in 'org.apache.flink.streaming.api.util'

      And there is package 'org.apache.flink.streaming.util' in project.

      I think move StreamingFunctionUtils.java to 'org.apache.flink.streaming.api.util' is make code more clear.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

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

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

          Fixed via 03889ae1f25bab0fc42ae695bdf45cf45658eab6

          Show
          StephanEwen Stephan Ewen added a comment - Fixed via 03889ae1f25bab0fc42ae695bdf45cf45658eab6
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user StephanEwen commented on the issue:

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

          Agreed, the utilities are not part of the API, so they should not be in a subdirectory of the API.

          Merging this...

          Show
          githubbot ASF GitHub Bot added a comment - Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/3532 Agreed, the utilities are not part of the API, so they should not be in a subdirectory of the API. Merging this...
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user liuyuzhong7 opened a pull request:

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

          FLINK-6041[flink-streaming-java] Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'

          StreamingFunctionUtils.java is in 'org.apache.flink.streaming.api.util'
          And there is package 'org.apache.flink.streaming.util' in project.
          I think move StreamingFunctionUtils.java to 'org.apache.flink.streaming.api.util' is make code more clear.

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

          $ git pull https://github.com/liuyuzhong7/flink FLINK-6041

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

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


          commit 2ec1f0edaf2c665aece4bda024c8ee301740e760
          Author: liuyuzhong7 <liuyuzhong7@gmail.com>
          Date: 2017-03-14T12:23:25Z

          FLINK-6041 Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user liuyuzhong7 opened a pull request: https://github.com/apache/flink/pull/3532 FLINK-6041 [flink-streaming-java] Move StreamingFunctionUtils to 'org.apache.flink.streaming.util' StreamingFunctionUtils.java is in 'org.apache.flink.streaming.api.util' And there is package 'org.apache.flink.streaming.util' in project. I think move StreamingFunctionUtils.java to 'org.apache.flink.streaming.api.util' is make code more clear. You can merge this pull request into a Git repository by running: $ git pull https://github.com/liuyuzhong7/flink FLINK-6041 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3532.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 #3532 commit 2ec1f0edaf2c665aece4bda024c8ee301740e760 Author: liuyuzhong7 <liuyuzhong7@gmail.com> Date: 2017-03-14T12:23:25Z FLINK-6041 Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'

            People

            • Assignee:
              Unassigned
              Reporter:
              liuyuzhong7 liuyuzhong7
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development