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

CheckpointNotifier should be removed from docs

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.1.3
    • Fix Version/s: 1.2.0, 1.3.0
    • Component/s: Documentation
    • Labels:
      None

      Description

      I was following the official documentation: https://ci.apache.org/projects/flink/flink-docs-release-1.1/apis/streaming/state.html

      Looks like this is the right one to be using: import org.apache.flink.runtime.state.CheckpointListener;

      Abhishek

      On Dec 9, 2016, at 4:30 PM, Abhishek R. Singh <abhishsi@tetrationanalytics.com> wrote:

      I can’t seem to find CheckpointNotifier. Appreciate help !

      CheckpointNotifier is not a member of package org.apache.flink.streaming.api.checkpoint

      From my pom.xml:

      <dependency>
      <groupId>org.apache.flink</groupId>
      <artifactId>flink-scala_2.11</artifactId>
      <version>1.1.3</version>
      </dependency>
      <dependency>
      <groupId>org.apache.flink</groupId>
      <artifactId>flink-streaming-scala_2.11</artifactId>
      <version>1.1.3</version>
      </dependency>
      <dependency>
      <groupId>org.apache.flink</groupId>
      <artifactId>flink-clients_2.11</artifactId>
      <version>1.1.3</version>
      </dependency>
      <dependency>
      <groupId>org.apache.flink</groupId>
      <artifactId>flink-statebackend-rocksdb_2.11</artifactId>
      <version>1.1.3</version>
      </dependency>

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user abhishsi opened a pull request:

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

          FLINK-5323 CheckpointNotifier should be removed from docs

          Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
          If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the [How To Contribute guide](http://flink.apache.org/how-to-contribute.html).
          In addition to going through the list, please provide a meaningful description of your changes.

          • [ ] General
          • The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
          • The pull request addresses only one issue
          • Each commit in the PR has a meaningful commit message (including the JIRA id)
          • [ ] Documentation
          • Documentation has been added for new functionality
          • Old documentation affected by the pull request has been updated
          • JavaDoc for public methods has been added
          • [ ] Tests & Build
          • Functionality added by the pull request is covered by tests
          • `mvn clean verify` has been executed successfully locally or a Travis build has passed

          The documentation was stale

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

          $ git pull https://github.com/abhishsi/flink master

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

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


          commit 4fb31251e681d7975bc1de3b1e02f1500f05e09c
          Author: Abhishek R. Singh <abhishsi@tetrationanalytics.com>
          Date: 2016-12-14T14:05:11Z

          FLINK-5323 CheckpointNotifier should be removed from docs

          The documentation was stale


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user abhishsi opened a pull request: https://github.com/apache/flink/pull/3006 FLINK-5323 CheckpointNotifier should be removed from docs Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration. If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the [How To Contribute guide] ( http://flink.apache.org/how-to-contribute.html ). In addition to going through the list, please provide a meaningful description of your changes. [ ] General The pull request references the related JIRA issue (" [FLINK-XXX] Jira title text") The pull request addresses only one issue Each commit in the PR has a meaningful commit message (including the JIRA id) [ ] Documentation Documentation has been added for new functionality Old documentation affected by the pull request has been updated JavaDoc for public methods has been added [ ] Tests & Build Functionality added by the pull request is covered by tests `mvn clean verify` has been executed successfully locally or a Travis build has passed The documentation was stale You can merge this pull request into a Git repository by running: $ git pull https://github.com/abhishsi/flink master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3006.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 #3006 commit 4fb31251e681d7975bc1de3b1e02f1500f05e09c Author: Abhishek R. Singh <abhishsi@tetrationanalytics.com> Date: 2016-12-14T14:05:11Z FLINK-5323 CheckpointNotifier should be removed from docs The documentation was stale
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rmetzger commented on the issue:

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

          +1 to merge

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

          Github user abhishsi commented on the issue:

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

          retest this please

          Show
          githubbot ASF GitHub Bot added a comment - Github user abhishsi commented on the issue: https://github.com/apache/flink/pull/3006 retest this please
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user abhishsi commented on the issue:

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

          @rmetzger - not sure why CI failed. How do I rerun it?

          Show
          githubbot ASF GitHub Bot added a comment - Github user abhishsi commented on the issue: https://github.com/apache/flink/pull/3006 @rmetzger - not sure why CI failed. How do I rerun it?
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

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

          merging.

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

          Github user tzulitai commented on the issue:

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

          @zentol This hasn't been merged yet, has it?

          Show
          githubbot ASF GitHub Bot added a comment - Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3006 @zentol This hasn't been merged yet, has it?
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

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

          @tzulitai No it wasn't merged yet, but will do so in ~30 minutes.

          Show
          githubbot ASF GitHub Bot added a comment - Github user zentol commented on the issue: https://github.com/apache/flink/pull/3006 @tzulitai No it wasn't merged yet, but will do so in ~30 minutes.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

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

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

          1.2: bb46fffe310f9cd6f667293df14d98e90011d591
          1.3: 4415dfbd5753196c09940b59ad39de9f4e9402b9

          Show
          Zentol Chesnay Schepler added a comment - 1.2: bb46fffe310f9cd6f667293df14d98e90011d591 1.3: 4415dfbd5753196c09940b59ad39de9f4e9402b9

            People

            • Assignee:
              Unassigned
              Reporter:
              abhishekrs Abhishek Singh
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development