[jira] [Closed] (TINKERPOP-1123) Execution time in console output

2018-03-05 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1123.
---
Resolution: Won't Do

closing given previous comment. if there is some fresh interest with a good 
idea how to implement it, we can rethink it.

> Execution time in console output
> 
>
> Key: TINKERPOP-1123
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1123
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: console
>Affects Versions: 3.1.0-incubating
>Reporter: Jeremy Hanna
>Priority: Minor
>
> Often I'd like to have the execution time for queries.  It would be nice to 
> have this as a default or at least a startup option in the gremlin shell.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Closed] (TINKERPOP-892) Saving Graph to file won't save graph variables

2018-03-05 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-892?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-892.
--
Resolution: Won't Do

Added a note about this to the "future" doc as alluded to in the previous 
comment:

https://github.com/apache/tinkerpop/commit/512a4b8b05fe036c828c3562bd72a50fc1b79e76

> Saving Graph to file won't save graph variables
> ---
>
> Key: TINKERPOP-892
> URL: https://issues.apache.org/jira/browse/TINKERPOP-892
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: structure
>Affects Versions: 3.0.2-incubating
>Reporter: Flavio Cordova
>Priority: Trivial
>
> When you save a graph using writeGraph graph variables are not saved along 
> with all the vertices and edges, so you can't fully recover the graph when 
> you read the file.
> In the forum there's a thread called "Saving graph variables to File" that 
> will discuss the matter in details but basically "I understand the API should 
> handle variables and persist then properly, probably adapting it as a vertex 
> to be compliant to the format, but it should be transparent to the developer, 
> something like neo4J does for multi-properties. IF the way they are written 
> to the file were not good enough [for me] then I could be able to change it 
> writting a custom writer/reader, but not the way around."



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (TINKERPOP-1758) RemoteStrategy should be before all other DecorationStrategies.

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386959#comment-16386959
 ] 

ASF GitHub Bot commented on TINKERPOP-1758:
---

GitHub user spmallette opened a pull request:

https://github.com/apache/tinkerpop/pull/811

TINKERPOP-1758 Apply RemoteStrategy before all DecorationStrategy instances

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

Played around with a few ways to test this, but none seemed especially 
good. Everything I try to do seems fairly contrived or redundant. In the end, I 
ended up feeling like it was safe to just rely on the TraversalStrategies 
sorting system with posts/priors. That's pretty solidly tested so perhaps that 
is enough...

Here's the result of a quick manual test:

```text
gremlin> g = 
TinkerGraph.open().traversal().withRemote('conf/remote-graph.properties').withStrategies(SubgraphStrategy.build().vertices(identity()).create())
==>graphtraversalsource[tinkergraph[vertices:0 edges:0], standard]
gremlin> g.V().out().out().values('name').explain()
==>Traversal Explanation

===
Original Traversal [GraphStep(vertex,[]), 
VertexStep(OUT,vertex), VertexStep(OUT,vertex), PropertiesStep([name],value)]

RemoteStrategy   [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
SubgraphStrategy [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
ConnectiveStrategy   [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
RangeByIsCountStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
RepeatUnrollStrategy [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
MatchPredicateStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
FilterRankingStrategy[O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
InlineFilterStrategy [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
IncidentToAdjacentStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
AdjacentToIncidentStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
PathRetractionStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
LazyBarrierStrategy  [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
TinkerGraphCountStrategy [P]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
TinkerGraphStepStrategy  [P]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
ProfileStrategy  [F]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
StandardVerificationStrategy [V]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]

Final Traversal
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
```

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

VOTE +1

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/tinkerpop TINKERPOP-1758

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/tinkerpop/pull/811.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #811


commit 528851411c6494bf92f2f929324a79ba28a941b2
Author: Stephen Mallette 
Date:   2018-03-05T18:18:35Z

TINKERPOP-1758 Apply RemoteStrategy before all DecorationStrategy instances

Played around with a few ways to test this, but none seemed especially 
good. Everything I try to do seems fairly contrived or redundant. In the end, I 
ended up feeling like it was safe to just rely on the TraversalStrategies 
sorting system with posts/priors. Perhaps that is enough...




> RemoteStrategy should be before all other DecorationStrategies.
> ---
>
> Key: TINKERPOP-1758
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1758
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.6
>Reporter: Marko A. Rodriguez
>Assignee: stephen mallette
>Priority: Major
>
> {code}
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
>

[GitHub] tinkerpop pull request #811: TINKERPOP-1758 Apply RemoteStrategy before all ...

2018-03-05 Thread spmallette 
GitHub user spmallette opened a pull request:

https://github.com/apache/tinkerpop/pull/811

TINKERPOP-1758 Apply RemoteStrategy before all DecorationStrategy instances

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

Played around with a few ways to test this, but none seemed especially 
good. Everything I try to do seems fairly contrived or redundant. In the end, I 
ended up feeling like it was safe to just rely on the TraversalStrategies 
sorting system with posts/priors. That's pretty solidly tested so perhaps that 
is enough...

Here's the result of a quick manual test:

```text
gremlin> g = 
TinkerGraph.open().traversal().withRemote('conf/remote-graph.properties').withStrategies(SubgraphStrategy.build().vertices(identity()).create())
==>graphtraversalsource[tinkergraph[vertices:0 edges:0], standard]
gremlin> g.V().out().out().values('name').explain()
==>Traversal Explanation

===
Original Traversal [GraphStep(vertex,[]), 
VertexStep(OUT,vertex), VertexStep(OUT,vertex), PropertiesStep([name],value)]

RemoteStrategy   [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
SubgraphStrategy [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
ConnectiveStrategy   [D]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
RangeByIsCountStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
RepeatUnrollStrategy [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
MatchPredicateStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
FilterRankingStrategy[O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
InlineFilterStrategy [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
IncidentToAdjacentStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
AdjacentToIncidentStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
PathRetractionStrategy   [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
LazyBarrierStrategy  [O]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
TinkerGraphCountStrategy [P]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
TinkerGraphStepStrategy  [P]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
ProfileStrategy  [F]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
StandardVerificationStrategy [V]   
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]

Final Traversal
[RemoteStep(DriverServerConnection-localhost/127.0.0.1:8182 [graph=g])]
```

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

VOTE +1

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/tinkerpop TINKERPOP-1758

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/tinkerpop/pull/811.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #811


commit 528851411c6494bf92f2f929324a79ba28a941b2
Author: Stephen Mallette 
Date:   2018-03-05T18:18:35Z

TINKERPOP-1758 Apply RemoteStrategy before all DecorationStrategy instances

Played around with a few ways to test this, but none seemed especially 
good. Everything I try to do seems fairly contrived or redundant. In the end, I 
ended up feeling like it was safe to just rely on the TraversalStrategies 
sorting system with posts/priors. Perhaps that is enough...




---

Re: Documentation generation with Docker

2018-03-05 Thread Florian Hockmann 
You can save images as a tar file with docker save:
https://docs.docker.com/engine/reference/commandline/save/

|docker save -o tinkerpop.tar tinkerpop|

Since I also have the problem where the build fails at
olap-spark-yarn.asciidoc, I just exported my images and uploaded them here:

https://uploadfiles.io/eew3d

