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

ListStateDescriptor example uses wrong constructor

    Details

      Description

      The Working with state docs contain an example for using a ListStateDescriptor.
      In the example however a default value is passed to the constructor, which however is only possible for ValueStateDescriptors.

        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/4389

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

          1.3: 0c9bee69665e00ab17fe0a216c746b0bc8435263
          1.4: b139147998d3619f30e4ca260531e8cbeab3b38b

          Show
          Zentol Chesnay Schepler added a comment - 1.3: 0c9bee69665e00ab17fe0a216c746b0bc8435263 1.4: b139147998d3619f30e4ca260531e8cbeab3b38b
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

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

          merging.

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

          Github user tzulitai commented on the issue:

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

          Nice catch! +1, LGTM.

          Show
          githubbot ASF GitHub Bot added a comment - Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/4389 Nice catch! +1, LGTM.
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user zentol opened a pull request:

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

          FLINK-7255 [docs] Remove default value from ListStateDescriptor con…

          This PR corrects the docs regarding the ListStateDescriptor; it no longer passes a default value to the constructor.

          This applies to 1.3 and master.

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

          $ git pull https://github.com/zentol/flink 7255

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

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


          commit ec4dd10fb2decb86345c4d646cb9c6eeb222cb57
          Author: zentol <chesnay@apache.org>
          Date: 2017-07-24T13:04:49Z

          FLINK-7255 [docs] Remove default value from ListStateDescriptor constructor


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/4389 FLINK-7255 [docs] Remove default value from ListStateDescriptor con… This PR corrects the docs regarding the ListStateDescriptor; it no longer passes a default value to the constructor. This applies to 1.3 and master. You can merge this pull request into a Git repository by running: $ git pull https://github.com/zentol/flink 7255 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/4389.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 #4389 commit ec4dd10fb2decb86345c4d646cb9c6eeb222cb57 Author: zentol <chesnay@apache.org> Date: 2017-07-24T13:04:49Z FLINK-7255 [docs] Remove default value from ListStateDescriptor constructor

            People

            • Assignee:
              Zentol Chesnay Schepler
              Reporter:
              Zentol Chesnay Schepler
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development