bharathv commented on pull request #2130: URL: https://github.com/apache/hbase/pull/2130#issuecomment-679252642
> The only concern is about the IllegalStateException. We used to throw IOException for this case. I added a checked exception. TBH I don't fully understand the concern. Like I explained it happens only if an invariant fails, which means a code bug, so unchecked exception should be fine. Either way its not a big deal what exception we throw, so made the changed. Also, fixed the check style issues. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
