creaper icon indicating copy to clipboard operation
creaper copied to clipboard

ManagementClient.online should declare more checked exceptions

Open Ladicek opened this issue 9 years ago • 3 comments

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.

Ladicek avatar Apr 25 '16 13:04 Ladicek

It should probably declare throws Exception.

Ladicek avatar May 16 '16 14:05 Ladicek

The newly created SSL options (see #105) are also affected.

Ladicek avatar May 19 '16 12:05 Ladicek

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.

Ladicek avatar Jun 30 '16 12:06 Ladicek