[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/467
  
pretty: VOTE +1 - give the last comment @okram laid out there it sounds 
like he will handle the merge now that you have 3X +1.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread seanbarzilay 
Github user seanbarzilay commented on the issue:

https://github.com/apache/tinkerpop/pull/467
  
@spmallette done


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [TinkerPop] New Committer: Robert Dale

2016-10-27 Thread Jason Plurad 
+1!

-- Jason
On Thu, Oct 27, 2016 at 8:17 AM Jean-Baptiste Musso 
wrote:

> Welcome aboard Robert!
>
> On Thursday, 27 October 2016, Stephen Mallette 
> wrote:
>
> The Project Management Committee (PMC) for Apache TinkerPop has asked
> Robert Dale to become a committer and we are pleased to announce that he
> has accepted.
>
> Robert has submitted a number of quality pull requests to TinkerPop
> covering both new features and bug/documentation fixes and has been engaged
> in many different development discussions that have shaped the direction
> TinkerPop has taken. He's also been helpful on the user mailing list,
> providing some nice answers to those who have needed support. All of that
> activity made him a sound choice for committership.
>
> Let this proclamation traverse the entirety of The TinkerPop so that all
> may know to welcome Robert to the TinkerPop Guild!
>
>
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/gremlin-users/CAA-H43-DpGHAAxxgb6ib-qXKML-etQnP%2BRP5jfeJm4hRGrt_vg%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>
>
>
> --
> Jean-Baptiste
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/gremlin-users/CAC3-3jirKkyn%3DByYC%3DROdEr2dLJnm-Cb9KC25RpS%2BnByDvn2%2BQ%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>

[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/467
  
@seanbarzilay do you mind cleaning up the commit history a bit? It would be 
nice to have one commit that we can merge for this small change.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #458: Message scope initialization in PeerPressureVertexProg...

2016-10-27 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/458
  
All tests pass with `docker/build.sh -t -n -i`

VOTE +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Commented] (TINKERPOP-833) Remove deprecated GremlinGroovyScriptEngine constructor and plugins()

2016-10-27 Thread stephen mallette (JIRA) 

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

stephen mallette commented on TINKERPOP-833:


Work for this was done on TINKERPOP-1420

> Remove deprecated GremlinGroovyScriptEngine constructor and plugins()
> -
>
> Key: TINKERPOP-833
> URL: https://issues.apache.org/jira/browse/TINKERPOP-833
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: groovy
>Affects Versions: 3.0.2-incubating
>Reporter: stephen mallette
>Priority: Minor
>  Labels: breaking, deprecation
>
> The removal of this constructor and {{plugins()}} method is relatively 
> low-risk as {{GremlinGroovyScriptEngine}} is more of an internally used class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-1420) Remove deprecated ConcurrentBindings in gremlin-groovy

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

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

GitHub user spmallette opened a pull request:

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

TINKERPOP-1420 Removal of previously deprecated gremlin-groovy 
classes/methods

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

While this commit is bound to TINKERPOP-1420 it actually also covers 
TINKERPOP-1283, TINKERPOP-1040 and TINKERPOP-833 which are all "deprecation 
removal" type tickets. The removed class/methods are all pretty "low-level" for 
the average user to be using in their code so of all the deprecation we have 
this one isn't a big deal for removal.

Builds nicely with `mvn clean install && mvn verify -pl gremlin-server 
-DskipIntegrationTests=false`

VOTE +1

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

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

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

https://github.com/apache/tinkerpop/pull/468.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 #468


commit 6c72a9684a8db6b189970d89bca42c7275782019
Author: Stephen Mallette 
Date:   2016-10-27T16:22:49Z

TINKERPOP-1420 Removal of previously deprecated gremlin-groovy 
classes/methods.

While this commit is bound to TINKERPOP-1420 it actually also covers 
TINKERPOP-1283, TINKERPOP-1040 and TINKERPOP-833.




> Remove deprecated ConcurrentBindings in gremlin-groovy
> --
>
> Key: TINKERPOP-1420
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1420
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: groovy
>Affects Versions: 3.2.1
>Reporter: stephen mallette
>Priority: Minor
>  Labels: breaking, deprecation
>
> {{ConcurrentBindings}} were moved to to {{gremlin-core}} - the old class in 
> {{gremlin-groovy}} was deprecated and can be removed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] tinkerpop pull request #468: TINKERPOP-1420 Removal of previously deprecated...

