[GitHub] [tinkerpop] spmallette opened pull request #1046: TINKERPOP-2138 Provide way to disable the global function cache

["spmallette (GitHub)" <gi...@apache.org>]

https://issues.apache.org/jira/browse/TINKERPOP-2138

Added a new `globalFunctionCacheEnabled` configuration to the `GroovyCompilerGremlinPlugin` and an overriding setting for sessions on `SessionOpProcessor` that disables the global function cache.  In cases where you don't need the cache (i.e. don't use global functions) it does remove a bit of reflection based processing from every single script request and obviously saves on memory formerly used by the cache. 

All tests pass with `docker/build.sh -t -n -i`

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1046 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] spmallette closed pull request #1046: TINKERPOP-2138 Provide way to disable the global function cache

["spmallette (GitHub)" <gi...@apache.org>]

[ pull request closed by spmallette ]

[ Full content available at: https://github.com/apache/tinkerpop/pull/1046 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] jorgebay commented on issue #1046: TINKERPOP-2138 Provide way to disable the global function cache

["jorgebay (GitHub)" <gi...@apache.org>]

Makes sense, VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1046 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] dkuppitz commented on issue #1046: TINKERPOP-2138 Provide way to disable the global function cache

["dkuppitz (GitHub)" <gi...@apache.org>]

LGTM.

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1046 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org