Change this "try" to a try-with-resource 14/79814/1
authorDriptaroop Das <driptaroop.das@in.ibm.com>
Wed, 6 Mar 2019 19:59:18 +0000 (01:29 +0530)
committerDriptaroop Das <driptaroop.das@in.ibm.com>
Wed, 6 Mar 2019 19:59:24 +0000 (01:29 +0530)
Change this "try" to a try-with-resource

Issue-ID: DMAAP-1075
Change-Id: I2d5816bdf63301cc28d80250e2507f3a1cb96a0c
Signed-off-by: Driptaroop Das <driptaroop.das@in.ibm.com>
src/main/java/org/onap/dmaap/dbcapi/database/LoadSchema.java

index 1f1b078..349bc46 100644 (file)
@@ -25,7 +25,6 @@ import java.sql.*;
 
 import com.att.eelf.configuration.EELFLogger;
 import com.att.eelf.configuration.EELFManager;
-;
 
 public class LoadSchema        {
        private static final EELFLogger logger = EELFManager.getInstance().getLogger(LoadSchema.class);
@@ -44,12 +43,12 @@ public class LoadSchema     {
        }
        static void upgrade() throws SQLException {
                ConnectionFactory cf = ConnectionFactory.getDefaultInstance();
-               Connection c = null;
-               Statement stmt = null;
                InputStream is = null;
-               try { 
-                       c = cf.get(true);
-                       stmt = c.createStatement();
+               try(
+                               Connection c = cf.get(true);
+                               Statement stmt = c.createStatement();
+               ) {
+
                        
                        // this sets the PG search_path to a consistent schema, otherwise sometimes
                        // we get public, and sometimes we get dmaap_admin
@@ -120,21 +119,6 @@ public class LoadSchema    {
                        }
                } catch (IOException ioe) {
                        throw new SQLException(ioe);
-               } finally {
-                       if (stmt != null) { 
-                               try { 
-                                       stmt.close(); 
-                               } catch (Exception e) {
-                                       logger.error("Error", e);
-                               }
-                       }
-                       if (c != null) { 
-                               try { 
-                                       c.close(); 
-                                       } catch (Exception e) {
-                                               logger.error("Error", e);
-                                       }
-                               }
                }
        }
        public static void main(String[] args) throws Exception {