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.txt
        9 kB
        Jeremy Hanna
      2. 2511-newer-types.txt
        2 kB
        Jeremy Hanna

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        26m 36s 1 Jeremy Hanna 19/Apr/11 22:49
        Patch Available Patch Available Resolved Resolved
        5m 5s 1 Jonathan Ellis 19/Apr/11 22:54
        Gavin made changes -
        Workflow patch-available, re-open possible [ 12751429 ] reopen-resolved, no closed status, patch-avail, testing [ 12757752 ]
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12610980 ] patch-available, re-open possible [ 12751429 ]
        Jeremy Hanna made changes -
        Attachment 2511-newer-types.txt [ 12476799 ]
        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.
        Jonathan Ellis made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Reviewer jbellis
        Resolution Fixed [ 1 ]
        Hide
        Jonathan Ellis added a comment -

        committed, thanks!

        Show
        Jonathan Ellis added a comment - committed, thanks!
        Jonathan Ellis made changes -
        Fix Version/s 0.8 [ 12314820 ]
        Affects Version/s 0.8 [ 12314820 ]
        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.
        Jeremy Hanna made changes -
        Attachment 2511.txt [ 12476786 ]
        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.
        Jeremy Hanna made changes -
        Field Original Value New Value
        Status Open [ 1 ] Patch Available [ 10002 ]
        Jeremy Hanna created issue -

          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