The images were created by running ./docker/build.sh -d on master after
deleting all images I had locally.


Am 05.03.2018 um 22:05 schrieb Stephen Mallette:
> I'm up to try anything to solve this so, sure, happy to try to share. Sorry
> though, but what's the "easy" way to share an image?
>
> On Fri, Mar 2, 2018 at 3:03 PM, Robert Dale  wrote:
>
>> is it possible to share your images and see if it's an image issue or a
>> docker environment issue?
>>
>> Robert Dale
>>
>> On Wed, Feb 28, 2018 at 3:34 PM, Stephen Mallette 
>> wrote:
>>
>>> I pushed the fix for master, so I don't have the console problem anymore.
>>> So now, it's just the issue I guess I've always had.
>>>
>>> On Wed, Feb 28, 2018 at 12:18 PM, Robert Dale  wrote:
>>>
 That works for me.

 Robert Dale

 On Wed, Feb 28, 2018 at 11:23 AM, Florian Hockmann <
>>> f...@florian-hockmann.de
 wrote:

> I also ran into the exact same problem in my feature branch for the
> docker images which is why haven't added my own vote for the PR yet.
>>> But
> it's good to see that it's really completely unrelated to my changes.
>
>
> Am 28.02.2018 um 17:15 schrieb Stephen Mallette:
>> I'm still a bust - same kind of error I keep having -
>>
>>  * source:   /usr/src/tinkerpop/docs/src/recipes/olap-spark-yarn.
> asciidoc
>>target:
>>  /usr/src/tinkerpop/target/postprocess-asciidoc/recipes/
> olap-spark-yarn.asciidoc
>>progress:
>> [===
>> ==>
>>   ] 65%java.io.IOException: No input paths
>> specified in job
>> Type ':help' or ':h' for help.
>> Display stack trace? [yN]pb(94); ''
>>
>>
>> Last 10 lines of
>> /usr/src/tinkerpop/target/postprocess-asciidoc/recipes/
> olap-spark-yarn.asciidoc:
>> gremlin> conf.setProperty('spark.executor.extraLibraryPath',
>> "$hadoop/lib/native:$hadoop/lib/native/Linux-amd64-64")
>> ==>null
>> gremlin> conf.setProperty('gremlin.spark.persistContext', 'true')
>> ==>null
>> gremlin> graph = GraphFactory.open(conf)
>> ==>hadoopgraph[gryoinputformat->gryooutputformat]
>> gremlin> g = graph.traversal().withComputer(SparkGraphComputer)
>> ==>graphtraversalsource[hadoopgraph[gryoinputformat->
 gryooutputformat],
>> sparkgraphcomputer]
>> gremlin> g.V().group().by(values('name')).by(both().count())
>> gremlin>
>>
>> xargs: /usr/src/tinkerpop/docs/preprocessor/preprocess-file.sh:
>>> exited
> with
>> status 255; aborting
>>
>>
>>
>>
>> On Wed, Feb 28, 2018 at 10:46 AM, Robert Dale 
 wrote:
>>> Yup, it's a step in the release docs.  Once updated, master builds
 docs
> for
>>> me.
>>>
>>> Robert Dale
>>>
>>> On Wed, Feb 28, 2018 at 10:42 AM, Stephen Mallette <
> spmalle...@gmail.com>
>>> wrote:
>>>
 ah - i forgot to do that step.i'm running tp32 now to see if
>>> that
 works, but i'll fix that issue on master.

 On Wed, Feb 28, 2018 at 10:41 AM, Robert Dale  wrote:
> Could it be that 'bin/gremlin.sh' is linked to a specific
>> version?
> Does
> this have to be updated every release?
>
> $ ll gremlin-console/bin/gremlin.sh
> gremlin-console/bin/gremlin.sh ->
> ../target/apache-tinkerpop-gremlin-console-3.3.2-
 SNAPSHOT-standalone/bin/
> gremlin.sh
>
> Robert Dale
>
> On Wed, Feb 28, 2018 at 10:37 AM, Robert Dale <
>> robd...@gmail.com>
>>> wrote:
>> I got it too. tp32: good.  tp33: good. master: bad.
>>
>> Robert Dale
>>
>> On Wed, Feb 28, 2018 at 10:20 AM, Stephen Mallette <
 spmalle...@gmail.com
>> wrote:
>>
>>> I'm having no success generating docs with Docker. On master
>> i'm
> currently
>>> getting:
>>>
>>> Starting namenodes on [localhost]
>>> localhost: Warning: Permanently added 'localhost' (ECDSA) to
>> the
>>> list
 of
>>> known hosts.
>>> localhost: starting namenode, logging to
>>> /usr/local/lib/hadoop-2.7.2/logs/hadoop-root-namenode-
 61cfd8f63c77.out
>>> localhost: Warning: Permanently added 'localhost' (ECDSA) to
>> the
>>> list
 of
>>> known hosts.
>>> localhost: starting datanode, logging to
>>>

Re: [DISCUSS] Name of 3.4.x

2018-03-05 Thread Stephen Mallette 
it just needs to be "music related" and "have something numeric" about it.

On Mon, Mar 5, 2018 at 4:20 PM, David Brown  wrote:

> Does it need to have a 3 or 4 in it?
>
> - 3 Little Gremlins (Marley)
> - 4 Gremlins of the Apocalypse (Kind of like Clash)
>
> Idk I do like the Gambler...
>
>
> On Mon, Mar 5, 2018 at 12:53 PM Stephen Mallette 
> wrote:
>
> > So...we have a few good suggestions in hand (and I added one):
> >
> > 1. 4 Seasons of Gremlin - unclear if we're talking Boyz II Men or Vivaldi
> > here...
> > 2. Four Rusted Gremlins - Marilyn Manson
> > 3. Three Times a Gremlin - from the Commodores, though I think Gremlin
> > would prefer the Kenny Rogers styling of it - I mean, look at this
> > https://www.youtube.com/watch?v=Ok7becfXoJE :D
> >
> > Sorry Jason, gotta disqualify your suggestion as it doesn't have a
> "number
> > reference" in it.
> >
> > Anyone have any other ideas?
> >
> >
> >
> > On Thu, Mar 1, 2018 at 8:33 AM, Jason Plurad  wrote:
> >
> > > Boyz II Men - ha, I hadn't considered that angle. I assumed the 4
> Seasons
> > > was a Vivaldi reference.
> > >
> > > How about David Bowie? The Gremlin Who Sold the World
> > > On Thu, Mar 1, 2018 at 8:06 AM Stephen Mallette 
> > > wrote:
> > >
> > > > I'm guessing at the reference for "4 Seasons of Gremlin" - are we
> > talking
> > > > Boyz || Men and "4 Seasons of Loneliness"?
> > > >
> > > > >  Yeah, I'd like to see what Gremlyn Manson would look like!
> > > >
> > > > would he look that much different than our "Nine Inch Nails Gremlin"?
> > > >
> > > >
> > > >
> > > > On Tue, Feb 27, 2018 at 10:58 AM, Robert Dale 
> > wrote:
> > > >
> > > > > Yeah, I'd like to see what Gremlyn Manson would look like!
> > > > >
> > > > > Robert Dale
> > > > >
> > > > > On Tue, Feb 27, 2018 at 10:40 AM, Daniel Kuppitz 
> > > > wrote:
> > > > >
> > > > > > Four Rusted Gremlins (reference
> > > > > > ) :)
> > > > > >
> > > > > > On Tue, Feb 27, 2018 at 7:47 AM, Stephen Mallette <
> > > > spmalle...@gmail.com>
> > > > > > wrote:
> > > > > >
> > > > > > > We need a release name and associated Gremlin logo for the
> 3.4.x
> > > > series
> > > > > > of
> > > > > > > TinkerPop. Recall that names should be related to music and
> > > include a
> > > > > > > number. Thus far we've had:
> > > > > > >
> > > > > > > * Gremlin Symphony #40 in G Minor (3.3.x)
> > > > > > > * Nine Inch Gremlins (3.2.x)
> > > > > > > * A 187 On The Undercover Gremlinz (3.1.x)
> > > > > > > * A Gremlin Rāga in 7/16 Time (3.0.x)
> > > > > > >
> > > > > > > Please submit a name with a logo idea - a name where we can't
> > get a
> > > > > logo
> > > > > > > made doesn't really work too well. Thanks.
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>
>
> --
> David M. Brown
> R.A. CulturePlex Lab, Western University
>

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread Branden Moore (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386786#comment-16386786
 ] 

