GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/805
CLOUDSTACK-8822 - Replacing Runnable by Callable That's the first part of the refactor, which will touch the ManagedContextRunnable and all its subclasses. However, in order to reduce impact, the first part comprises the Agent and Nio related classes (NioConnection, NioServer and NioClient). * All the sub-classes were also updated according to the changes in the super-classes * Improved exception handling * There were also code formatting changes Changes were structural and the NioTest covered them without need to modify the unit test. This PR is quite extensive. Please, wait for the Test Report in order to proceed with further review. ping @remibergsma @miguelaferreira @bhaisaab @karuturi @wido @DaanHoogland @borisroman @K0zka Cheers, Wilder You can merge this pull request into a Git repository by running: $ git pull https://github.com/ekholabs/cloudstack improvement/callable_CLOUDSTACK-8822 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/805.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #805 ---- commit 79a3f8c5774c50dc128968c29da5096dc3dde39e Author: wilderrodrigues <wrodrig...@schubergphilis.com> Date: 2015-09-08T10:12:55Z CLOUDSTACK-8822 - Replacing Runnable by Callable in the Taks and NioConnection classes - All the sub-classes were also updated according to the changes in the super-classes - There were also code formatting changes ---- --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---