Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Fix Version/s: 2.0 beta 1
    • Component/s: None
    • Labels:

      Description

      LazilyCompactedRow reads all data twice to compact a row which is obviously inefficient. The main reason we do that is to compute the row header. However, CASSANDRA-2319 have removed the main part of that row header. What remains is the size in bytes and the number of columns, but it should be relatively simple to remove those, which would then remove the need for the two-phase compaction.

      1. scrub-error.txt
        20 kB
        Jason Brown

        Issue Links

          Activity

          Aleksey Yeschenko made changes -
          Component/s Core [ 12312978 ]
          Matt Byrd made changes -
          Link This issue is related to CASSANDRA-7543 [ CASSANDRA-7543 ]
          Matt Byrd made changes -
          Link This issue is related to CASSANDRA-7543 [ CASSANDRA-7543 ]
          Jonathan Ellis made changes -
          Link This issue is duplicated by CASSANDRA-6918 [ CASSANDRA-6918 ]
          Jason Brown made changes -
          Link This issue incorporates CASSANDRA-5429 [ CASSANDRA-5429 ]
          Jonathan Ellis made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Gavin made changes -
          Link This issue is depended upon by CASSANDRA-5454 [ CASSANDRA-5454 ]
          Gavin made changes -
          Link This issue blocks CASSANDRA-5454 [ CASSANDRA-5454 ]
          Gavin made changes -
          Link This issue depends upon CASSANDRA-4885 [ CASSANDRA-4885 ]
          Gavin made changes -
          Link This issue depends on CASSANDRA-4885 [ CASSANDRA-4885 ]
          Jonathan Ellis made changes -
          Reviewer jasobrown krummas
          Sylvain Lebresne made changes -
          Link This issue blocks CASSANDRA-5454 [ CASSANDRA-5454 ]
          Jason Brown made changes -
          Attachment scrub-error.txt [ 12576521 ]
          Jonathan Ellis made changes -
          Assignee Jason Brown [ jasobrown ] Jonathan Ellis [ jbellis ]
          Reviewer yukim jasobrown
          Jonathan Ellis made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Jonathan Ellis made changes -
          Summary Single-pass compaction Single-pass compaction for LCR
          Jonathan Ellis made changes -
          Summary Remove 2-phase compaction Single-pass compaction
          Gavin made changes -
          Workflow patch-available, re-open possible [ 12753023 ] reopen-resolved, no closed status, patch-avail, testing [ 12755688 ]
          Gavin made changes -
          Workflow no-reopen-closed, patch-avail [ 12663314 ] patch-available, re-open possible [ 12753023 ]
          Jonathan Ellis made changes -
          Reviewer slebresne yukim
          Jonathan Ellis made changes -
          Assignee Yuki Morishita [ yukim ] Jason Brown [ jasobrown ]
          Jonathan Ellis made changes -
          Link This issue depends on CASSANDRA-4885 [ CASSANDRA-4885 ]
          Jonathan Ellis made changes -
          Fix Version/s 1.3 [ 12322954 ]
          Fix Version/s 1.2.0 [ 12319262 ]
          Jonathan Ellis made changes -
          Field Original Value New Value
          Labels compaction
          Assignee Sylvain Lebresne [ slebresne ] Yuki Morishita [ yukim ]
          Reviewer slebresne
          Sylvain Lebresne created issue -

            People

            • Assignee:
              Jonathan Ellis
              Reporter:
              Sylvain Lebresne
              Reviewer:
              Marcus Eriksson
            • Votes:
              1 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development