Details
-
Improvement
-
Status: Closed
-
Critical
-
Resolution: Fixed
-
0.90.4
-
None
-
None
-
Reviewed
-
Created a new class OperationStatus that wraps OperationStatusCode and an exception message, most of the patch consists of the former replacing the latter.
Description
It's kind of a dumb one, in HRegion.doMiniBatchPut we do:
LOG.warn("No such column family in batch put", nscf);
batchOp.retCodes[lastIndexExclusive] = OperationStatusCode.BAD_FAMILY;
So we lose the family here, all we know is there's a bad one, that's what's in HRS.multi:
} else if (code == OperationStatusCode.BAD_FAMILY) { result = new NoSuchColumnFamilyException();
We can't just throw the exception like that, we need to say which one is bad even if it requires testing all passed MultiActions.