Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Compaction with timestamp: input boundaries #6645

Closed
wants to merge 6 commits into from
Closed

Compaction with timestamp: input boundaries #6645

wants to merge 6 commits into from

Commits on Apr 10, 2020

  1. Make compaction input boundary computation compatible with timestamp 

    Summary:
During compaction, timestamp should also be taken into accound when computing
overlapping ranges. If not, point lookup and range scan will return incorrect
results.

Test Plan:
make check
    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    ec90741 View commit details
    Browse the repository at this point in the history

  2. Fix assertion error after enabling auto compaction

    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    1ae73ac View commit details
    Browse the repository at this point in the history

  3. Add unit test

    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    a2c25f7 View commit details
    Browse the repository at this point in the history

  4. Fix lint

    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    26f3113 View commit details
    Browse the repository at this point in the history

  5. Update a unit test

    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    f1cbbfd View commit details
    Browse the repository at this point in the history

  6. Address review comments

    @riversand963
    riversand963 committed Apr 10, 2020
    Configuration menu
    Copy the full SHA
    7b0155e View commit details
    Browse the repository at this point in the history