Revert "Fixed sonar fix in DataSourceWrap.java" 54/88654/1
authorTimoney, Dan (dt5972) <dtimoney@att.com>
Tue, 28 May 2019 13:15:39 +0000 (09:15 -0400)
committerTimoney, Dan (dt5972) <dtimoney@att.com>
Tue, 28 May 2019 13:16:09 +0000 (09:16 -0400)
This reverts commit 6c1dee5b7d0e66338100c65efd51917d4f69998f.
Sonar gave bad advice - this change causes connection to be closed
on return to caller, which is useless.

Change-Id: Id428c9058b5e33d72d58bbd951e6cdac2d2d151d
Signed-off-by: Timoney, Dan (dt5972) <dtimoney@att.com>
resource-assignment/provider/src/main/java/org/onap/ccsdk/sli/adaptors/util/db/DataSourceWrap.java

index 259b9d8..2aebb83 100644 (file)
@@ -4,7 +4,6 @@
  * ================================================================================
  * Copyright (C) 2017 AT&T Intellectual Property. All rights
  *                         reserved.
- * Modifications Copyright (C) 2018 IBM.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -75,22 +74,25 @@ public class DataSourceWrap implements DataSource {
 
     @Override
     public Connection getConnection() throws SQLException {
-        try (Connection c = dataSource.getConnection()){
-            log.debug("getConnection: " + c.getClass().getName());
-            c.setAutoCommit(true);
-            return c;
-        }
+        Connection c = dataSource.getConnection();
+
+        log.debug("getConnection: " + c.getClass().getName());
+
+        c.setAutoCommit(true);
+        return c;
     }
+
     @Override
     public Connection getConnection(String username, String password) throws SQLException {
-        try (Connection c = dataSource.getConnection(username, password)){
-            log.debug("getConnection: " + c.getClass().getName());
-            c.setAutoCommit(true);
-            return c;
-        }
+        Connection c = dataSource.getConnection(username, password);
+
+        log.debug("getConnection: " + c.getClass().getName());
+
+        c.setAutoCommit(true);
+        return c;
     }
 
-        public void setDataSource(DataSource dataSource) {
+    public void setDataSource(DataSource dataSource) {
         this.dataSource = dataSource;
     }
 }