You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2015/02/19 15:37:14 UTC

incubator-tinkerpop git commit: Re-use the ScriptEngineManager in the ScriptEngines class.

Repository: incubator-tinkerpop
Updated Branches:
  refs/heads/master 44c5c46cd -> 402920454


Re-use the ScriptEngineManager in the ScriptEngines class.


Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/40292045
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/40292045
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/40292045

Branch: refs/heads/master
Commit: 402920454303936830d7a498e64b06ff739c4068
Parents: 44c5c46
Author: Stephen Mallette <sp...@apache.org>
Authored: Thu Feb 19 09:04:29 2015 -0500
Committer: Stephen Mallette <sp...@apache.org>
Committed: Thu Feb 19 09:04:29 2015 -0500

----------------------------------------------------------------------
 .../apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/40292045/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java
----------------------------------------------------------------------
diff --git a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java
index dc4c6d4..2763ef0 100644
--- a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java
+++ b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/ScriptEngines.java
@@ -56,6 +56,8 @@ import java.util.stream.Collectors;
 public class ScriptEngines implements AutoCloseable {
     private static final Logger logger = LoggerFactory.getLogger(ScriptEngines.class);
 
+    private final static ScriptEngineManager SCRIPT_ENGINE_MANAGER = new ScriptEngineManager();
+
     /**
      * {@code ScriptEngine} objects configured for the server keyed on the language name.
      */
@@ -294,8 +296,7 @@ public class ScriptEngines implements AutoCloseable {
             return Optional.of((ScriptEngine) new GremlinGroovyScriptEngine(
                     new DefaultImportCustomizerProvider(imports, staticImports), securityCustomizerProvider));
         } else {
-            final ScriptEngineManager manager = new ScriptEngineManager();
-            return Optional.ofNullable(manager.getEngineByName(language));
+            return Optional.ofNullable(SCRIPT_ENGINE_MANAGER.getEngineByName(language));
         }
     }