Details

    • Type: Sub-task Sub-task
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Client
    • Labels:
      None

      Description

      This ticket encapsulates a micro-benchmark of serialization implementation and any performance enhancements that fall out. Specifically, the implementation posted in HBASE-8201 should be compared to the performance of Phoenix's PDataType implementations.

        Activity

        Hide
        Nick Dimiduk added a comment -

        Here's the output of an initial benchmark suite over Long and Double encodings. This is vs. code from yesterday; it does not include today's refactor to ByteRange or the introduction of Guava Precondition checking. Benchmark code is in a repository on github and include basic instructions for reproduction.

        Preliminary results indicate Phoenix's DECIMAL is ~2x faster and uses ~3x fewer heap objects than my Numeric implementation. Nice job James Taylor! I'll look into cribbing off of your implementation tomorrow In the mean time, have a look at the implementations and make sure I'm doing the other encoding libraries justice.

        Show
        Nick Dimiduk added a comment - Here's the output of an initial benchmark suite over Long and Double encodings. This is vs. code from yesterday; it does not include today's refactor to ByteRange or the introduction of Guava Precondition checking. Benchmark code is in a repository on github and include basic instructions for reproduction. Preliminary results indicate Phoenix's DECIMAL is ~2x faster and uses ~3x fewer heap objects than my Numeric implementation. Nice job James Taylor ! I'll look into cribbing off of your implementation tomorrow In the mean time, have a look at the implementations and make sure I'm doing the other encoding libraries justice.
        Hide
        stack added a comment -

        Moving this improvement out of 0.95/0.96. Pull back if I have it wrong.

        Show
        stack added a comment - Moving this improvement out of 0.95/0.96. Pull back if I have it wrong.

          People

          • Assignee:
            Nick Dimiduk
            Reporter:
            Nick Dimiduk
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:

              Development