[GitHub] tinkerpop pull request #418: Fixed issue in NumberSerializer that could caus...

GitHub user admiralFedora opened a pull request: https://github.com/apache/tinkerpop/pull/418 Fixed issue in NumberSerializer that could cause integer out of bounds In python, unless a number is specified with an L, it will be an int whether it's a 32 or 64 bit integer. When a 64

[jira] [Commented] (TINKERPOP-1280) VertexPrograms should declare traverser requirements

[ https://issues.apache.org/jira/browse/TINKERPOP-1280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15492091#comment-15492091 ] ASF GitHub Bot commented on TINKERPOP-1280: --- Github user twilmes commented on the issue:

[GitHub] tinkerpop issue #404: TINKERPOP-1280 VertexPrograms should declare traverser...

Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/404 Most excellent and good tests. 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

Re: SQL-Gremlin logo?

The logo looks great and the bio is spot-on. I love the "SELECT V FROM G". My only request would be to remove the wavy-grid part on the right side of the logo since it is part of the Microsoft SQL Server logo, just to remove any confusion over whether there is some sort of relationship between

[jira] [Commented] (TINKERPOP-1330) by()-modulation for where()

[ https://issues.apache.org/jira/browse/TINKERPOP-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491767#comment-15491767 ] ASF GitHub Bot commented on TINKERPOP-1330: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #417: TINKERPOP-1330: by()-modulation for where()

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/417 I think we did a pretty bad job on upgrade docs in 3.2.2 so i'm kinda on the lookout for things that we might add to them - this might be a nice thing to call attention to in "user" section

[GitHub] tinkerpop issue #415: TINKERPOP-1448 gremlin-python should be Python 2/3 com...

Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/415 1c5e698 add simple package metadata. --- 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

[jira] [Commented] (TINKERPOP-1448) gremlin-python should be Python 2/3 compatible

[ https://issues.apache.org/jira/browse/TINKERPOP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491666#comment-15491666 ] ASF GitHub Bot commented on TINKERPOP-1448: --- Github user davebshow commented on the issue:

[jira] [Commented] (TINKERPOP-1330) by()-modulation for where()

[ https://issues.apache.org/jira/browse/TINKERPOP-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491536#comment-15491536 ] ASF GitHub Bot commented on TINKERPOP-1330: --- Github user dkuppitz commented on the issue:

[GitHub] tinkerpop issue #417: TINKERPOP-1330: by()-modulation for where()

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/417 Regarding the `match()` translation, it's indeed very easy: ``` gremlin> g.V().as("a").outE("created").as("b"). ..1> inV().as("c").in("created").as("d"). ..2>

[jira] [Assigned] (TINKERPOP-1330) by()-modulation for where()

[ https://issues.apache.org/jira/browse/TINKERPOP-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez reassigned TINKERPOP-1330: - Assignee: Marko A. Rodriguez > by()-modulation for where() >

[jira] [Commented] (TINKERPOP-1330) by()-modulation for where()

[ https://issues.apache.org/jira/browse/TINKERPOP-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491520#comment-15491520 ] ASF GitHub Bot commented on TINKERPOP-1330: --- GitHub user okram opened a pull request:

[GitHub] tinkerpop pull request #417: TINKERPOP-1330: by()-modulation for where()

GitHub user okram opened a pull request: https://github.com/apache/tinkerpop/pull/417 TINKERPOP-1330: by()-modulation for where() https://issues.apache.org/jira/browse/TINKERPOP-1330 Added `by()`-modulation support to `where()` predicate-based steps. Added 3 solid

[GitHub] tinkerpop issue #411: TINKERPOP-14444: Benchmark bytecode->Traversal creatio...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/411 I'm thinking that the cache hit count would be pretty low given the fact that bytecode contains parameters. since the hit count will be low, we'll just have the overhead of a cache without

[jira] [Commented] (TINKERPOP-1391) issue with where filter

[ https://issues.apache.org/jira/browse/TINKERPOP-1391?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491335#comment-15491335 ] ASF GitHub Bot commented on TINKERPOP-1391: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #406: TINKERPOP-1391 issue with where filter

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/406 All tests pass with `docker/build.sh -t -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

[jira] [Commented] (TINKERPOP-1431) Documentation generation requires tests to execute on gremlin-python

[ https://issues.apache.org/jira/browse/TINKERPOP-1431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491321#comment-15491321 ] stephen mallette commented on TINKERPOP-1431: - Implemented on TINKERPOP-1449 >

[GitHub] tinkerpop issue #415: Tinkerpop 1448: gremlin-python should be Python 2/3 co...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/415 ok - fair enough > The only difference is now we can add Python 3 to the package metadata. can you tack that into this PR please? --- If your project is set up for it, you

[GitHub] tinkerpop issue #415: Tinkerpop 1448: gremlin-python should be Python 2/3 co...

Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/415 Maintaining compatibility should be pretty simple, but as you can see from the PR, there are little things that can go wrong. I think the main issue we will run into with this library is

[GitHub] tinkerpop issue #415: Tinkerpop 1448: gremlin-python should be Python 2/3 co...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/415 @davebshow what are the risks of building python 2 and not 3 or the other way around? will we easily break compatibility such that not building against both on a constant basis will cause

No more -DglvPython

Among other improvements to the gremlin-python build, this pull request gets rid of -DglvPython: https://github.com/apache/tinkerpop/pull/416 Now, you can setup your system to just automatically build gremlin-python (in full - with the native python tests) with a regular old: mvn clean install

[jira] [Commented] (TINKERPOP-1449) Streamline gremlin-python build

[ https://issues.apache.org/jira/browse/TINKERPOP-1449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491146#comment-15491146 ] ASF GitHub Bot commented on TINKERPOP-1449: --- GitHub user spmallette opened a pull request:

[jira] [Commented] (TINKERPOP-1391) issue with where filter

[ https://issues.apache.org/jira/browse/TINKERPOP-1391?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491023#comment-15491023 ] ASF GitHub Bot commented on TINKERPOP-1391: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #405: TINKERPOP-1391 issue with where filter

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/405 All tests pass with `docker/build.sh -t -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

[GitHub] tinkerpop issue #414: Release name generator

Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/414 That's ok. I don't actually expect this to get merged. Just for fun. This was a convenient way to share. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] tinkerpop issue #415: Tinkerpop 1448: gremlin-python should be Python 2/3 co...

Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/415 Sounds good to me. I'll make the changes. --- 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

[GitHub] tinkerpop issue #415: Tinkerpop 1448: gremlin-python should be Python 2/3 co...

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/415 I like this. It works for me against Python 2.7.10. The one thing I think would be nice is to move the `compat.py` code into `statics.py`. Why? One less file. The `compat.py` code is all about

[jira] [Commented] (TINKERPOP-1447) Add some JavaScript intelligence to the documentation so that comments and output are not copied in a copy paste

[ https://issues.apache.org/jira/browse/TINKERPOP-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490918#comment-15490918 ] Robert Dale commented on TINKERPOP-1447: Easy to do when there's a single statement. Harder to

[jira] [Commented] (TINKERPOP-1448) gremlin-python should be Python 2/3 compatible

[ https://issues.apache.org/jira/browse/TINKERPOP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490904#comment-15490904 ] ASF GitHub Bot commented on TINKERPOP-1448: --- GitHub user davebshow opened a pull request:

[GitHub] tinkerpop pull request #415: Tinkerpop 1448: gremlin-python should be Python...

GitHub user davebshow opened a pull request: https://github.com/apache/tinkerpop/pull/415 Tinkerpop 1448: gremlin-python should be Python 2/3 compatible https://issues.apache.org/jira/browse/TINKERPOP-1448 This PR adds code to promote Python 2/3 compatibility in the

[jira] [Created] (TINKERPOP-1449) Streamline gremlin-python build

stephen mallette created TINKERPOP-1449: --- Summary: Streamline gremlin-python build Key: TINKERPOP-1449 URL: https://issues.apache.org/jira/browse/TINKERPOP-1449 Project: TinkerPop

[GitHub] tinkerpop issue #414: Release name generator

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/414 Ha. For sure this is bad ass, but I don't know about putting it into the repo... Well, perhaps off `bin/` or something dunno. VOTE +0. --- If your project is set up for it, you can

Re: [DISCUSS] 3.3.0 Branching strategy

I concur. I think we should spend ~3 weeks cleaning up our 3.2.2 chaos and do a 3.2.3 release before going down the 3.3.0 rabbit hole. Marko. http://markorodriguez.com > On Sep 14, 2016, at 7:42 AM, Stephen Mallette wrote: > > I don't really like how 3.2.2 is

[jira] [Commented] (TINKERPOP-1369) Replace REST API with HTTP API

[ https://issues.apache.org/jira/browse/TINKERPOP-1369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490628#comment-15490628 ] ASF GitHub Bot commented on TINKERPOP-1369: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #399: TINKERPOP-1369 Renamed REST API to HTTP API

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/399 I will merge this once 3.3.x is opened up for development. --- 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

[GitHub] tinkerpop issue #399: TINKERPOP-1369 Renamed REST API to HTTP API

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/399 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

[jira] [Updated] (TINKERPOP-1448) gremlin-python should be Python 2/3 compatible

[ https://issues.apache.org/jira/browse/TINKERPOP-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David M. Brown updated TINKERPOP-1448: -- Component/s: language-variant > gremlin-python should be Python 2/3 compatible >

[jira] [Created] (TINKERPOP-1448) gremlin-python should be Python 2/3 compatible

David M. Brown created TINKERPOP-1448: - Summary: gremlin-python should be Python 2/3 compatible Key: TINKERPOP-1448 URL: https://issues.apache.org/jira/browse/TINKERPOP-1448 Project: TinkerPop

[GitHub] tinkerpop pull request #414: Release name generator

GitHub user robertdale opened a pull request: https://github.com/apache/tinkerpop/pull/414 Release name generator $ groovy release-name-generator.groovy Twenty-Four Gremlin From Tulsa $ groovy release-name-generator.groovy (I'd) Wait a Million Gremlin $ groovy

[jira] [Commented] (TINKERPOP-1442) Killing session should make better attempt to cleanup

[ https://issues.apache.org/jira/browse/TINKERPOP-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15489780#comment-15489780 ] ASF GitHub Bot commented on TINKERPOP-1442: --- Github user PommeVerte commented on the issue:

[GitHub] tinkerpop issue #412: TINKERPOP-1442 Improved session cleanup on client clos...

Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/412 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

[GitHub] tinkerpop issue #413: TINKERPOP-1442 Improved session cleanup on client clos...

Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/413 VOTE +1 this is a nice fix to have. Unit tests passed for me too. --- 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

[jira] [Commented] (TINKERPOP-1369) Replace REST API with HTTP API

[ https://issues.apache.org/jira/browse/TINKERPOP-1369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15489771#comment-15489771 ] ASF GitHub Bot commented on TINKERPOP-1369: --- Github user PommeVerte commented on the issue:

[GitHub] tinkerpop issue #399: TINKERPOP-1369 Renamed REST API to HTTP API

Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/399 +1 from me :) --- 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,