GitHub user spmallette opened a pull request:
https://github.com/apache/tinkerpop/pull/672
TINKERPOP-1427 GraphSON 3.0
https://issues.apache.org/jira/browse/TINKERPOP-1427
https://issues.apache.org/jira/browse/TINKERPOP-1574
https://issues.apache.org/jira/browse/TINKERPOP-1414
GraphSON 3.0 is basically 2.0 with the additional types of `g;Map`,
`g:List` and `g:Set`. Includes data file renaming for consistency and
versioning. Packaged data files that are part of zips are now only include 3.0
instances. Defaulted to GraphSON and Gryo 3.0 in all places. IO docs have been
updated.
Upgrade docs needs some work but I'll do that later.
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-1427
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/672.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 #672
----
commit 911d777c113f72324782dd9307a764099f0ccc22
Author: Stephen Mallette <[email protected]>
Date: 2017-06-30T16:34:52Z
TINKERPOP-1427 Make GraphSON 3.0 serializers equivalent to 2.0
There were some internal changes to GraphSON 2.0 deserializers that
improved performance. Migrated all that to 3.0.
commit c7a81531b1ed04a5ae6357544038f9cc77ec9cc3
Author: Stephen Mallette <[email protected]>
Date: 2017-06-30T18:00:13Z
TINKERPOP-1427 Should use GraphSON 3.0 in the serializer provider
commit 548baab384d8c667f953503171fbf2e3edf83f5f
Author: Stephen Mallette <[email protected]>
Date: 2017-07-01T18:16:57Z
TINKERPOP-1427 Added g:Map for GraphSON 3.0
commit 65598953a0fb4f7c3d72e4f1c6d792d370fd780b
Author: Stephen Mallette <[email protected]>
Date: 2017-07-01T19:08:53Z
TINKERPOP-1427 Migrate GraphSON v2 performance enhancement to v3
This was for TinkerGraph whole graph serialization.
commit 9949c38da14a77ddf99e017f0f7c8c91d127c020
Author: Stephen Mallette <[email protected]>
Date: 2017-07-02T21:51:18Z
TINKERPOP-1427 Added g:Set and g:List for GraphSON 3.0
commit caec00199e0ab595ea3e36656f1c311f90547c43
Author: Marko A. Rodriguez <[email protected]>
Date: 2017-07-06T21:56:46Z
99 percent complete GraphSON 3.0 working with Gremlin-Python. Extended test
suite to support both GraphSON 2.0 and GraphSON 3.0 testing. There is one last
requirement to do on the code -- a version flag on GraphSONMessageSerializer.
However, for some reason application/gremlin-v3.0 mime type stuff doesn't work
for the Python->GremlinServer tests... Committing what I have so far as I think
the issue might be an easy .yaml file fix or something. Dunno.
commit c06d9feee4b16c39e41a2067692d8d487f25cb62
Author: Stephen Mallette <[email protected]>
Date: 2017-07-07T15:35:34Z
TINKERPOP-1427 Fixed py server config
Should have used maching TinkerGraph IORegistry for the version.
commit ff89b2a8fcdd1042b5cc2f172ca69015c83d3cd5
Author: Stephen Mallette <[email protected]>
Date: 2017-07-07T17:52:15Z
TINKERPOP-1427 Fixed problems with v2 vs v3 in gremlin-python
There were several problems. In protocol.py there was an expectation as the
to the format of the ResponseMessage that changed between v2 and v3. I added a
bit of a sketchy hack to deal with that detects v2/v3 and then parses
accordingly. That should be nicer. There were some issues with tests as well
that bound assertions to v2 so that when the default serializer swapped from v2
to v3 those assertions started failing. I forced those specific tests to v2 to
get them to pass. Ultimately, we need to more generally test v2 and v3, but at
least gremlin-python is defaulted to v3 at this point and all tests are passing.
commit 3d793ac50911cece1abc6a72b0f6661717cb7d4f
Author: Marko A. Rodriguez <[email protected]>
Date: 2017-07-07T20:52:30Z
added GraphSONReader tests for set/list/map to test_graphsonV3d0.py.
Already tested in the Jython infrastruture, but why not have it also localized
in the pure Python tests.
commit 3dc0b03f0d17ab7bbcbf013c82f32e59d27f29c6
Author: Stephen Mallette <[email protected]>
Date: 2017-07-10T16:50:28Z
TINKERPOP-1427 Coerced Map.Entry to a Map with single entry
commit 228e578f89b82c633424ff2e59a068c58adc6971
Author: Marko A. Rodriguez <[email protected]>
Date: 2017-07-10T19:42:44Z
GraphSONRecordReader/Writer can now be configured to use either GraphSON V2
or V3. Added a test case to verfify proper behavior of parser from GraphSON V3
on HDFS.
commit 4df07fd795b6a4c80b077dcc05bc313dfaca7019
Author: Stephen Mallette <[email protected]>
Date: 2017-07-10T21:25:07Z
TINKERPOP-1427 Fixed a bad variable naming after a rebase
commit 06553498adb16a31e35d8579ad1e7cc207c39148
Author: Stephen Mallette <[email protected]>
Date: 2017-07-11T19:02:57Z
TINKERPOP-1427 Dropped graphson configs prior to 3.0
commit 5b5f850f15e00bd9ec81458c61f428b4abbb1588
Author: Stephen Mallette <[email protected]>
Date: 2017-07-12T11:12:25Z
TINKERPOP-1427 Regenerated/renamed all data files
Added versions to file names. Dropped packaged data files that were not 3.0.
commit 8d973c396a8d9dde6927f2f2c2f4bf0fdab08a16
Author: Stephen Mallette <[email protected]>
Date: 2017-07-12T13:57:24Z
TINKERPOP-1427 Specified gryo versions in tests
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
