Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-1049

SlicePredicate does not always round-trip correctly

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Fix Version/s: 0.6.2
    • Component/s: None
    • Labels:
      None
    1. SliceRangeSerializationTest.java
      2 kB
      Scott Fines
    2. 1049-test.txt
      3 kB
      Jonathan Ellis
    3. 1049-0.6.txt
      5 kB
      Jonathan Ellis
    4. 1049.txt
      5 kB
      Jonathan Ellis

      Activity

      Hide
      jbellis Jonathan Ellis added a comment -

      converted Mark Schnitzius's example from the ML to a failing unit test

      Show
      jbellis Jonathan Ellis added a comment - converted Mark Schnitzius's example from the ML to a failing unit test
      Hide
      jbellis Jonathan Ellis added a comment -

      Jeremy reports that he can reproduce using Thrift 0.3 rc, too.

      Show
      jbellis Jonathan Ellis added a comment - Jeremy reports that he can reproduce using Thrift 0.3 rc, too.
      Hide
      scottfines Scott Fines added a comment -

      I can confirm that this is also an issue with SliceRange itself, not just with SlicePredicate. I am attaching my UnitTest to this report as well(Though perhaps it should be set as a linked Issue instead).

      I'm a bit new to Cassandra, so please let me know if there is something amiss with this unittest.

      Show
      scottfines Scott Fines added a comment - I can confirm that this is also an issue with SliceRange itself, not just with SlicePredicate. I am attaching my UnitTest to this report as well(Though perhaps it should be set as a linked Issue instead). I'm a bit new to Cassandra, so please let me know if there is something amiss with this unittest.
      Hide
      jbellis Jonathan Ellis added a comment - - edited

      The bug is, thrift json serialization is broken.

      patch to switch to using binary serializer, converted to a hex string. inefficient but this is not a performance-sensitive method.

      Show
      jbellis Jonathan Ellis added a comment - - edited The bug is, thrift json serialization is broken. patch to switch to using binary serializer, converted to a hex string. inefficient but this is not a performance-sensitive method.
      Hide
      urandom Eric Evans added a comment -

      Can you rebase please?

      Show
      urandom Eric Evans added a comment - Can you rebase please?
      Hide
      jbellis Jonathan Ellis added a comment -

      rebased

      Show
      jbellis Jonathan Ellis added a comment - rebased
      Hide
      jbellis Jonathan Ellis added a comment -

      rebased to 0.6

      Show
      jbellis Jonathan Ellis added a comment - rebased to 0.6
      Hide
      jbellis Jonathan Ellis added a comment -

      0.6 patch w/ unit test

      Show
      jbellis Jonathan Ellis added a comment - 0.6 patch w/ unit test
      Hide
      urandom Eric Evans added a comment -

      +1

      Show
      urandom Eric Evans added a comment - +1
      Hide
      jbellis Jonathan Ellis added a comment -

      committed

      Show
      jbellis Jonathan Ellis added a comment - committed

        People

        • Assignee:
          jbellis Jonathan Ellis
          Reporter:
          jbellis Jonathan Ellis
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development