Branden Moore commented on TINKERPOP-1896:
--

> Anyway, If a traversal has to use a lambda it should make uniform use of a 
> single scripting language.

I agree completely...  However, it is your suggested work-around to this issue 
that leads me down the path of needing to use multiple script engines.   
GremlinGroovy does not work with server-side DSLs, and python-lambdas do not 
work in all cases (this issue).    So I am currently back to no work-around for 
this issue.

 

The entire reason I'm attempting to use a lambda is because Python Dictionaries 
are not hashable, and thus cannot be used as keys to another dictionary.   If 
there is an alternate way to convert, server-side, a Map to something constant, 
perhaps a string?

    What I'm wanting to do is essentially:
{code:java}
g.E().hasLabel('mylabel').match(
__.as('x').outV().label().as('from'),
__.as('x').inV().label().as('to')
).select('from', 'to').groupCount(){code}
 

 

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

Re: [DISCUSS] Name of 3.4.x

2018-03-05 Thread David Brown 
Does it need to have a 3 or 4 in it?

- 3 Little Gremlins (Marley)
- 4 Gremlins of the Apocalypse (Kind of like Clash)

Idk I do like the Gambler...


On Mon, Mar 5, 2018 at 12:53 PM Stephen Mallette 
wrote:

> So...we have a few good suggestions in hand (and I added one):
>
> 1. 4 Seasons of Gremlin - unclear if we're talking Boyz II Men or Vivaldi
> here...
> 2. Four Rusted Gremlins - Marilyn Manson
> 3. Three Times a Gremlin - from the Commodores, though I think Gremlin
> would prefer the Kenny Rogers styling of it - I mean, look at this
> https://www.youtube.com/watch?v=Ok7becfXoJE :D
>
> Sorry Jason, gotta disqualify your suggestion as it doesn't have a "number
> reference" in it.
>
> Anyone have any other ideas?
>
>
>
> On Thu, Mar 1, 2018 at 8:33 AM, Jason Plurad  wrote:
>
> > Boyz II Men - ha, I hadn't considered that angle. I assumed the 4 Seasons
> > was a Vivaldi reference.
> >
> > How about David Bowie? The Gremlin Who Sold the World
> > On Thu, Mar 1, 2018 at 8:06 AM Stephen Mallette 
> > wrote:
> >
> > > I'm guessing at the reference for "4 Seasons of Gremlin" - are we
> talking
> > > Boyz || Men and "4 Seasons of Loneliness"?
> > >
> > > >  Yeah, I'd like to see what Gremlyn Manson would look like!
> > >
> > > would he look that much different than our "Nine Inch Nails Gremlin"?
> > >
> > >
> > >
> > > On Tue, Feb 27, 2018 at 10:58 AM, Robert Dale 
> wrote:
> > >
> > > > Yeah, I'd like to see what Gremlyn Manson would look like!
> > > >
> > > > Robert Dale
> > > >
> > > > On Tue, Feb 27, 2018 at 10:40 AM, Daniel Kuppitz 
> > > wrote:
> > > >
> > > > > Four Rusted Gremlins (reference
> > > > > ) :)
> > > > >
> > > > > On Tue, Feb 27, 2018 at 7:47 AM, Stephen Mallette <
> > > spmalle...@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > We need a release name and associated Gremlin logo for the 3.4.x
> > > series
> > > > > of
> > > > > > TinkerPop. Recall that names should be related to music and
> > include a
> > > > > > number. Thus far we've had:
> > > > > >
> > > > > > * Gremlin Symphony #40 in G Minor (3.3.x)
> > > > > > * Nine Inch Gremlins (3.2.x)
> > > > > > * A 187 On The Undercover Gremlinz (3.1.x)
> > > > > > * A Gremlin Rāga in 7/16 Time (3.0.x)
> > > > > >
> > > > > > Please submit a name with a logo idea - a name where we can't
> get a
> > > > logo
> > > > > > made doesn't really work too well. Thanks.
> > > > > >
> > > > >
> > > >
> > >
> >
>


-- 
David M. Brown
R.A. CulturePlex Lab, Western University

[jira] [Commented] (TINKERPOP-1909) Gremlin.Net does not have complete object model as compared to other client drivers and unable to de-serialize properties for vertex/edge graphSON.

2018-03-05 Thread Ashwini Singh (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386779#comment-16386779
 ] 

Ashwini Singh commented on TINKERPOP-1909:
--

 

Few more questions. 

I understand the future direction for TinkerPop 4.X. But as of now what do you 
think about filling the gaps between .net and other drivers in TInkerPop3.X for 
customers who have taken the dependency on the older version?

What is our plan for bulk insert story (data insert) on TinkerPop 4.X? For a 
query, it makes sense to return reference data for different reasons(perf 
etc.). But, what is our plan for reading data from full graphSON file and 
insert into the store?, Without the full object model.  

> Gremlin.Net does not have complete object model as compared to other client 
> drivers and unable to de-serialize properties for vertex/edge graphSON.  
> -
>
> Key: TINKERPOP-1909
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1909
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.3.1
>Reporter: Ashwini Singh
>Priority: Major
>
> Looks like the object model for Gremlin.Net client driver is not as par with 
> Java driver. We cannot deserialize a GraphSON response to tinkerpop object 
> completely. For example, Gremlin.Net object model cannot deserialize 
> properties from a graphSON response object (vertex/edges). Currently, we only 
> deserialize id and label field from a vertex/edge graphSON.
>  
> So, to desterilize the object model, users have to write a custom 
> deserialization code and create the object. The current de-serializers (for 
> vertex/edge) will strip off details like properties.
>  
> I am filing it as a bug but it could fall into improvement as well.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386771#comment-16386771
 ] 

stephen mallette commented on TINKERPOP-1896:
-

> What is the expected behavior when multiple lambdas exist in a traversal, and 
> they specify different engines? 

i don't think the system was designed to handle that. i would fully expect you 
to get a weird unexplainable error as I doubt that was handled gracefully. we 
should probably look into that and create an issue to do some proper validation 
- if you have time to investigate what happens with that and create an issue as 
needed we'd appreciated it. 

