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

Remove unnecessary setTotalOrderForSeek for Rocks iterator

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    Description

      From FLINK-17800, we have to explicitly add setTotalOrderForSeek to ensure data correctness if user choose to set OptimizeForPointLookup.

      However, this is unnecessary since we upgraded the RocksDB version (please refer to rocksDB PR) which removes the prefix extractor and hash-based indexing.

      Attachments

        Issue Links

        depends upon

        Improvement - An improvement or enhancement to an existing feature or task. FLINK-14482 Bump up rocksdb version

        • Critical - Crashes, loss of data, severe memory leak.
        • Closed
        relates to

        Bug - A problem which impairs or prevents the functions of the product. FLINK-17800 RocksDB optimizeForPointLookup results in missing time windows

        • Critical - Crashes, loss of data, severe memory leak.
        • Closed
        links to

        Web Link GitHub Pull Request #16834

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            yunta Yun Tang
            yunta Yun Tang
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment