Cassandra
  1. Cassandra
  2. CASSANDRA-3558

Compression chunk_length_kb is not correctly returned for thrift/avro

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Fix Version/s: 1.0.6
    • Component/s: Core
    • Labels:

      Description

      CASSANDRA-3492 fixed the interpretation of chunk_length_kb as a size in bytes but infortunately forgot to convert it back to kb when returning it for thrift/avro. In particular, this means that a describe cf would return things like chunk_length_kb: 65535.

      I'm afraid that because migration uses Avro this is kind of a problem. One may have to issue an 'update column family' with the right chunk_length_kb to be sure to be in a safe place.

        Activity

        Sylvain Lebresne created issue -
        Hide
        Sylvain Lebresne added a comment -

        Fix attached. I've added a test for this in the distributed tests (at https://github.com/riptano/cassandra-dtest).

        We could have a unit test for this but without CASSANDRA-3559, it's more pain that it should be.

        Show
        Sylvain Lebresne added a comment - Fix attached. I've added a test for this in the distributed tests (at https://github.com/riptano/cassandra-dtest ). We could have a unit test for this but without CASSANDRA-3559 , it's more pain that it should be.
        Sylvain Lebresne made changes -
        Field Original Value New Value
        Attachment 0001-Correctly-handle-chunk_length_in_kb.patch [ 12505878 ]
        Sylvain Lebresne made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Jonathan Ellis made changes -
        Labels compression
        Reviewer xedin
        Hide
        Pavel Yaskevich added a comment -

        Committed.

        Show
        Pavel Yaskevich added a comment - Committed.
        Pavel Yaskevich made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12644318 ] patch-available, re-open possible [ 12749299 ]
        Gavin made changes -
        Workflow patch-available, re-open possible [ 12749299 ] reopen-resolved, no closed status, patch-avail, testing [ 12756918 ]

          People

          • Assignee:
            Sylvain Lebresne
            Reporter:
            Sylvain Lebresne
            Reviewer:
            Pavel Yaskevich
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development