[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15342872#comment-15342872 ] ASF GitHub Bot commented on TINKERPOP-1332: --- Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/337 duh I totally missed that line when looking through the changes. Late to the party >.< > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15342870#comment-15342870 ] ASF GitHub Bot commented on TINKERPOP-1332: --- Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/337 Note that there is a fall back if the `getWidth()` is "too small" ... people say when it fails, it returns -1 so the fall back will kick in. > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15342501#comment-15342501 ] ASF GitHub Bot commented on TINKERPOP-1332: --- Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/337 Can someone else VOTE on this please? > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15334420#comment-15334420 ] Marko A. Rodriguez commented on TINKERPOP-1332: --- I now have {{prettyPrint(int maxLineWidth)}} and in the GremlinConsole, that line width is determined from JLine. Thus, we get it as such: {code} gremlin> g.V().out().out().out().out().repeat(out()).explain() ==>Traversal Explanation = Original Traversal [GraphStep(vertex,[]), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), RepeatStep([VertexStep(OUT,vertex), RepeatEndStep],until(false),emit(false))] ConnectiveStrategy [D] [GraphStep(vertex,[]), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), RepeatStep([VertexStep(OUT,vertex), RepeatEndStep],until(false),emit(false))] ... {code} Make the window smaller in the same session and: {code} gremlin> g.V().out().out().out().out().out().explain() ==>Traversal Explanation === Original Traversal [GraphStep(vertex,[]), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex)] ConnectiveStrategy [D] [GraphStep(vertex,[]), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex), VertexStep(OUT,vertex)] ... {code} The {{toString()}} is simply {{prettyPrint(Integer.MAX_LENGTH)}}. > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), >
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15332039#comment-15332039 ] ASF GitHub Bot commented on TINKERPOP-1332: --- Github user RussellSpitzer commented on the issue: https://github.com/apache/tinkerpop/pull/337 I still think this is pretty verbose, but I'm happy with the inclusion of the ```graphfilter[[VertexStep(OUT,edge)]]``` so +1 on that > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331811#comment-15331811 ] stephen mallette commented on TINKERPOP-1332: - I sort of agree with [~rjbriody] about pushing formatting into the output. Maybe we should do a {{prettyPrint()}} method on {{TraversalExplanation}} that does some formatting or something and leave {{toString()}} alone. In the console we could probably detect a {{TraversalExplanation}} being eval'd out in the console and then call the {{prettyPrint()}} (i think). Perhaps that would solve the problem? > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331802#comment-15331802 ] Bob Briody commented on TINKERPOP-1332: --- I don't like the idea of introducing word wrap here. That seems like something that should be left up to the view presenter (the terminal, another app, whatever). I realize that it's common to view this output in the terminal but 1) this isn't something you can easily reverse when you don't want it and 2) It will only look good for the hard coded terminal size. If you do decide to keep it though, then width should be 80 characters. That's the standard default terminal character width. > It would be nice if this was set to the console window size in some way... :/ > ... any thoughts on how to make this more dynamic? No, but again, we should be thinking about more than just terminal output. > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Assignee: Marko A. Rodriguez >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331610#comment-15331610 ] Marko A. Rodriguez commented on TINKERPOP-1332: --- I have done two things. 1. TraversalExplanation.toString() word wraps at ",\n" break points. 2. Every {{VertexProgramStep}} toString() has the {{GraphFilter.toString()}} with it. > Improve .explain() Dialogue > > > Key: TINKERPOP-1332 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1332 > Project: TinkerPop > Issue Type: Improvement > Components: process >Affects Versions: 3.2.0-incubating >Reporter: Russell Alexander Spitzer >Priority: Minor > > Currently the output of explain gives you a long list of strategies but no > details about their application > {code} > ==>Traversal Explanation > > Original Traversal [GraphStep(vertex,[]), CountGlobalStep] > HaltedTraverserStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > ConnectiveStrategy [D] [GraphStep(vertex,[]), CountGlobalStep] > VertexProgramStrategy[D] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > OrderLimitStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IdentityRemovalStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > FilterRankingStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > IncidentToAdjacentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > RangeByIsCountStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > AdjacentToIncidentStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > MatchPredicateStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > GraphFilterStrategy [O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > PathProcessorStrategy[O] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkInterceptorStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > SparkSingleIterationStrategy [P] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ProfileStrategy [F] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > LambdaRestrictionStrategy[V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > ComputerVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > StandardVerificationStrategy [V] > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > Final Traversal > [TraversalVertexProgramStep([GraphStep(vertex,[]), CountGlobalStep]), > ComputerResultStep] > {code} > It would be helpful if filter strategies for example would list the filters > used. -- This message was sent by Atlassian JIRA (v6.3.4#6332)