[KUDU-407] Unintentional fall-through in concurrent btree

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: M4.5
• Component/s: tablet
• Target Version/s: M4.5
• Code Review: http://gerrit.sjc.cloudera.com:8080/#/c/3673/
• Assignee: Todd Lipcon
• Reporter: Todd Lipcon
• Created: 23/Jul/14 19:39
• Updated: 20/Jan/15 16:48
• Resolved: 17/Dec/14 14:13

Description

    switch (node->Update(mutation, val)) {
      case LeafNode<Traits>::UPDATE_NOTFOUND:
        node->Unlock();
      case LeafNode<Traits>::UPDATE_SUCCESS:
        node->Unlock();
        return true;

would result in a double unlock