CacheClient.close() should adhere to contract

[galderz]

#136

[cruftex]

Comments relating to the PR from #136

Thanks for the PR. In the PR you now wrap all exceptions to IOExceptions.

There is no interface contract on Closeable, which says, that only IOExceptions are allowed.
Wrapping arbitrary RuntimeExceptions is not needed and contrary to the intention of the interface, which says: "@throws IOException if an I/O error occurs". A RuntimeException needs to be properly handled as well by any interface user.

If there is any exception on close, something is odd at another place. I cannot see how the PR relates to the problems you are experiencing.

[eolivelli]

This branch needs rebase
ERROR] Failed to execute goal on project cache-tests: Could not resolve dependencies for project javax.cache:cache-tests:jar:1.1.0-SNAPSHOT: Failure to find javax.cache:cache-api:jar:1.1.0-SNAPSHOT in https://repository-jsr107.forge.cloudbees.com/snapshot was cached in the local repository, resolution will not be reattempted until the update interval of cloudbees-snapshots has elapsed or updates are forced -> [Help 1]

Anyway I don't think this change is really needed in the TCK. I have read the discussion at #136 .
Is there still any strong need for this change ?