[jira] [Commented] (TINKERPOP-1560) Cache in GroovyClassLoader may continue to grow
[
https://issues.apache.org/jira/browse/TINKERPOP-1560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15677969#comment-15677969
]
Benjamin Anderson commented on TINKERPOP-1560:
--
If I recall correctly this may be related to these:
https://issues.apache.org/jira/browse/GROOVY-6704
https://bugs.openjdk.java.net/browse/JDK-8136353
I've not had luck using any of the approaches suggested there to avoid the
issue.
> Cache in GroovyClassLoader may continue to grow
> ---
>
> Key: TINKERPOP-1560
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1560
> Project: TinkerPop
> Issue Type: Improvement
> Components: groovy
>Affects Versions: 3.2.3
>Reporter: stephen mallette
>Assignee: stephen mallette
>
> The {{GroovyClassLoader}} has a {{classCache}} that seems to continue to grow
> for each non-parameterized script that is passed through the
> {{GremlinGroovyScriptEngine}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1418) CoreTraversalTests depend on missing functionality
[
https://issues.apache.org/jira/browse/TINKERPOP-1418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15437427#comment-15437427
]
Benjamin Anderson commented on TINKERPOP-1418:
--
Alrighty. There's a dozen or so references in the comments for
{{Graph.vertices}} and {{Graph.edges}} that pretty clearly point to element ids.
Out of curiosity, why was this behavior removed from GraphStep and pushed down
to the implementor?
> CoreTraversalTests depend on missing functionality
> --
>
> Key: TINKERPOP-1418
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1418
> Project: TinkerPop
> Issue Type: Bug
> Components: structure, test-suite
>Affects Versions: 3.2.1
>Reporter: Benjamin Anderson
>Priority: Minor
>
> The {{shouldLoadVerticesViaVertices}} and {{shouldLoadEdgesViaEdges}} tests
> in
> [CoreTraversalTests.java|https://github.com/apache/tinkerpop/blob/master/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/CoreTraversalTest.java#L94-L110]
> appear to depend on unspecified functionality.
> The comment for the {{.vertices}} method in the Graph structure class does
> not specify that it should accept Vertex objects (except in [one odd
> place|https://github.com/apache/tinkerpop/blob/d696b38ab774dd46c6723d5c59aa7329f96b41e1/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java#L219])
> - as an implementor it reads as if it's intended to be called only with
> identifiers.
> [When these two tests were
> added|https://github.com/apache/tinkerpop/commit/da30a525ef626b8f13ffba0b7abada89b68eff6b]
> some functionality was included in GraphStep to handle mapping Vertex
> objects to their IDs; this functionality was [soon
> removed|https://github.com/apache/tinkerpop/commit/59028c8ad18f23609e31220a59717a2bc05b78af],
> however, and added to TinkerGraph. Incidentally, Titan [also implements the
> same
> functionality|https://github.com/thinkaurelius/titan/blob/titan10/titan-core/src/main/java/com/thinkaurelius/titan/graphdb/tinkerpop/TitanBlueprintsTransaction.java#L113-L141].
> Is {{Graph.vertices(Object...)}} intended to gracefully handle an array of
> Vertex elements as well as an array of Vertex identifiers? If so, then it
> appears that the documentation in Graph.java needs some reworking. It'd also
> perhaps be unfortunate, stylistically, to add a third behavior to that
> method. If it's not intended to support that call, then it seems that these
> two methods (and perhaps others?) should be removed from the test suite.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Created] (TINKERPOP-1418) CoreTraversalTests depend on missing functionality
Benjamin Anderson created TINKERPOP-1418:
Summary: CoreTraversalTests depend on missing functionality
Key: TINKERPOP-1418
URL: https://issues.apache.org/jira/browse/TINKERPOP-1418
Project: TinkerPop
Issue Type: Bug
Components: structure, test-suite
Affects Versions: 3.2.1
Reporter: Benjamin Anderson
Priority: Minor
The {{shouldLoadVerticesViaVertices}} and {{shouldLoadEdgesViaEdges}} tests in
[CoreTraversalTests.java|https://github.com/apache/tinkerpop/blob/master/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/CoreTraversalTest.java#L94-L110]
appear to depend on unspecified functionality.
The comment for the {{.vertices}} method in the Graph structure class does not
specify that it should accept Vertex objects (except in [one odd
place|https://github.com/apache/tinkerpop/blob/d696b38ab774dd46c6723d5c59aa7329f96b41e1/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java#L219])
- as an implementor it reads as if it's intended to be called only with
identifiers.
[When these two tests were
added|https://github.com/apache/tinkerpop/commit/da30a525ef626b8f13ffba0b7abada89b68eff6b]
some functionality was included in GraphStep to handle mapping Vertex objects
to their IDs; this functionality was [soon
removed|https://github.com/apache/tinkerpop/commit/59028c8ad18f23609e31220a59717a2bc05b78af],
however, and added to TinkerGraph. Incidentally, Titan [also implements the
same
functionality|https://github.com/thinkaurelius/titan/blob/titan10/titan-core/src/main/java/com/thinkaurelius/titan/graphdb/tinkerpop/TitanBlueprintsTransaction.java#L113-L141].
Is {{Graph.vertices(Object...)}} intended to gracefully handle an array of
Vertex elements as well as an array of Vertex identifiers? If so, then it
appears that the documentation in Graph.java needs some reworking. It'd also
perhaps be unfortunate, stylistically, to add a third behavior to that method.
If it's not intended to support that call, then it seems that these two methods
(and perhaps others?) should be removed from the test suite.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
