> +             System.out.println("SQLException: " + ex.getMessage());
> +             System.out.println("SQLState: " + ex.getSQLState());
> +             System.out.println("VendorError: " + ex.getErrorCode());
> +             return false;
> +         } finally {
> +             // it is a good idea to release
> +             // resources in a finally{} block
> +             // in reverse-order of their creation
> +             // if they are no-longer needed
> +
> +             if (rs != null) {
> +                 try {
> +                     rs.close();
> +                 } catch (SQLException sqlEx) { } // ignore
> +
> +                 rs = null;

Does rs really need to be set to null here? It's scoped to the method.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/6/files#r4753149

Reply via email to