diff options
author | Ryan Ernst <ryan@iernst.net> | 2017-05-24 13:19:27 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-05-24 13:19:27 -0700 |
commit | 0ddd21942315092d76b43786b45c5ebb015849a5 (patch) | |
tree | 167c0e2c339754d58ac64499c4377ee2a1ea0170 /modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java | |
parent | 7a6db074ee95f1df7d4f678ea44332e9f655f4c8 (diff) |
Scripting: Add default implementation of close() for ScriptEngine (#24851)
Since groovy was removed, we no longer have any ScriptEngines with
resources to release. We may want to keep the option open for a script
engine to close resources, but this would not be common. This commit
adds a default implementation to ScriptEngine for `close()` to reduce
the boiler plate that must be added for a ScriptEngine implementation.
Diffstat (limited to 'modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java')
-rw-r--r-- | modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java b/modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java index 11a10247e0..73d1af32c0 100644 --- a/modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java +++ b/modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java @@ -252,7 +252,4 @@ public class ExpressionScriptEngine extends AbstractComponent implements ScriptE public ExecutableScript executable(CompiledScript compiledScript, Map<String, Object> vars) { return new ExpressionExecutableScript(compiledScript, vars); } - - @Override - public void close() {} } |