2016-10-27 Thread spmallette 
GitHub user spmallette opened a pull request:

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

TINKERPOP-1420 Removal of previously deprecated gremlin-groovy 
classes/methods

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

While this commit is bound to TINKERPOP-1420 it actually also covers 
TINKERPOP-1283, TINKERPOP-1040 and TINKERPOP-833 which are all "deprecation 
removal" type tickets. The removed class/methods are all pretty "low-level" for 
the average user to be using in their code so of all the deprecation we have 
this one isn't a big deal for removal.

Builds nicely with `mvn clean install && mvn verify -pl gremlin-server 
-DskipIntegrationTests=false`

VOTE +1

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

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

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

https://github.com/apache/tinkerpop/pull/468.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 #468


commit 6c72a9684a8db6b189970d89bca42c7275782019
Author: Stephen Mallette 
Date:   2016-10-27T16:22:49Z

TINKERPOP-1420 Removal of previously deprecated gremlin-groovy 
classes/methods.

While this commit is bound to TINKERPOP-1420 it actually also covers 
TINKERPOP-1283, TINKERPOP-1040 and TINKERPOP-833.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
LGTM


> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
@spmallette I pushed the windows changes. can you review?


> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread robertdale 
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
@spmallette I pushed the windows changes. can you review?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
Tested all the command line options and all work nicely. Good documentation.

It did occur to me that we don't have analogous features for windows in 
that the gremlin-server.sh does not behave like the gremlin-server.bat. I'd 
suggest two things:

1.  Maybe just add a "WARNING:" to the docs to mention that discrepancy and 
that gremlin-server.bat always starts Gremlin Server in the foreground.
2. Change gremlin-server.bat to use "install" instead of "-i"

Other than that, looks good to me:

VOTE +1


> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Re: [TinkerPop] New Committer: Robert Dale

2016-10-27 Thread Jean-Baptiste Musso 
Welcome aboard Robert!

On Thursday, 27 October 2016, Stephen Mallette  wrote:

> The Project Management Committee (PMC) for Apache TinkerPop has asked
> Robert Dale to become a committer and we are pleased to announce that he
> has accepted.
>
> Robert has submitted a number of quality pull requests to TinkerPop
> covering both new features and bug/documentation fixes and has been engaged
> in many different development discussions that have shaped the direction
> TinkerPop has taken. He's also been helpful on the user mailing list,
> providing some nice answers to those who have needed support. All of that
> activity made him a sound choice for committership.
>
> Let this proclamation traverse the entirety of The TinkerPop so that all
> may know to welcome Robert to the TinkerPop Guild!
>
>
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com
> 
> .
> To view this discussion on the web visit https://groups.google.com/d/
> msgid/gremlin-users/CAA-H43-DpGHAAxxgb6ib-qXKML-etQnP%
> 2BRP5jfeJm4hRGrt_vg%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>


-- 
Jean-Baptiste

[jira] [Created] (TINKERPOP-1533) Storage and IoRegistry

2016-10-27 Thread Marko A. Rodriguez (JIRA) 
Marko A. Rodriguez created TINKERPOP-1533:
-

 Summary: Storage and IoRegistry
 Key: TINKERPOP-1533
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1533
 Project: TinkerPop
  Issue Type: Bug
  Components: hadoop, io, structure
Affects Versions: 3.2.3, 3.1.4
Reporter: Marko A. Rodriguez
 Fix For: 3.3.0


If you have custom types and you use storage, then you will get failures.

{code}
spark.head("output", GryoInputFormat.class)
hdfs.head("output", GraphSONInputFormat.class)
{code}

You can see why -- we don't pass any configuration information for the 
IoRegistry. Dar



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
Rebased. Retested.

VOTE +1



> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Re: [TinkerPop] TinkerPop Web Site in Git Repo

2016-10-27 Thread Robert Dale 
Yay!

Robert Dale

On Thu, Oct 27, 2016 at 10:00 AM, Stephen Mallette 
wrote:

