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

            links to

            Web Link GitHub Pull Request #4389

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              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
              Permalink
              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

                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