Anyway, If a traversal has to use a lambda it should make uniform use of a 
single scripting language.

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread Branden Moore (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386768#comment-16386768
 ] 

Branden Moore commented on TINKERPOP-1896:
--

That makes a certain amount of sense.  What is the expected behavior when 
multiple lambdas exist in a traversal, and they specify different engines?  Or 
does Groovy trump all others, with the expectation that the Translator step 
will translate a python-lambda to a groovy-lambda?

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

Re: Documentation generation with Docker

2018-03-05 Thread Stephen Mallette 
I'm up to try anything to solve this so, sure, happy to try to share. Sorry
though, but what's the "easy" way to share an image?

On Fri, Mar 2, 2018 at 3:03 PM, Robert Dale  wrote:

> is it possible to share your images and see if it's an image issue or a
> docker environment issue?
>
> Robert Dale
>
> On Wed, Feb 28, 2018 at 3:34 PM, Stephen Mallette 
> wrote:
>
> > I pushed the fix for master, so I don't have the console problem anymore.
> > So now, it's just the issue I guess I've always had.
> >
> > On Wed, Feb 28, 2018 at 12:18 PM, Robert Dale  wrote:
> >
> > > That works for me.
> > >
> > > Robert Dale
> > >
> > > On Wed, Feb 28, 2018 at 11:23 AM, Florian Hockmann <
> > f...@florian-hockmann.de
> > > >
> > > wrote:
> > >
> > > > I also ran into the exact same problem in my feature branch for the
> > > > docker images which is why haven't added my own vote for the PR yet.
> > But
> > > > it's good to see that it's really completely unrelated to my changes.
> > > >
> > > >
> > > > Am 28.02.2018 um 17:15 schrieb Stephen Mallette:
> > > > > I'm still a bust - same kind of error I keep having -
> > > > >
> > > > >  * source:   /usr/src/tinkerpop/docs/src/recipes/olap-spark-yarn.
> > > > asciidoc
> > > > >target:
> > > > >  /usr/src/tinkerpop/target/postprocess-asciidoc/recipes/
> > > > olap-spark-yarn.asciidoc
> > > > >progress:
> > > > > [===
> ==>
> > > > >   ] 65%java.io.IOException: No input paths
> > > > > specified in job
> > > > > Type ':help' or ':h' for help.
> > > > > Display stack trace? [yN]pb(94); ''
> > > > >
> > > > >
> > > > > Last 10 lines of
> > > > > /usr/src/tinkerpop/target/postprocess-asciidoc/recipes/
> > > > olap-spark-yarn.asciidoc:
> > > > >
> > > > > gremlin> conf.setProperty('spark.executor.extraLibraryPath',
> > > > > "$hadoop/lib/native:$hadoop/lib/native/Linux-amd64-64")
> > > > > ==>null
> > > > > gremlin> conf.setProperty('gremlin.spark.persistContext', 'true')
> > > > > ==>null
> > > > > gremlin> graph = GraphFactory.open(conf)
> > > > > ==>hadoopgraph[gryoinputformat->gryooutputformat]
> > > > > gremlin> g = graph.traversal().withComputer(SparkGraphComputer)
> > > > > ==>graphtraversalsource[hadoopgraph[gryoinputformat->
> > > gryooutputformat],
> > > > > sparkgraphcomputer]
> > > > > gremlin> g.V().group().by(values('name')).by(both().count())
> > > > > gremlin>
> > > > >
> > > > > xargs: /usr/src/tinkerpop/docs/preprocessor/preprocess-file.sh:
> > exited
> > > > with
> > > > > status 255; aborting
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > On Wed, Feb 28, 2018 at 10:46 AM, Robert Dale 
> > > wrote:
> > > > >
> > > > >> Yup, it's a step in the release docs.  Once updated, master builds
> > > docs
> > > > for
> > > > >> me.
> > > > >>
> > > > >> Robert Dale
> > > > >>
> > > > >> On Wed, Feb 28, 2018 at 10:42 AM, Stephen Mallette <
> > > > spmalle...@gmail.com>
> > > > >> wrote:
> > > > >>
> > > > >>> ah - i forgot to do that step.i'm running tp32 now to see if
> > that
> > > > >>> works, but i'll fix that issue on master.
> > > > >>>
> > > > >>> On Wed, Feb 28, 2018 at 10:41 AM, Robert Dale  >
> > > > wrote:
> > > > >>>
> > > >  Could it be that 'bin/gremlin.sh' is linked to a specific
> version?
> > > > Does
> > > >  this have to be updated every release?
> > > > 
> > > >  $ ll gremlin-console/bin/gremlin.sh
> > > >  gremlin-console/bin/gremlin.sh ->
> > > >  ../target/apache-tinkerpop-gremlin-console-3.3.2-
> > > > >>> SNAPSHOT-standalone/bin/
> > > >  gremlin.sh
> > > > 
> > > >  Robert Dale
> > > > 
> > > >  On Wed, Feb 28, 2018 at 10:37 AM, Robert Dale <
> robd...@gmail.com>
> > > > >> wrote:
> > > > > I got it too. tp32: good.  tp33: good. master: bad.
> > > > >
> > > > > Robert Dale
> > > > >
> > > > > On Wed, Feb 28, 2018 at 10:20 AM, Stephen Mallette <
> > > > >>> spmalle...@gmail.com
> > > > > wrote:
> > > > >
> > > > >> I'm having no success generating docs with Docker. On master
> i'm
> > > >  currently
> > > > >> getting:
> > > > >>
> > > > >> Starting namenodes on [localhost]
> > > > >> localhost: Warning: Permanently added 'localhost' (ECDSA) to
> the
> > > > >> list
> > > > >>> of
> > > > >> known hosts.
> > > > >> localhost: starting namenode, logging to
> > > > >> /usr/local/lib/hadoop-2.7.2/logs/hadoop-root-namenode-
> > > > >>> 61cfd8f63c77.out
> > > > >> localhost: Warning: Permanently added 'localhost' (ECDSA) to
> the
> > > > >> list
> > > > >>> of
> > > > >> known hosts.
> > > > >> localhost: starting datanode, logging to
> > > > >> /usr/local/lib/hadoop-2.7.2/logs/hadoop-root-datanode-
> > > > >>> 61cfd8f63c77.out
> > > > >> Starting secondary namenodes [0.0.0.0]
> > > > >> 0.0.0.0: Warning: Permanently added

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386764#comment-16386764
 ] 

stephen mallette commented on TINKERPOP-1896:
-

cool - at least the groovy lambda works. there's been some competing issues 
with lambdas and i'm struggling to keep them all straight. thanks for 
confirming that.

when you add a lambda with the GLV, the traversal gets constructed through the 
scriptengine language that you choose to write the lambda. If you send Groovy, 
the {{GremlinGroovyScriptEngine}} puts the bytecode through a {{Translator}} 
which converts it to a Groovy script which evals to a {{Traversal}} that is 
then iterated out for the result on the server.

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

Re: [DISCUSS] Name of 3.4.x

2018-03-05 Thread Stephen Mallette 
So...we have a few good suggestions in hand (and I added one):