> We can now accept changes for the web site (like the TinkerPop home page
> at http://tinkerpop.apache.org) via the standard pull request model
> because the source for those pages has been migrated to git. You can find
> the source here:
>
> https://github.com/apache/tinkerpop/tree/e5f2f6dd0f6ccc8b73929b85942ab3
> a024321fd0/docs/site/home
>
> Hopefully, having the source in git will make it easier for folks to
> submit fixes and such.
>
> Thanks,
>
> Stephen
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/
> msgid/gremlin-users/CAA-H438RYoCOv5OUbksw5WXnn-p-
> 33mxJjOUmd_1JXyOrNtuxQ%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>

TinkerPop Web Site in Git Repo

2016-10-27 Thread Stephen Mallette 
We can now accept changes for the web site (like the TinkerPop home page at
http://tinkerpop.apache.org) via the standard pull request model because
the source for those pages has been migrated to git. You can find the
source here:

https://github.com/apache/tinkerpop/tree/e5f2f6dd0f6ccc8b73929b85942ab3a024321fd0/docs/site/home

Hopefully, having the source in git will make it easier for folks to submit
fixes and such.

Thanks,

Stephen

[jira] [Commented] (TINKERPOP-1485) Move source for TinkerPop site to source code repo

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

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

Github user asfgit closed the pull request at:

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


> Move source for TinkerPop site to source code repo
> --
>
> Key: TINKERPOP-1485
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1485
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.2
>Reporter: stephen mallette
>Assignee: Daniel Kuppitz
> Fix For: 3.3.0
>
>
> Some time ago there was discussion on the Mailing List to move the web site 
> to the source code repo so that it could be contributed to via pull request. 
> It would be nice because then we could better rely on some basic static site 
> generation to help deal with repetitive and sometimes error prone process of 
> updating the header for the pages.
> Anyway, a few thoughts:
> 1. need a shell script to publish the site to svn - something like 
> {{bin/publish-site.sh}}
> 2. place files in the root of the project in a directory called {{site/}}
> 3. if possible, do some basic site generation with maybe some groovy script 
> or just the shell script mentioned above to inject a "header file" for each 
> page.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Closed] (TINKERPOP-1485) Move source for TinkerPop site to source code repo

2016-10-27 Thread stephen mallette (JIRA) 

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

stephen mallette closed TINKERPOP-1485.
---
Resolution: Done

> Move source for TinkerPop site to source code repo
> --
>
> Key: TINKERPOP-1485
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1485
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.2
>Reporter: stephen mallette
>Assignee: Daniel Kuppitz
> Fix For: 3.3.0
>
>
> Some time ago there was discussion on the Mailing List to move the web site 
> to the source code repo so that it could be contributed to via pull request. 
> It would be nice because then we could better rely on some basic static site 
> generation to help deal with repetitive and sometimes error prone process of 
> updating the header for the pages.
> Anyway, a few thoughts:
> 1. need a shell script to publish the site to svn - something like 
> {{bin/publish-site.sh}}
> 2. place files in the root of the project in a directory called {{site/}}
> 3. if possible, do some basic site generation with maybe some groovy script 
> or just the shell script mentioned above to inject a "header file" for each 
> page.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Closed] (TINKERPOP-1468) GraphTraversal.optional throws a NPE if no traversal is provided

2016-10-27 Thread Ted Wilmes (JIRA) 

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

Ted Wilmes closed TINKERPOP-1468.
-
Resolution: Won't Fix
  Assignee: Ted Wilmes

Since this is related to Groovy and a 0 arg method call (see 
[here|https://objectpartners.com/2015/09/02/groovy-gotcha-passing-zero-arguments-to-a-method-that-expects-one/]
 for details) we are closing this as "won't fix".

> GraphTraversal.optional throws a NPE if no traversal is provided
> 
>
> Key: TINKERPOP-1468
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1468
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.2
>Reporter: Ted Wilmes
>Assignee: Ted Wilmes
>  Labels: trivial
>
> {{GraphTraversal.optional}} throws a NPE if no traversal is provided.  We 
> should probably throw a friendlier exception with instructions to provide a 
> traversal as an argument.
> {code}
> gremlin> g.V().optional()
> java.lang.NullPointerException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.lang.NullPointerException
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.optional(GraphTraversal.java:1316)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] tinkerpop pull request #459: TINKERPOP-1485 Move source for TinkerPop site t...

2016-10-27 Thread asfgit 
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (TINKERPOP-1532) Make use of KryoPool

2016-10-27 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1532:
---

 Summary: Make use of KryoPool
 Key: TINKERPOP-1532
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1532
 Project: TinkerPop
  Issue Type: Improvement
  Components: io
Affects Versions: 3.2.3
Reporter: stephen mallette
Priority: Minor


