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

HBASE-24850 CellComparator perf improvement #2802

Merged
merged 12 commits into from
Dec 28, 2020
Merged

HBASE-24850 CellComparator perf improvement #2802

merged 12 commits into from
Dec 28, 2020

Commits on Dec 22, 2020

  1. Using ContiguousCellFormat as a marker alone

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    297e898 View commit details
    Browse the repository at this point in the history

  2. Commit the new file

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    cafe8af View commit details
    Browse the repository at this point in the history

  3. Fix the comparator logic that was an oversight

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    849faf1 View commit details
    Browse the repository at this point in the history

  4. Fix the sequenceId check order

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    82272a4 View commit details
    Browse the repository at this point in the history

  5. Adding few more static methods that helps in scan flow like query 

    matcher where we have more cols
    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    9884df3 View commit details
    Browse the repository at this point in the history

  6. Remove ContiguousCellFormat and ensure compare() can be inlined

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    77074b4 View commit details
    Browse the repository at this point in the history

  7. applying negation as per review comment

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    beabe69 View commit details
    Browse the repository at this point in the history

  8. Fix checkstyle comments

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    34b6e55 View commit details
    Browse the repository at this point in the history

  9. fix review comments

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    96fb77a View commit details
    Browse the repository at this point in the history

  10. Address review comments

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    b8f67db View commit details
    Browse the repository at this point in the history

  11. Fix the checkstyle issues

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    11015ba View commit details
    Browse the repository at this point in the history

  12. Fix javadoc

    @ramkrish86
    ramkrish86 committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    572169b View commit details
    Browse the repository at this point in the history