You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemml.apache.org by mb...@apache.org on 2020/06/05 15:46:12 UTC

[systemml] branch master updated: [MINOR] Fix reading dml scripts from dist fs / object store

This is an automated email from the ASF dual-hosted git repository.

mboehm7 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/systemml.git


The following commit(s) were added to refs/heads/master by this push:
     new 9d5999d  [MINOR] Fix reading dml scripts from dist fs / object store
9d5999d is described below

commit 9d5999dc91df37beaddbdd48fe3c7487188f52a7
Author: Matthias Boehm <mb...@gmail.com>
AuthorDate: Fri Jun 5 17:45:36 2020 +0200

    [MINOR] Fix reading dml scripts from dist fs / object store
---
 src/main/java/org/apache/sysds/api/DMLScript.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/sysds/api/DMLScript.java b/src/main/java/org/apache/sysds/api/DMLScript.java
index a069182..6854ebb 100644
--- a/src/main/java/org/apache/sysds/api/DMLScript.java
+++ b/src/main/java/org/apache/sysds/api/DMLScript.java
@@ -112,8 +112,6 @@ public class DMLScript
 	public static String _uuid = IDHandler.createDistributedUniqueID();
 	private static final Log LOG = LogFactory.getLog(DMLScript.class.getName());
 
-	private static FileSystem fs = null;
-
 	///////////////////////////////
 	// public external interface
 	////////
@@ -283,7 +281,7 @@ public class DMLScript
 					|| IOUtilFunctions.isObjectStoreFileScheme(new Path(fileName)) )
 				{ 
 					Path scriptPath = new Path(fileName);
-					fs = IOUtilFunctions.getFileSystem(scriptPath);
+					FileSystem fs = IOUtilFunctions.getFileSystem(scriptPath);
 					in = new BufferedReader(new InputStreamReader(fs.open(scriptPath)));
 				}
 				// from local file system
@@ -303,8 +301,6 @@ public class DMLScript
 				throw ex;
 			}
 			finally {
-				if(fs != null)
-					fs.close();
 				IOUtilFunctions.closeSilently(in);
 			}