1. 4 Seasons of Gremlin - unclear if we're talking Boyz II Men or Vivaldi
here...
2. Four Rusted Gremlins - Marilyn Manson
3. Three Times a Gremlin - from the Commodores, though I think Gremlin
would prefer the Kenny Rogers styling of it - I mean, look at this
https://www.youtube.com/watch?v=Ok7becfXoJE :D

Sorry Jason, gotta disqualify your suggestion as it doesn't have a "number
reference" in it.

Anyone have any other ideas?



On Thu, Mar 1, 2018 at 8:33 AM, Jason Plurad  wrote:

> Boyz II Men - ha, I hadn't considered that angle. I assumed the 4 Seasons
> was a Vivaldi reference.
>
> How about David Bowie? The Gremlin Who Sold the World
> On Thu, Mar 1, 2018 at 8:06 AM Stephen Mallette 
> wrote:
>
> > I'm guessing at the reference for "4 Seasons of Gremlin" - are we talking
> > Boyz || Men and "4 Seasons of Loneliness"?
> >
> > >  Yeah, I'd like to see what Gremlyn Manson would look like!
> >
> > would he look that much different than our "Nine Inch Nails Gremlin"?
> >
> >
> >
> > On Tue, Feb 27, 2018 at 10:58 AM, Robert Dale  wrote:
> >
> > > Yeah, I'd like to see what Gremlyn Manson would look like!
> > >
> > > Robert Dale
> > >
> > > On Tue, Feb 27, 2018 at 10:40 AM, Daniel Kuppitz 
> > wrote:
> > >
> > > > Four Rusted Gremlins (reference
> > > > ) :)
> > > >
> > > > On Tue, Feb 27, 2018 at 7:47 AM, Stephen Mallette <
> > spmalle...@gmail.com>
> > > > wrote:
> > > >
> > > > > We need a release name and associated Gremlin logo for the 3.4.x
> > series
> > > > of
> > > > > TinkerPop. Recall that names should be related to music and
> include a
> > > > > number. Thus far we've had:
> > > > >
> > > > > * Gremlin Symphony #40 in G Minor (3.3.x)
> > > > > * Nine Inch Gremlins (3.2.x)
> > > > > * A 187 On The Undercover Gremlinz (3.1.x)
> > > > > * A Gremlin Rāga in 7/16 Time (3.0.x)
> > > > >
> > > > > Please submit a name with a logo idea - a name where we can't get a
> > > logo
> > > > > made doesn't really work too well. Thanks.
> > > > >
> > > >
> > >
> >
>

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread Branden Moore (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386752#comment-16386752
 ] 

Branden Moore commented on TINKERPOP-1896:
--

I can confirm that the groovy lambda works without DSL involvement.

 

I am the one with the server-side Java DSL.   This does appear to be the same 
issue I was having with the Gremlin-Groovy ScriptEngine.    I just wouldn't 
have expected that using a 'gremlin-groovy' lambda would cause my entire 
traversal to be executed via 'GremlinGroovyScriptEngine'.

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

[jira] [Closed] (TINKERPOP-1503) TraverserSet initial size should be specified for barriers.

2018-03-05 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1503.
---
Resolution: Won't Do

Closed per the attached DISCUSS thread.

> TraverserSet initial size should be specified for barriers.
> ---
>
> Key: TINKERPOP-1503
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1503
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: benchmark, process
>Affects Versions: 3.2.2
>Reporter: Marko A. Rodriguez
>Priority: Major
>
> {{TraverserSet}} is one of the most important objects in the Gremlin 
> traversal machine. Each step is backed by a {{TraverserSet}}. Typically, 
> steps will have only a few traversers ever in their set, but some steps can 
> have lots. Examples include: RepeatStep, NoOpBarrierStep, OrderGlobalStep, 
> DedupGlobalStep, SampleGlobalStep, LambdaCollectingBarrierStep...
> In particular, I'm looking at {{NoOpBarrierStep}} as that is used extensively 
> as an optimization by LazyBarrierStrategy, PathRetractionStrategy, 
> RepeatUnrollStrategy. If we already know the max size of the barrier, we 
> should be setting the "initial size" of the {{TraverserSet}} (which is 
> basically a wrapper around a {{LinkedHashMap}}). The default of 16 seems not 
> smart. Perhaps [~twilmes] or [~mbroecheler] have some thoughts on what the 
> initial size of the {{LinkedHashMap}} should be given a known 
> {{maxBarrierSize}}. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Re: [DISCUSS] Initial TraverserSet Size

2018-03-05 Thread Stephen Mallette 
I will just close this one as no one has provided any positive ideas on
making such a change. If someone thinks of something smart later we can
discuss/re-open.

On Fri, Mar 2, 2018 at 10:17 AM, Daniel Kuppitz  wrote:

> I would prefer to keep it like we have it now. A barrier size of 1000 mean
> doesn't necessarily mean, that we will get a lot of traversers. In fact, it
> will mostly be way fewer in a good traversal executed on a well-modeled
> graph. Thus I'd say let's better keep the memory footprint low. However, I
> might underestimate the cost of resizing, so this is purely based on my
> opinion / gut feeling, not on any kind of benchmarks.
>
> Cheers,
> Daniel
>
>
> On Thu, Mar 1, 2018 at 4:36 PM, Stephen Mallette 
> wrote:
>
> > I came across this JIRA:
> >
> > https://issues.apache.org/jira/browse/TINKERPOP-1503
> >
> > which wonders about the initial size of TraverserSet instances for a
> given
> > barrier size. Anyone have any opinion about how to size that?
> >
>

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386683#comment-16386683
 ] 

stephen mallette commented on TINKERPOP-1896:
-

Sorry, but do you mind backing up a bit? Can you first confirm if the "groovy" 
lambda works for you without a DSL involved?

If it does work, then I think you need to clarify this a bit:

> it appears that my anonymous traversal source has been reset from my DSL's 
> anonymous traversal source back to the default 
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.DefaultGraphTraversal,

