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

description of datastream rescaling doesn't match the figure

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.2.0, 1.1.4
    • Component/s: Documentation
    • Labels:
      None

      Description

      In https://ci.apache.org/projects/flink/flink-docs-release-1.2/dev/datastream_api.html#physical-partitioning the figure shows a parallelism of 2 rescaling to 6 and back to 2 – while the text assumes 4, rather than 6. This makes the explanation very confusing.

        Issue Links

          Activity

          Hide
          fhueske Fabian Hueske added a comment -

          Fixed in 1.1.4 with c567b0c285a848e89cef2553a78e790a6cc20804
          Fixed in 1.2.0 with 96239b3056fa437f450e414d94e61157b77f7e07

          Show
          fhueske Fabian Hueske added a comment - Fixed in 1.1.4 with c567b0c285a848e89cef2553a78e790a6cc20804 Fixed in 1.2.0 with 96239b3056fa437f450e414d94e61157b77f7e07
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

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

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

          Github user fhueske commented on the issue:

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

          merging

          Show
          githubbot ASF GitHub Bot added a comment - Github user fhueske commented on the issue: https://github.com/apache/flink/pull/2932 merging
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user fhueske commented on the issue:

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

          +1 to merge

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

          GitHub user alpinegizmo opened a pull request:

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

          FLINK-5249 [docs] description of datastream rescaling doesn't match…

          In docs/dev/datastream_api.html#physical-partitioning the figure shows a parallelism of 2 rescaling to 6 and back to 2 – while the text assumes 4, rather than 6. This makes the explanation very confusing.

          See the figure at the end of [this section](https://ci.apache.org/projects/flink/flink-docs-release-1.2/dev/datastream_api.html#physical-partitioning).

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

          $ git pull https://github.com/alpinegizmo/flink rescaling-description

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

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


          commit 7e448955180b0525bf1324c3fd4296dbc2a9a71e
          Author: David Anderson <david@alpinegizmo.com>
          Date: 2016-12-04T14:32:23Z

          FLINK-5249 [docs] description of datastream rescaling doesn't match the figure


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user alpinegizmo opened a pull request: https://github.com/apache/flink/pull/2932 FLINK-5249 [docs] description of datastream rescaling doesn't match… In docs/dev/datastream_api.html#physical-partitioning the figure shows a parallelism of 2 rescaling to 6 and back to 2 – while the text assumes 4, rather than 6. This makes the explanation very confusing. See the figure at the end of [this section] ( https://ci.apache.org/projects/flink/flink-docs-release-1.2/dev/datastream_api.html#physical-partitioning ). You can merge this pull request into a Git repository by running: $ git pull https://github.com/alpinegizmo/flink rescaling-description Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/2932.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 #2932 commit 7e448955180b0525bf1324c3fd4296dbc2a9a71e Author: David Anderson <david@alpinegizmo.com> Date: 2016-12-04T14:32:23Z FLINK-5249 [docs] description of datastream rescaling doesn't match the figure

            People

            • Assignee:
              alpinegizmo David Anderson
              Reporter:
              alpinegizmo David Anderson
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development