Cassandra
  1. Cassandra
  2. CASSANDRA-3803

snapshot-before-compaction snapshots entire keyspace

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Fix Version/s: 1.0.8, 1.1.0
    • Component/s: Core
    • Labels:

      Description

      Should only snapshot the CF being compacted

      1. 3803.txt
        2 kB
        Jonathan Ellis

        Activity

        Jonathan Ellis created issue -
        Hide
        Jonathan Ellis added a comment -

        I can't think of a good reason to snapshot the entire keyspace, and doing so can dramatically increase the space needed to enable snapshot_before_compaction.

        Show
        Jonathan Ellis added a comment - I can't think of a good reason to snapshot the entire keyspace, and doing so can dramatically increase the space needed to enable snapshot_before_compaction.
        Jonathan Ellis made changes -
        Field Original Value New Value
        Attachment 3803.txt [ 12512239 ]
        Jonathan Ellis made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Sylvain Lebresne added a comment -

        I think the attached patch is the wrong one.

        Show
        Sylvain Lebresne added a comment - I think the attached patch is the wrong one.
        Jonathan Ellis made changes -
        Attachment 3803.txt [ 12512239 ]
        Hide
        Jonathan Ellis added a comment -

        2nd try. Also switches to snapshot-without-flush.

        Show
        Jonathan Ellis added a comment - 2nd try. Also switches to snapshot-without-flush.
        Jonathan Ellis made changes -
        Attachment 3803.txt [ 12513362 ]
        Hide
        Sylvain Lebresne added a comment -

        +1, but I would actually suggest pushing this in 1.0.8. The fact we were flushing all CFs on each compaction is pretty bad (not sure anyone actually uses snapshot-before-compaction but still).

        Show
        Sylvain Lebresne added a comment - +1, but I would actually suggest pushing this in 1.0.8. The fact we were flushing all CFs on each compaction is pretty bad (not sure anyone actually uses snapshot-before-compaction but still).
        Hide
        Sylvain Lebresne added a comment -

        As a side not, a nice alternative would be to snapshot only the files we're going to compact. But again, we probably don't care about that feature that much.

        Show
        Sylvain Lebresne added a comment - As a side not, a nice alternative would be to snapshot only the files we're going to compact. But again, we probably don't care about that feature that much.
        Hide
        Jonathan Ellis added a comment - - edited

        committed to 1.0.8 + 1.1

        Show
        Jonathan Ellis added a comment - - edited committed to 1.0.8 + 1.1
        Jonathan Ellis made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s 1.0.9 [ 12319856 ]
        Resolution Fixed [ 1 ]
        Jonathan Ellis made changes -
        Fix Version/s 1.0.8 [ 12319453 ]
        Fix Version/s 1.0.9 [ 12319856 ]
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12650918 ] patch-available, re-open possible [ 12748977 ]
        Gavin made changes -
        Workflow patch-available, re-open possible [ 12748977 ] reopen-resolved, no closed status, patch-avail, testing [ 12753982 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        13m 45s 1 Jonathan Ellis 27/Jan/12 22:18
        Patch Available Patch Available Resolved Resolved
        12d 5h 38m 1 Jonathan Ellis 09/Feb/12 03:56

          People

          • Assignee:
            Jonathan Ellis
            Reporter:
            Jonathan Ellis
            Reviewer:
            Sylvain Lebresne
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development