Anonymous traversals are not spawned from {{DefaultGraphTraversal}} - they are 
spawned from the {{__}} class, so i don't completely follow what's happening to 
you. I also don't follow where you are seeing this error. You have a Python DSL 
that is misbehaving? or was this related to a server side Java DSL (I seem to 
recall someone doing that recently and i can't recall if it was you or not)?



> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-05 Thread Branden Moore (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386660#comment-16386660
 ] 

Branden Moore commented on TINKERPOP-1896:
--

Thanks Stephen,

  If I take my actual targeted traversal, and use a groovy-based lambda, all of 
a sudden, it appears that my anonymous traversal source has been reset from my 
DSL's anonymous traversal source back to the default 
org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.DefaultGraphTraversal, 
and so my custom steps are no longer usable.

  How does changing which engine executes the lambda affect the overall script 
engine?  What would be the expected outcome if a single traversal had multiple 
lambdas, using different target-languages?

 

> gremlin-python lambdas error when working on Maps
> -
>
> Key: TINKERPOP-1896
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1896
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.1
>Reporter: Branden Moore
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> Gremlin-python lambdas throw an error on the server when the preceding step 
> produces maps.
> {code}
> Traceback (most recent call last):
>   File "foo.py", line 15, in 
>     print g.V().has('name').match(__.as_('x').label().as_('lbl'), 
> __.as_('x').id().as_('id')).select('lbl', 'id').map(lambda: "lambda x: 
> type(x)").toList()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 52, in toList
>     return list(iter(self))
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 70, in next
>     return self.__next__()
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 43, in __next__
>     self.traversal_strategies.apply_strategies(self)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/process/traversal.py",
>  line 352, in apply_strategies
>     traversal_strategy.apply(traversal)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/remote_connection.py",
>  line 143, in apply
>     remote_traversal = self.remote_connection.submit(traversal.bytecode)
>   File 
> "/user/.local/lib/python2.7/site-packages/gremlin_python/driver/driver_remote_connection.py",
>  line 54, in submit
>     results = result_set.all().result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 462, in result
>     return self.__get_result()
>   File 
> "/user/.local/lib/python2.7/site-packages/concurrent/futures/_base.py", line 
> 414, in __get_result
>     raise exception_type, self._exception, self._traceback
> gremlin_python.driver.protocol.GremlinServerError: 599: AttributeError: type 
> object 'org.apache.tinkerpop.gremlin.process.traversal.dsl' has no attribute 
> 'as_' in

RE: [Discuss] Expose metadata from Gremlin Server to Clients.

2018-03-05 Thread Ashwini Singh 

To summarize what we have discussed so far:
1.  For API using GremlinRequest/QueryScript, expose response attribute 
as part of result. Using an approach to similar to Java client driver (using 
ResultSet) . [Priority0]
-- We rely on the last message for response attributes.
2. For GLV, add response attribute as part of Traversal. [Priority 0]
--Rely on the last message for attributes.
3. Expose other server details (like server setting).  I would suggest 
to split this design discussion into two directions: 
a. Metadata for request execution: Only focuses on details 
related to request execution. Can be achieved through #1 and #2.
b. Metadata for Gremlin Server:  Focuses on overall metadata 
for the server. Could be a separate request and fetch the data once for a 
connection. Any other suggestion ?

Please add if I am missing something.

Thanks a lot,
Ashwini Singh
Software Engineer @ Azure Cosmos DB

-Original Message-
From: Stephen Mallette  
Sent: Friday, March 2, 2018 4:49 AM
To: dev@tinkerpop.apache.org
Subject: Re: [Discuss] Expose metadata from Gremlin Server to Clients.

Perhaps another useful piece of data to return from Gremlin Server:

https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FTINKERPOP-1636=04%7C01%7Cashwsing%40microsoft.com%7C621c0a838bd24f13eb4e08d5803bed5d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636555917242398527%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D%7C-1=bVd71RV9uDiWWgsjY7SYAIYEhWtLdi7Ijgv9um%2F1JoM%3D=0

On Thu, Mar 1, 2018 at 6:31 PM, Stephen Mallette 
wrote:

> I just came across this:
>
> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fissue
> s.apache.org%2Fjira%2Fbrowse%2FTINKERPOP-1494=04%7C01%7Cashwsing%
> 40microsoft.com%7C621c0a838bd24f13eb4e08d5803bed5d%7C72f988bf86f141af9
> 1ab2d7cd011db47%7C1%7C0%7C636555917242398527%7CUnknown%7CTWFpbGZsb3d8e
> yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D%7C-1
> =1kY8r73w2Y3euQVnLEGIuAO0fX6ZySOzl%2BVUgbhodpE%3D=0
>
> There's some ideas for what the open source Gremlin Server could 
> return therefor example, perhaps we send back the the host that 
> executed the script as the metadata for Gremlin Server?
>
> On Wed, Feb 28, 2018 at 5:37 PM, Ashwini Singh  invalid> wrote:
>
>>
>> Completely agree on making the change for other drivers too. The plan 
>> from our side is to make changes to other drivers as well.  My focus 
>> on Gremlin.Net was just for an example . Yes, we should make change 
>> to Gremlin Server to write the metadata for testing.
>>
>> I agree to adding this to ResultSet and exposing this as part of 
>> every response for all the drivers and follow the same pattern as JAVA.
>>
>> On the gremlin GVL part,  I agree we should support this on traversal 
>> for bytecode as well. For now, we do not have support for bytecode 
>> but we are actively working on that and need support for metadata there as 
>> well.
>> Thanks for bringing this up. Just wanted to understand how we track 
>> issues at tinkerpop, Can we spilt the change to support for gremlin 
>> request/script first and bytecode/traversal separately? If we prefer 
>> to split the change in chunks, but completely fine with one change.
>>
>> Thanks,
>> Ashwini Singh
>> Software Engineer @ Azure Cosmos DB
>>
>> -Original Message-
>> From: Stephen Mallette 
>> Sent: Wednesday, February 28, 2018 9:58 AM
>> To: dev@tinkerpop.apache.org
>> Subject: Re: [Discuss] Expose metadata from Gremlin Server to Clients.
>>
>> This newly created issue might be related to this in some way:
>>
>> https://na01.safelinks.protection.outlook.com/?url=https%3A%
>> 2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FTINKERPOP-1906&
>> data=04%7C01%7Cashwsing%40microsoft.com%7C614b32ff7f98
>> 4c66554808d57ed4d22b%7C72f988bf86f141af91ab2d7cd011db47%7C1%
>> 7C0%7C636554374885412678%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4
>> wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D%7C-1&
>> sdata=w7pyg6JF4KMbPdpBg6t%2F8cE%2BflbnCQQdyMUgc6HDrXw%3D=0
>>
>> On Wed, Feb 28, 2018 at 11:37 AM, Florian Hockmann < 
>> f...@florian-hockmann.de>
>> wrote:
>>
>> > I agree that we also should make this metadata available for 
>> > traversals since we want to move users away from sending Gremlin 
>> > scripts as strings and instead use Bytecode based GLVs.
>> >
>> > Regarding Gremlin.Net: I think that the implementation would be 
>> > very similar to how it can be implemented in the Java driver as we 
>> > tried to stay close to the Java driver in general. The only 
>> > difference is probably that we currently don't have a ResultSet in 
>> > Gremlin.Net, but that's only because I didn't see much value in 
>> > adding that. Metadata would of course be a good argument to also 
>> > implement a

[jira] [Commented] (TINKERPOP-1897) Provide Docker images of Gremlin Server and Console

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386574#comment-16386574
 ] 

ASF GitHub Bot commented on TINKERPOP-1897:
---

Github user FlorianHockmann commented on the issue:

https://github.com/apache/tinkerpop/pull/802
  
I just verified my assumption that `docker login` should be enough to get 
deployment of Docker images working through Maven and added a short description 
of that to the dev docs.

> if i understand you correctly, you're saying that if you push the image 
to 3.3.2-SNAPSHOT and then make changes and push again, the 3.3.2-SNAPSHOT 
would update...is that right? if that is the case, then that behavior seems to 
match the nature of a java "SNAPSHOT" (except that there is no archival of the 
older SNAPSHOT).

Yes, exactly. Changing something about the image means that the ID will 
change and the tag will be added to the new image. So the old one won't be 
available any more through Docker Hub as it isn't tagged any more. That way, we 
only have one image per tag available and it will always be the most recent one.

>  Either way though, I don't think we need to pollute the TinkerPop 
dockerhub with SNAPSHOT deployments. I think we should look for ways to disable 
the docker plugin on deploy if it is a SNAPSHOT version.

I agree, I just don't know how this could be done. The plugin [contains 
`skip` 
flags](https://github.com/spotify/dockerfile-maven#skip-docker-goals-bound-to-maven-phases),
 but we have to activate them somehow for SNAPSHOT versions.


> Provide Docker images of Gremlin Server and Console
> ---
>
> Key: TINKERPOP-1897
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1897
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: console, server
>Affects Versions: 3.2.7
>Reporter: Florian Hockmann
>Assignee: Florian Hockmann
>Priority: Major
>
> TinkerPop should provide Docker images of Gremlin Server and Gremlin Console 
> that are deployed together with each release.
> This originated from the mailing list:
> [https://lists.apache.org/thread.html/744ae19afa9b2fd1984c1e4dddc588e98786d9c21b633aab8bfa@%3Cdev.tinkerpop.apache.org%3E]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] tinkerpop issue #802: Add docker images for console and server TINKERPOP-189...

2018-03-05 Thread FlorianHockmann 
Github user FlorianHockmann commented on the issue:

https://github.com/apache/tinkerpop/pull/802
  
I just verified my assumption that `docker login` should be enough to get 
deployment of Docker images working through Maven and added a short description 
of that to the dev docs.

> if i understand you correctly, you're saying that if you push the image 
to 3.3.2-SNAPSHOT and then make changes and push again, the 3.3.2-SNAPSHOT 
would update...is that right? if that is the case, then that behavior seems to 
match the nature of a java "SNAPSHOT" (except that there is no archival of the 
older SNAPSHOT).

Yes, exactly. Changing something about the image means that the ID will 
change and the tag will be added to the new image. So the old one won't be 
available any more through Docker Hub as it isn't tagged any more. That way, we 
only have one image per tag available and it will always be the most recent one.

>  Either way though, I don't think we need to pollute the TinkerPop 
dockerhub with SNAPSHOT deployments. I think we should look for ways to disable 
the docker plugin on deploy if it is a SNAPSHOT version.

I agree, I just don't know how this could be done. The plugin [contains 
`skip` 
flags](https://github.com/spotify/dockerfile-maven#skip-docker-goals-bound-to-maven-phases),
 but we have to activate them somehow for SNAPSHOT versions.


---

[jira] [Commented] (TINKERPOP-1738) Proper functioning of GraphSONReader depends on order of elements in String representation

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386250#comment-16386250
 ] 

ASF GitHub Bot commented on TINKERPOP-1738:
---

Github user TomCools commented on the issue:

https://github.com/apache/tinkerpop/pull/807
  
VOTE: +1 same issue as Joengenduvel. OWASP scan is required for our project 
and this is listed as a Major level issue.


> Proper functioning of GraphSONReader depends on order of elements in String 
> representation
> --
>
> Key: TINKERPOP-1738
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1738
> Project: TinkerPop
>  Issue Type: Bug
>  Components: io
>Affects Versions: 3.3.0
>Reporter: Daniel Kuppitz
>Assignee: stephen mallette
>Priority: Major
>
> The following two snippets are basically using the same stringified JSON 
> object, it's just the order of elements that is different.
> {noformat}
> gremlin> GraphSONReader.build().create().readObject(new 
> java.io.ByteArrayInputStream('{"@value":{"step":[["addV","poc_int"],["property","bigint1value",{"@value":-4294967295,"@type":"g:Int32"}]]},"@type":"g:Bytecode"}'.getBytes()),
>  Bytecode.class)
> ==>[[], [addV(poc_int), property(bigint1value, 1)]]
> {noformat}
> {noformat}
> gremlin> GraphSONReader.build().create().readObject(new 
> java.io.ByteArrayInputStream('{"@type":"g:Bytecode","@value":{"step":[["addV","poc_int"],["property","bigint1value",{"@type":"g:Int32","@value":-4294967295}]]}}'.getBytes()),
>  Bytecode.class)
> Could not deserialize the JSON value as required. Nested exception: 
> org.apache.tinkerpop.shaded.jackson.databind.JsonMappingException: Could not 
> deserialize the JSON value as required. Nested exception: 
> org.apache.tinkerpop.shaded.jackson.core.JsonParseException: Numeric value 
> (-4294967295) out of range of int
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 126]
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 115] 
> (through reference chain: java.util.ArrayList[1]->java.util.ArrayList[2])
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 115]
> Type ':help' or ':h' for help.
> Display stack trace? [yN]
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] tinkerpop issue #807: TINKERPOP-1738 Bump to jackson 2.9.4

