creaper
creaper copied to clipboard
ManagementClient.online should declare more checked exceptions
This is so that all exceptions from OnlineOptions.createModelControllerClient don't have to be wrapped in IllegalStateException. It is source-incompatible, so it has to be done in a major release.
It should probably declare throws Exception.
The newly created SSL options (see #105) are also affected.
There's also OnlineManagementClientImpl.apply that catches InterruptedException and throws a different one. This needs careful examination, if the interruption status could possibly be lost in AutomaticErrorHandlingForCommands.