[GitHub] tinkerpop issue #691: TINKERPOP-1747 Streamline inheritance for gremlin-pyth...

2017-08-11 Thread davebshow
Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/691 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 th

[GitHub] tinkerpop issue #691: TINKERPOP-1747 Streamline inheritance for gremlin-pyth...

2017-08-11 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/691 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 t

[GitHub] tinkerpop issue #691: TINKERPOP-1747 Streamline inheritance for gremlin-pyth...

2017-08-10 Thread davebshow
Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/691 Yeah well there are different approaches we could have taken, e.g. leaving the custom reader/writer options for the v2 and v3 serializer classes, but it seems to me that the point of those class

[GitHub] tinkerpop issue #691: TINKERPOP-1747 Streamline inheritance for gremlin-pyth...

2017-08-10 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/691 I am not too hip to the Python code anymore, but I do know that adding the GraphSON3 support was a little "sketchy". Thus, if this is a Python-esque improvement upon my non-Python skills, then great