Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Fix Version/s: 0.7 beta 1
    • Component/s: Core
    • Labels:
      None

      Activity

      Hide
      Jonathan Ellis added a comment -

      wfm

      Show
      Jonathan Ellis added a comment - wfm
      Hide
      Gary Dusbabek added a comment -

      Your point is valid, but the CFs need to be dropped somewhere. I'll change the comment so it doesn't give the impression of protection.

      Show
      Gary Dusbabek added a comment - Your point is valid, but the CFs need to be dropped somewhere. I'll change the comment so it doesn't give the impression of protection.
      Hide
      Jonathan Ellis added a comment -

      I don't think this is necessary:

      + // first remove all cfs from the table instance, so that any pending row mutations error out.

      My reasoning is, with or without that code it is possible to start a mutation that is invalid by the time it is processed, so why bother?

      +1 the rest.

      Show
      Jonathan Ellis added a comment - I don't think this is necessary: + // first remove all cfs from the table instance, so that any pending row mutations error out. My reasoning is, with or without that code it is possible to start a mutation that is invalid by the time it is processed, so why bother? +1 the rest.

        People

        • Assignee:
          Gary Dusbabek
          Reporter:
          Gary Dusbabek
        • Votes:
          1 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development