[PHOENIX-3513] Throw SQLException instead of IllegalStateException when max commit size exceeded

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.11.0
• Assignee: Thomas D'Silva
• Reporter: James R. Taylor
• Created: 01/Dec/16 01:52
• Updated: 02/Jun/17 21:23
• Resolved: 02/Jun/17 21:23

Description

We should change this code in MutationState and implement the TODO here:

    private void throwIfTooBig() {
        if (numRows > maxSize) {
            // TODO: throw SQLException ?
            throw new IllegalArgumentException("MutationState size of " + numRows + " is bigger than max allowed size of " + maxSize);
        }
    }

Otherwise, it's difficult for clients to react to the exception.

Issue Links

• is superceded by: PHOENIX-3782 MutationStates with too many rows should transparently split requests instead of throwing exception