Recent version of Kryo have {{KryoPool}} which basically has the same features 
a {{GryoPool}}. We should look to deprecate {{GryoPool}} and use {{KryoPool}} 
if possible.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Re: New Committer: Robert Dale

2016-10-27 Thread Ted Wilmes 
Welcome Robert!

--Ted

On Thu, Oct 27, 2016 at 8:30 AM, Stephen Mallette 
wrote:

> The Project Management Committee (PMC) for Apache TinkerPop has asked
> Robert Dale to become a committer and we are pleased to announce that he
> has accepted.
>
> Robert has submitted a number of quality pull requests to TinkerPop
> covering both new features and bug/documentation fixes and has been engaged
> in many different development discussions that have shaped the direction
> TinkerPop has taken. He's also been helpful on the user mailing list,
> providing some nice answers to those who have needed support. All of that
> activity made him a sound choice for committership.
>
> Let this proclamation traverse the entirety of The TinkerPop so that all
> may know to welcome Robert to the TinkerPop Guild!
>

[jira] [Commented] (TINKERPOP-1468) GraphTraversal.optional throws a NPE if no traversal is provided

2016-10-27 Thread Ted Wilmes (JIRA) 

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

Ted Wilmes commented on TINKERPOP-1468:
---

I haven't kept up as much with the GLV development, but thinking through that 
case where at some future point, I would think many
folks would not be running an embedded graph db and instead would be using a 
client that would be passing Gremlin byte code, which
would then be reconstructed on the server side, we'd probably avoid this issue 
anyway or maybe catch it with byte code level validation?

Regardless, I'm okay with closing it "won't fix".

> GraphTraversal.optional throws a NPE if no traversal is provided
> 
>
> Key: TINKERPOP-1468
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1468
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.2
>Reporter: Ted Wilmes
>  Labels: trivial
>
> {{GraphTraversal.optional}} throws a NPE if no traversal is provided.  We 
> should probably throw a friendlier exception with instructions to provide a 
> traversal as an argument.
> {code}
> gremlin> g.V().optional()
> java.lang.NullPointerException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.lang.NullPointerException
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.optional(GraphTraversal.java:1316)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

New Committer: Robert Dale

2016-10-27 Thread Stephen Mallette 
The Project Management Committee (PMC) for Apache TinkerPop has asked
Robert Dale to become a committer and we are pleased to announce that he
has accepted.

Robert has submitted a number of quality pull requests to TinkerPop
covering both new features and bug/documentation fixes and has been engaged
in many different development discussions that have shaped the direction
TinkerPop has taken. He's also been helpful on the user mailing list,
providing some nice answers to those who have needed support. All of that
activity made him a sound choice for committership.

Let this proclamation traverse the entirety of The TinkerPop so that all
may know to welcome Robert to the TinkerPop Guild!

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
@PommeVerte  do you mind if we move the ubuntu script to a separate pull 
request so this one can move forward?


> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-980) Add a service script or daemon mode in the distribution

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on TINKERPOP-980:
--

Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
I believe @PommeVerte is on a month long holiday at this point. imo, i 
think we could move the ubuntu script off to a different ticket as you have a 
lot of work in place here. I would just rebase on master, move your 
changelog/upgrade docs stuff to be under 3.3.0 and we can move to vote. 

And of course, now that you're a committer @robertdale you should VOTE on 
your own stuff for a free +1 :)


> Add a service script or daemon mode in the distribution
> ---
>
> Key: TINKERPOP-980
> URL: https://issues.apache.org/jira/browse/TINKERPOP-980
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.0.2-incubating
>Reporter: Jeremy Hanna
>Assignee: Dylan Millikin
>Priority: Minor
>  Labels: breaking
>
> Based on this discussion, it looks like there was an example from [~dkuppitz] 
> on how to create a gremlin server service on linux:
> https://groups.google.com/forum/#!msg/gremlin-users/uA48IQ3YJcw/4KnUKIS8HI4J
> Here is a link to the gist for the service:
> https://gist.github.com/dkuppitz/20bda51e3465a612cd9b
> I think it would be great to include this or a way to daemonize the server 
> into the tinkerpop distribution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread robertdale 
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/439
  
@PommeVerte  do you mind if we move the ubuntu script to a separate pull 
request so this one can move forward?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Commented] (TINKERPOP-1468) GraphTraversal.optional throws a NPE if no traversal is provided

