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

Deprecate DataSetUtils#checksumHashCode

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Implemented
    • Affects Version/s: 1.3.0
    • Fix Version/s: 1.3.0
    • Component/s: Java API
    • Labels:
      None

      Description

      This is likely only used by Gelly and we have a more featureful implementation allowing for multiple outputs and setting the job name. Deprecation will allow this to be removed in Flink 2.0.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user greghogan opened a pull request:

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

          FLINK-6009 [java api] Deprecate DataSetUtils#checksumHashCode

          This is likely only used by Gelly and we have a more featureful implementation allowing for multiple outputs and setting the job name. Deprecation will allow this to be removed in Flink 2.0.

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

          $ git pull https://github.com/greghogan/flink 6009_deprecate_datasetutils_checksumhashcode

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

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


          commit dfdf7cbb283f87bac5ff7dd13b0a44db4d953d8b
          Author: Greg Hogan <code@greghogan.com>
          Date: 2017-03-09T20:19:35Z

          FLINK-6009 [java api] Deprecate DataSetUtils#checksumHashCode

          This is likely only used by Gelly and we have a more featureful
          implementation allowing for multiple outputs and setting the job name.
          Deprecation will allow this to be removed in Flink 2.0.


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user greghogan opened a pull request: https://github.com/apache/flink/pull/3516 FLINK-6009 [java api] Deprecate DataSetUtils#checksumHashCode This is likely only used by Gelly and we have a more featureful implementation allowing for multiple outputs and setting the job name. Deprecation will allow this to be removed in Flink 2.0. You can merge this pull request into a Git repository by running: $ git pull https://github.com/greghogan/flink 6009_deprecate_datasetutils_checksumhashcode Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3516.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 #3516 commit dfdf7cbb283f87bac5ff7dd13b0a44db4d953d8b Author: Greg Hogan <code@greghogan.com> Date: 2017-03-09T20:19:35Z FLINK-6009 [java api] Deprecate DataSetUtils#checksumHashCode This is likely only used by Gelly and we have a more featureful implementation allowing for multiple outputs and setting the job name. Deprecation will allow this to be removed in Flink 2.0.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

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

          Looks good to me. +1

          Show
          githubbot ASF GitHub Bot added a comment - Github user zentol commented on the issue: https://github.com/apache/flink/pull/3516 Looks good to me. +1
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rmetzger commented on the issue:

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

          +1

          Show
          githubbot ASF GitHub Bot added a comment - Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/3516 +1
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user StephanEwen commented on the issue:

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

          Looks good, merging...

          Show
          githubbot ASF GitHub Bot added a comment - Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/3516 Looks good, merging...
          Hide
          StephanEwen Stephan Ewen added a comment -

          Fixed via 40a156e4974c942b021d20edb78b95c8bf261972

          Show
          StephanEwen Stephan Ewen added a comment - Fixed via 40a156e4974c942b021d20edb78b95c8bf261972
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

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

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

            People

            • Assignee:
              greghogan Greg Hogan
              Reporter:
              greghogan Greg Hogan
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development