2018-03-05 Thread TomCools 
Github user TomCools commented on the issue:

https://github.com/apache/tinkerpop/pull/807
  
VOTE: +1 same issue as Joengenduvel. OWASP scan is required for our project 
and this is listed as a Major level issue.


---

[jira] [Commented] (TINKERPOP-1738) Proper functioning of GraphSONReader depends on order of elements in String representation

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386240#comment-16386240
 ] 

ASF GitHub Bot commented on TINKERPOP-1738:
---

Github user Joengenduvel commented on the issue:

https://github.com/apache/tinkerpop/pull/807
  
VOTE: +1
We cannot proceed using Jackson 2.9.3 and below: 
https://nvd.nist.gov/vuln/detail/CVE-2017-17485 


> Proper functioning of GraphSONReader depends on order of elements in String 
> representation
> --
>
> Key: TINKERPOP-1738
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1738
> Project: TinkerPop
>  Issue Type: Bug
>  Components: io
>Affects Versions: 3.3.0
>Reporter: Daniel Kuppitz
>Assignee: stephen mallette
>Priority: Major
>
> The following two snippets are basically using the same stringified JSON 
> object, it's just the order of elements that is different.
> {noformat}
> gremlin> GraphSONReader.build().create().readObject(new 
> java.io.ByteArrayInputStream('{"@value":{"step":[["addV","poc_int"],["property","bigint1value",{"@value":-4294967295,"@type":"g:Int32"}]]},"@type":"g:Bytecode"}'.getBytes()),
>  Bytecode.class)
> ==>[[], [addV(poc_int), property(bigint1value, 1)]]
> {noformat}
> {noformat}
> gremlin> GraphSONReader.build().create().readObject(new 
> java.io.ByteArrayInputStream('{"@type":"g:Bytecode","@value":{"step":[["addV","poc_int"],["property","bigint1value",{"@type":"g:Int32","@value":-4294967295}]]}}'.getBytes()),
>  Bytecode.class)
> Could not deserialize the JSON value as required. Nested exception: 
> org.apache.tinkerpop.shaded.jackson.databind.JsonMappingException: Could not 
> deserialize the JSON value as required. Nested exception: 
> org.apache.tinkerpop.shaded.jackson.core.JsonParseException: Numeric value 
> (-4294967295) out of range of int
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 126]
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 115] 
> (through reference chain: java.util.ArrayList[1]->java.util.ArrayList[2])
>  at [Source: java.io.ByteArrayInputStream@6d367020; line: 1, column: 115]
> Type ':help' or ':h' for help.
> Display stack trace? [yN]
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] tinkerpop issue #807: TINKERPOP-1738 Bump to jackson 2.9.4

2018-03-05 Thread Joengenduvel 
Github user Joengenduvel commented on the issue:

