> +         catch (SQLException e){
> +            System.out.format("SQLException: %s%n", e.getMessage());
> +            System.out.format("SQLState: %s%n", e.getSQLState());
> +            System.out.format("VendorError: %s%n", e.getErrorCode());
> +            e.printStackTrace();
> +            
> +            return false;
> +         } 
> +         finally {            
> +            // Release resources in reverse order of creation.
> +            
> +            if (rs != null) {
> +               try {
> +                  rs.close();
> +               }
> +               catch (SQLException sqlEx) {

> I don't think the Exception needs to be renamed considering the comment below 
> it.

Fair enough. Was just wondering if we still need the comment if we rename the 
variable, but fine to keep as-is.

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

Reply via email to