Cassandra
  1. Cassandra
  2. CASSANDRA-2511

Need to forward merge parts of r1088800 to make the pig CassandraStorage build

    Details

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

      Description

      Parts of revision 1088800 weren't forward merged into 0.8/trunk. So the 0.8/trunk version of CassandraStorage doesn't currently build. Specifically, it needs the decompose method in the AbstractType hierarchy.

      1. 2511-newer-types.txt
        2 kB
        Jeremy Hanna
      2. 2511.txt
        9 kB
        Jeremy Hanna

        Activity

        Hide
        Jeremy Hanna added a comment -

        Forward merged the parts from CASSANDRA-2387 that hadn't made it in. The only question I had was whether it was worth it to embed a call to decompose in the TimeUUIDType.fromString method. It may not be worth it.

        Show
        Jeremy Hanna added a comment - Forward merged the parts from CASSANDRA-2387 that hadn't made it in. The only question I had was whether it was worth it to embed a call to decompose in the TimeUUIDType.fromString method. It may not be worth it.
        Hide
        Jeremy Hanna added a comment -

        So this makes it so the pig CassandraStorage can compile and run in 0.8. Patch is against 0.8-branch.

        Show
        Jeremy Hanna added a comment - So this makes it so the pig CassandraStorage can compile and run in 0.8. Patch is against 0.8-branch.
        Hide
        Jonathan Ellis added a comment -

        committed, thanks!

        Show
        Jonathan Ellis added a comment - committed, thanks!
        Hide
        Jeremy Hanna added a comment -

        The counter type may have an incorrect decompose (in its parent class) but I think the rest should be fine. Sorry about missing these the first time.

        Show
        Jeremy Hanna added a comment - The counter type may have an incorrect decompose (in its parent class) but I think the rest should be fine. Sorry about missing these the first time.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development