https://github.com/apache/tinkerpop/pull/807
  
VOTE: +1
We cannot proceed using Jackson 2.9.3 and below: 
https://nvd.nist.gov/vuln/detail/CVE-2017-17485 


---

[jira] [Closed] (TINKERPOP-1862) TinkerGraph VertexProgram message passing doesn't work properly when using Direction.BOTH

2018-03-05 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1862?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1862.
---
   Resolution: Fixed
 Assignee: stephen mallette
Fix Version/s: 3.3.2
   3.2.8

> TinkerGraph VertexProgram message passing doesn't work properly when using 
> Direction.BOTH
> -
>
> Key: TINKERPOP-1862
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1862
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process, tinkergraph
>Affects Versions: 3.2.7
>Reporter: Philip Graff
>Assignee: stephen mallette
>Priority: Major
> Fix For: 3.2.8, 3.3.2
>
>
> I think the messages are being sent properly in TinkerMessenger, but when I 
> call messenger.receiveMessages(), the vertex is getting messages from the 
> outVertex of their edges regardless of the edge direction. This is due to 
> line 71 of TinkerMessenger (linked below) which calls 
> Edge.vertices(direction).next(), thus getting the first result out of the 
> Vertex iterator. For IN or OUT, this isn't a problem. But for BOTH, following 
> line 124 of TinkerEdge (linked below), the outVertex is always returned 
> first. TinkerMessenger needs to be modified to return the correct vertex (I 
> think it's the one that != this.vertex).
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/process/computer/TinkerMessenger.java#L71
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerEdge.java#L124



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (TINKERPOP-1862) TinkerGraph VertexProgram message passing doesn't work properly when using Direction.BOTH

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386104#comment-16386104
 ] 

ASF GitHub Bot commented on TINKERPOP-1862:
---

Github user asfgit closed the pull request at:

https://github.com/apache/tinkerpop/pull/804


> TinkerGraph VertexProgram message passing doesn't work properly when using 
> Direction.BOTH
> -
>
> Key: TINKERPOP-1862
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1862
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process, tinkergraph
>Affects Versions: 3.2.7
>Reporter: Philip Graff
>Priority: Major
>
> I think the messages are being sent properly in TinkerMessenger, but when I 
> call messenger.receiveMessages(), the vertex is getting messages from the 
> outVertex of their edges regardless of the edge direction. This is due to 
> line 71 of TinkerMessenger (linked below) which calls 
> Edge.vertices(direction).next(), thus getting the first result out of the 
> Vertex iterator. For IN or OUT, this isn't a problem. But for BOTH, following 
> line 124 of TinkerEdge (linked below), the outVertex is always returned 
> first. TinkerMessenger needs to be modified to return the correct vertex (I 
> think it's the one that != this.vertex).
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/process/computer/TinkerMessenger.java#L71
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerEdge.java#L124



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] tinkerpop pull request #804: TINKERPOP-1862 Messenger proper handling of Dir...

2018-03-05 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/tinkerpop/pull/804


---

[jira] [Commented] (TINKERPOP-1862) TinkerGraph VertexProgram message passing doesn't work properly when using Direction.BOTH

2018-03-05 Thread ASF GitHub Bot (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386085#comment-16386085
 ] 

ASF GitHub Bot commented on TINKERPOP-1862:
---

Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/804
  
VOTE +1


> TinkerGraph VertexProgram message passing doesn't work properly when using 
> Direction.BOTH
> -
>
> Key: TINKERPOP-1862
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1862
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process, tinkergraph
>Affects Versions: 3.2.7
>Reporter: Philip Graff
>Priority: Major
>
> I think the messages are being sent properly in TinkerMessenger, but when I 
> call messenger.receiveMessages(), the vertex is getting messages from the 
> outVertex of their edges regardless of the edge direction. This is due to 
> line 71 of TinkerMessenger (linked below) which calls 
> Edge.vertices(direction).next(), thus getting the first result out of the 
> Vertex iterator. For IN or OUT, this isn't a problem. But for BOTH, following 
> line 124 of TinkerEdge (linked below), the outVertex is always returned 
> first. TinkerMessenger needs to be modified to return the correct vertex (I 
> think it's the one that != this.vertex).
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/process/computer/TinkerMessenger.java#L71
> https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerEdge.java#L124



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] tinkerpop issue #804: TINKERPOP-1862 Messenger proper handling of Direction....

2018-03-05 Thread robertdale 
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/804
  
VOTE +1


---

[jira] [Commented] (TINKERPOP-1909) Gremlin.Net does not have complete object model as compared to other client drivers and unable to de-serialize properties for vertex/edge graphSON.

2018-03-05 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386048#comment-16386048
 ] 

stephen mallette commented on TINKERPOP-1909:
-

I'm not completely for or against any particular ideas for TinkerPop 4.x at 
this point as we're largely just collecting ideas on how that version should be 
shaped. I'd established this document for that purpose:

http://tinkerpop.apache.org/docs/current/dev/future/

It has no really set format at this point - just a scratchpad. You can submit 
pull requests with thoughts if you like:

https://github.com/apache/tinkerpop/blob/master/docs/src/dev/future/index.asciidoc

Regarding the options you present:

1. As to your first suggestionI prefer the idea of just reference objects 
(i.e. id/label) for a variety of reasons, but going back to my initial comment, 
I think we can only do that reasonably if we have a method to properly handle 
remote transactions that doesn't require the full object model on the client. 
Of course, it seems to be possible to envision TINKERPOP-1417 without a full 
object model given the design marko put forward in the "future" doc. In fact, 
that works quite well in my mind now as I type this. It would truly be a 
unified way to do Gremlin. Whether you are talking to a local subgraph or a 
remote multi-billion edge graph your results are always just a "reference". The 
part that seems to fail would be in the gathering of an initial subgraph for a 
transaction. Somehow that local subgraph would have to be populated in an 
efficient fashion.

2. As to your second suggestionI think that an extension model could make 
sense. That would allow TinkerPop to stay on the track for "reference" objects 
but allow graph providers and users to extend that model. We've tried to offer 
such a model already with the {{IoRegistry}} system in java, but I'm not sure 
it has this entire use case in mind thought through and I'm not sure it follows 
through well to all the GLVs, like .NET. 

Anyway, I think that we should move any further 4.x discussion to the "future" 
doc. I'll leave this ticket open for a bit to see if you have any reply before 
I grab any key points from here and move them over there. Thanks for your input.



> Gremlin.Net does not have complete object model as compared to other client 
> drivers and unable to de-serialize properties for vertex/edge graphSON.  
> -
>
> Key: TINKERPOP-1909
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1909
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.3.1
>Reporter: Ashwini Singh
>Priority: Major
>
> Looks like the object model for Gremlin.Net client driver is not as par with 
> Java driver. We cannot deserialize a GraphSON response to tinkerpop object 
> completely. For example, Gremlin.Net object model cannot deserialize 
> properties from a graphSON response object (vertex/edges). Currently, we only 
> deserialize id and label field from a vertex/edge graphSON.
>  
> So, to desterilize the object model, users have to write a custom 
> deserialization code and create the object. The current de-serializers (for 
> vertex/edge) will strip off details like properties.
>  
> I am filing it as a bug but it could fall into improvement as well.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)