2016-10-27 Thread Robert Dale (JIRA) 

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

Robert Dale commented on TINKERPOP-1468:


Then again, you do have the stacktrace pointing right to the optional() method. 
I think the next logical step is to check the API. The API clearly shows that 
it takes a Traversal parameter. I hope at that point it's obvious what needs to 
be done. It may not be user friendly in the sense of hand-holding, but it's a 
lot better than most SQL error messages.  So, yes, I would close as "won't fix".

> GraphTraversal.optional throws a NPE if no traversal is provided
> 
>
> Key: TINKERPOP-1468
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1468
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.2
>Reporter: Ted Wilmes
>  Labels: trivial
>
> {{GraphTraversal.optional}} throws a NPE if no traversal is provided.  We 
> should probably throw a friendlier exception with instructions to provide a 
> traversal as an argument.
> {code}
> gremlin> g.V().optional()
> java.lang.NullPointerException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.lang.NullPointerException
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.optional(GraphTraversal.java:1316)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-1485) Move source for TinkerPop site to source code repo

2016-10-27 Thread ASF GitHub Bot (JIRA) 

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

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

Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/459
  
i had thought about putting it under `/docs` but it didn't seem to fit 
there as the "site" really included "docs" and not the other way around. i 
didn't want to up-end everything and move `/docs` to `/site` so i just put it 
at the root, thinking we might move `/docs` under `/site` at some point in the 
future. But perhaps that's too gnarly a change for too little logic. I've moved 
`/site` under `/docs`.


> Move source for TinkerPop site to source code repo
> --
>
> Key: TINKERPOP-1485
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1485
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.2
>Reporter: stephen mallette
>Assignee: Daniel Kuppitz
> Fix For: 3.3.0
>
>
> Some time ago there was discussion on the Mailing List to move the web site 
> to the source code repo so that it could be contributed to via pull request. 
> It would be nice because then we could better rely on some basic static site 
> generation to help deal with repetitive and sometimes error prone process of 
> updating the header for the pages.
> Anyway, a few thoughts:
> 1. need a shell script to publish the site to svn - something like 
> {{bin/publish-site.sh}}
> 2. place files in the root of the project in a directory called {{site/}}
> 3. if possible, do some basic site generation with maybe some groovy script 
> or just the shell script mentioned above to inject a "header file" for each 
> page.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] tinkerpop issue #459: TINKERPOP-1485 Move source for TinkerPop site to sourc...

2016-10-27 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/459
  
i had thought about putting it under `/docs` but it didn't seem to fit 
there as the "site" really included "docs" and not the other way around. i 
didn't want to up-end everything and move `/docs` to `/site` so i just put it 
at the root, thinking we might move `/docs` under `/site` at some point in the 
future. But perhaps that's too gnarly a change for too little logic. I've moved 
`/site` under `/docs`.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread dkuppitz 
Github user dkuppitz commented on the issue:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread seanbarzilay 
Github user seanbarzilay commented on the issue:

https://github.com/apache/tinkerpop/pull/467
  
I have my own implementation for ProjectStep in 
[Unipop](https://github.com/unipop-graph/unipop) that optimizes the number of 
queries I make


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread okram 
Github user okram commented on the issue:

https://github.com/apache/tinkerpop/pull/467
  
I'm happy to merge this. Question -- why do you need this information? (out 
of curiosity).

VOTE +1.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop pull request #467: Added getProjectKeys in ProjectStep

2016-10-27 Thread seanbarzilay 
GitHub user seanbarzilay opened a pull request:

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

Added getProjectKeys in ProjectStep

One of my strategies requires getting the project keys

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

$ git pull https://github.com/seanbarzilay/tinkerpop master

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

https://github.com/apache/tinkerpop/pull/467.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 #467


commit 8ba3119ecb044a9df1ce704c056ba4bb16bbf3ca
Author: sbarzilay 
Date:   2016-08-03T09:12:39Z

added missing feature requirement

commit 09eb2a568211deb08e5a92ccc701109969f3ab68
Author: sbarzilay 
Date:   2016-10-27T07:23:14Z

added getProjectKeys in ProjectStep

commit 9c5204eddfae53aa35818ad854f4eb87d73668ad
Author: sbarzilay 
Date:   2016-10-27T07:39:19Z

Merge remote-tracking branch 'upstream/master'




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---