[GitHub] [tinkerpop] vkagamlyk opened a new pull request, #1677: Finish graphbinary support in gremlin-python

2022-06-02 Thread GitBox


vkagamlyk opened a new pull request, #1677:
URL: https://github.com/apache/tinkerpop/pull/1677

   Finish graphbinary support in gremlin-python


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[tinkerpop] branch 3.6-dev updated (e23a963c92 -> 8e7a798fb5)

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch 3.6-dev
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from e23a963c92 Fixed build issues in recently merged GraphBinary for 
javascript
 add 8e7a798fb5 Fixed docker -d option

No new revisions were added by this update.

Summary of changes:
 docker/hadoop/install.sh |  6 +++---
 docker/scripts/build.sh  | 14 ++
 2 files changed, 13 insertions(+), 7 deletions(-)



[tinkerpop] branch master updated (45f7aba98e -> 179678ab7d)

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from 45f7aba98e Merge branch '3.6-dev'
 add 8e7a798fb5 Fixed docker -d option
 new 179678ab7d Merge branch '3.6-dev'

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 docker/hadoop/install.sh |  6 +++---
 docker/scripts/build.sh  | 14 ++
 2 files changed, 13 insertions(+), 7 deletions(-)



[tinkerpop] 01/01: Merge branch '3.6-dev'

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 179678ab7d5483eb5ac57a22eb42ed8c2e484277
Merge: 45f7aba98e 8e7a798fb5
Author: Stephen Mallette 
AuthorDate: Thu Jun 2 15:11:50 2022 -0400

Merge branch '3.6-dev'

 docker/hadoop/install.sh |  6 +++---
 docker/scripts/build.sh  | 14 ++
 2 files changed, 13 insertions(+), 7 deletions(-)



[GitHub] [tinkerpop] vkagamlyk opened a new pull request, #1676: Valentyn/an 1165 python graphbinary (#121)

2022-06-02 Thread GitBox


vkagamlyk opened a new pull request, #1676:
URL: https://github.com/apache/tinkerpop/pull/1676

   Finish graphbinary support in gremlin-python
   
   https://issues.apache.org/jira/browse/TINKERPOP-2693


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] codecov-commenter commented on pull request #1674: Bump logback version to 1.2.9 or greater

2022-06-02 Thread GitBox


codecov-commenter commented on PR #1674:
URL: https://github.com/apache/tinkerpop/pull/1674#issuecomment-1145178082

   # 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1674?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#1674](https://codecov.io/gh/apache/tinkerpop/pull/1674?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (1393f20) into 
[3.6-dev](https://codecov.io/gh/apache/tinkerpop/commit/f94e203a7b010052373446be7c247b678f317de0?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (f94e203) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@   Coverage Diff@@
   ##   3.6-dev#1674   +/-   ##
   
 Coverage63.84%   63.84%   
   
 Files   23   23   
 Lines 3596 3596   
   
 Hits  2296 2296   
 Misses1137 1137   
 Partials   163  163   
   ```
   
   
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1674?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1674?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Last update 
[f94e203...1393f20](https://codecov.io/gh/apache/tinkerpop/pull/1674?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Read the [comment 
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] chessvivek commented on pull request #1658: Fix non determinism in test shouldSerializeToTreeJson()

2022-06-02 Thread GitBox


chessvivek commented on PR #1658:
URL: https://github.com/apache/tinkerpop/pull/1658#issuecomment-1145162677

   Thanks for the votes of approval. If there's anything you would like me to 
improve upon in this PR, please let me know!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] lyndonbauto commented on pull request #1675: Upgrade docs for gremlin-javascript GraphBinary support

2022-06-02 Thread GitBox


lyndonbauto commented on PR #1675:
URL: https://github.com/apache/tinkerpop/pull/1675#issuecomment-1145079959

   VOTE+1


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] lyndonbauto commented on pull request #1671: TINKERPOP-2749: Gremlin Javascript, Gremlint Windows build error fixes

2022-06-02 Thread GitBox


lyndonbauto commented on PR #1671:
URL: https://github.com/apache/tinkerpop/pull/1671#issuecomment-1145077441

   @L0Lmaker Looks like we got a conflict now, could you rebase?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] jorgebay commented on pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


jorgebay commented on PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547#issuecomment-1144803017

   oh man, you saved my ass
   
   Thanks!!!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] spmallette commented on pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


spmallette commented on PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547#issuecomment-1144785237

   I've fixed the problems on 3.6-dev/master - had to do with the new `Merge` 
enum and new testing of `GraphComputer` in GLVs. Both needed to be accounted 
for in the new serializer: e23a963c92bd6190a9b6b070d8dd4999a42d7471
   
   > I usually advocate for squash & merge commit GH workflow but given that 
it's a large changeset (10K LOC) I think there's value in keeping the history.
   
   agree with that choice in this case. glad you kept the history.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[tinkerpop] branch 3.6-dev updated (6b8c1f38b2 -> e23a963c92)

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch 3.6-dev
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from 6b8c1f38b2 Merge branch '3.5-dev' into 3.6-dev
 add e23a963c92 Fixed build issues in recently merged GraphBinary for 
javascript

No new revisions were added by this update.

Summary of changes:
 .../strategy/decoration/RemoteStrategy.java|  3 --
 .../lib/process/graph-traversal.js | 34 +++---
 .../gremlin-javascript/lib/process/traversal.js|  2 +-
 .../lib/structure/io/binary/internals/DataType.js  |  1 +
 .../io/binary/internals/EnumSerializer.js  |  1 +
 .../test/unit/graphbinary/AnySerializer-test.js|  8 -
 6 files changed, 33 insertions(+), 16 deletions(-)



[tinkerpop] branch master updated (20d2cac2b9 -> 45f7aba98e)

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from 20d2cac2b9 Merge branch '3.6-dev'
 add e23a963c92 Fixed build issues in recently merged GraphBinary for 
javascript
 new 45f7aba98e Merge branch '3.6-dev'

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../strategy/decoration/RemoteStrategy.java|  3 --
 .../lib/process/graph-traversal.js | 34 +++---
 .../gremlin-javascript/lib/process/traversal.js|  2 +-
 .../lib/structure/io/binary/internals/DataType.js  |  1 +
 .../io/binary/internals/EnumSerializer.js  |  1 +
 .../test/unit/graphbinary/AnySerializer-test.js|  8 -
 6 files changed, 33 insertions(+), 16 deletions(-)



[tinkerpop] 01/01: Merge branch '3.6-dev'

2022-06-02 Thread spmallette
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 45f7aba98e3357e2eed1d909c57848fbd630869b
Merge: 20d2cac2b9 e23a963c92
Author: Stephen Mallette 
AuthorDate: Thu Jun 2 07:59:52 2022 -0400

Merge branch '3.6-dev'

 .../strategy/decoration/RemoteStrategy.java|  3 --
 .../lib/process/graph-traversal.js | 34 +++---
 .../gremlin-javascript/lib/process/traversal.js|  2 +-
 .../lib/structure/io/binary/internals/DataType.js  |  1 +
 .../io/binary/internals/EnumSerializer.js  |  1 +
 .../test/unit/graphbinary/AnySerializer-test.js|  8 -
 6 files changed, 33 insertions(+), 16 deletions(-)



[GitHub] [tinkerpop] codecov-commenter commented on pull request #1675: Upgrade docs for gremlin-javascript GraphBinary support

2022-06-02 Thread GitBox


codecov-commenter commented on PR #1675:
URL: https://github.com/apache/tinkerpop/pull/1675#issuecomment-1144601669

   # 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1675?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#1675](https://codecov.io/gh/apache/tinkerpop/pull/1675?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (feada75) into 
[3.5-dev](https://codecov.io/gh/apache/tinkerpop/commit/18a1ed7e046c273f5d3b17aada731ed5619e3848?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (18a1ed7) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@   Coverage Diff@@
   ##   3.5-dev#1675   +/-   ##
   
 Coverage63.35%   63.35%   
   
 Files   23   23   
 Lines 3553 3553   
   
 Hits  2251 2251   
 Misses1129 1129   
 Partials   173  173   
   ```
   
   
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1675?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1675?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Last update 
[18a1ed7...feada75](https://codecov.io/gh/apache/tinkerpop/pull/1675?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Read the [comment 
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] jorgebay commented on pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


jorgebay commented on PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547#issuecomment-1144600246

   I've merged it into 3.6-dev but it's failing when running cucumber tests 
with graphbinary: 
https://github.com/apache/tinkerpop/runs/6704524572?check_suite_focus=true#step:4:11048
   
   ```
   env CLIENT_MIMETYPE='application/vnd.graphbinary-v1.0' 
./node_modules/.bin/cucumber-js --require test/cucumber 
../../../../../gremlin-test/features/
   ```
   
   Any ideas @ihoro @spmallette ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] jorgebay opened a new pull request, #1675: Upgrade docs for gremlin-javascript GraphBinary support

2022-06-02 Thread GitBox


jorgebay opened a new pull request, #1675:
URL: https://github.com/apache/tinkerpop/pull/1675

   Upgrade docs for @ihoro 's #1547


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[tinkerpop] branch graphbinary-js-docs created (now feada75de2)

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a change to branch graphbinary-js-docs
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


  at feada75de2 Upgrade docs for gremlin-javascript GraphBinary support

This branch includes the following new commits:

 new feada75de2 Upgrade docs for gremlin-javascript GraphBinary support

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[tinkerpop] 01/01: Upgrade docs for gremlin-javascript GraphBinary support

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a commit to branch graphbinary-js-docs
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit feada75de298340883d77aa346684a36c8cd528f
Author: Jorge Bay 
AuthorDate: Thu Jun 2 10:22:51 2022 +0200

Upgrade docs for gremlin-javascript GraphBinary support
---
 CHANGELOG.asciidoc  |  1 +
 docs/src/upgrade/release-3.5.x.asciidoc | 21 +
 2 files changed, 22 insertions(+)

diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index d83bd89e00..19cd1c2ad0 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -26,6 +26,7 @@ 
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
 * Allowed `datetime()` syntax to accept zone offset with colon separators and 
seconds.
 * Fixed a minor problem in the Gremlin parser where a `GraphTraversalSource` 
may not have been initialized.
 * Added getters to high and low properties in RangeLocalStep.
+* TINKERPOP-2229 Added GraphBinary serialization support to gremlin-javascript.
 
 [[release-3-5-3]]
 === TinkerPop 3.5.3 (Release Date: April 4, 2022)
diff --git a/docs/src/upgrade/release-3.5.x.asciidoc 
b/docs/src/upgrade/release-3.5.x.asciidoc
index 852e59e42a..8ded2be6cd 100644
--- a/docs/src/upgrade/release-3.5.x.asciidoc
+++ b/docs/src/upgrade/release-3.5.x.asciidoc
@@ -21,6 +21,27 @@ 
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
 
 *The Sleeping Gremlin: No. 18 Entr'acte Symphonique*
 
+== TinkerPop 3.5.4
+
+*Release Date: NOT OFFICIALLY RELEASED YET*
+
+Please see the 
link:https://github.com/apache/tinkerpop/blob/3.5-dev/CHANGELOG.asciidoc#release-3-5-4[changelog]
 for a
+complete list of all the modifications that are part of this release.
+
+=== Upgrading for Users
+
+ GraphBinary support for gremlin-javascript
+
+Gremlin JavaScript now also supports GraphBinary. GraphSON3 still remains the 
default serialization format.
+
+To enable GraphBinary support, set the mime type in the connection options.
+
+[source,javascript]
+
+const options = { mimeType: 'application/vnd.graphbinary-v1.0' }
+const g = traversal().withRemote(new 
DriverRemoteConnection('ws://localhost:8182/gremlin', options));
+
+
 == TinkerPop 3.5.3
 
 *Release Date: April 4, 2022*



[tinkerpop] branch master updated (f8a2df03c7 -> 20d2cac2b9)

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from f8a2df03c7 Merge '3.6-dev' into master and solving conflicts
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling
 add bdca983795 Impl DateSerializer
 add 908583911a Impl VertexSerializer
 add 17e124c5cb Impl EdgeSerializer
 add 1ca2bbf130 CI: include gremlin-javascript GraphBinary unit tests
 add 20a8d85f49 Impl TimestampSerializer
 add 39695d997f Normalize style of GraphBinary unit tests
 add 85202c0cfa Impl ClassSerializer
 add 76be2bc178 Impl DoubleSerializer and FloatSerializer
 add d3296b3182 Impl SetSerializer
 add 5c1750e666 Impl PathSerializer
 add 525a3c8a71 Impl PropertySerializer
 add df55dfec6f Impl VertexPropertySerializer
 add 5c13ce8fd3 Impl LambdaSerializer
 add 2de6335fc6 Ignore Graph and Binding types
 add ad4a411d2a Impl ByteSerializer
 add fb3f18c942 Impl ByteBufferSerializer
 add 88759eaacd Impl TextPSerializer
 add fba5041d00 Impl TraversalStrategySerializer
 add 15084ad4e7 Impl BulkSetSerializer
 add b922456c79 Backup bigint based Long implementation as LongSerializerNg
 add 0b31ff973e LongSerializer: move away from bigint and align with 
GraphSON implementation
 add df8596b11b Impl NumberSerializationStrategy
 add 61b41f6328 AnySerializer-test: mention that Set is the same as List
 add a84f6b1d9e Turn off LongSerializerNg
 add cca51d5f46 BytecodeSerializer: fix sources serialization
 add 2faf27fc1e Make TraversalStrategySerializer use FQCN
 add 33c326d249 AnySerializer: fix LONG deserialization test
 add 668e7a94d5 Fix ReservedKeysVerificationStrategy.fqcn field value
 add 142d04c0e3 gremlin-javascript: run tests for two protocols 
(application/vnd.gremlin-v3.0+json a

[tinkerpop] 01/01: Merge branch '3.6-dev'

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 20d2cac2b9fd57d66d5d0708e0c71514fd487ce4
Merge: f8a2df03c7 6b8c1f38b2
Author: Jorge Bay 
AuthorDate: Thu Jun 2 09:59:31 2022 +0200

Merge branch '3.6-dev'

 .gitignore |   1 +
 .../gremlin-javascript/lib/driver/connection.js|  14 +-
 .../lib/process/traversal-strategy.js  |   2 +-
 .../lib/structure/io/binary/GraphBinary.js | 109 
 .../structure/io/binary/internals/AnySerializer.js |  95 
 .../io/binary/internals/ArraySerializer.js | 135 +
 .../io/binary/internals/BigIntegerSerializer.js| 151 ++
 .../io/binary/internals/BooleanSerializer.js   | 103 
 .../io/binary/internals/BulkSetSerializer.js   | 124 +
 .../io/binary/internals/ByteBufferSerializer.js| 124 +
 .../io/binary/internals/ByteSerializer.js  |  95 
 .../io/binary/internals/BytecodeSerializer.js  | 249 +
 .../lib/structure/io/binary/internals/DataType.js  |  97 
 .../io/binary/internals/DateSerializer.js  | 103 
 .../io/binary/internals/DoubleSerializer.js|  99 
 .../io/binary/internals/EdgeSerializer.js  | 215 
 .../io/binary/internals/EnumSerializer.js  | 148 +
 .../io/binary/internals/FloatSerializer.js |  99 
 .../io/binary/internals/GraphBinaryReader.js   |  80 +++
 .../io/binary/internals/GraphBinaryWriter.js   |  83 +++
 .../structure/io/binary/internals/IntSerializer.js | 112 
 .../io/binary/internals/LambdaSerializer.js|  70 +++
 .../io/binary/internals/LongSerializer.js  | 107 
 .../io/binary/internals/LongSerializerNg.js| 102 
 .../structure/io/binary/internals/MapSerializer.js | 153 ++
 .../internals/NumberSerializationStrategy.js   |  68 +++
 .../structure/io/binary/internals/PSerializer.js   | 151 ++
 .../io/binary/internals/PathSerializer.js  | 123 +
 .../io/binary/internals/PropertySerializer.js  | 141 +
 .../io/binary/internals/ShortSerializer.js |  99 
 .../io/binary/internals/StringSerializer.js| 114 
 .../io/binary/internals/TextPSerializer.js | 145 +
 .../internals/TraversalStrategySerializer.js   |  72 +++
 .../io/binary/internals/TraverserSerializer.js | 123 +
 .../binary/internals/UnspecifiedNullSerializer.js  |  75 +++
 .../io/binary/internals/UuidSerializer.js  | 119 +
 .../binary/internals/VertexPropertySerializer.js   | 172 ++
 .../io/binary/internals/VertexSerializer.js| 135 +
 .../lib/structure/io/binary/internals/utils.js |  59 ++
 .../javascript/gremlin-javascript/package.json |  12 +-
 .../javascript/gremlin-javascript/test/helper.js   |  17 +-
 .../test/integration/sasl-authentication-tests.js  |   4 +-
 .../test/integration/traversal-test.js |   2 +-
 .../test/unit/graphbinary/AnySerializer-test.js| 594 +
 .../graphbinary/ArraySerializerTestTemplate.js | 151 ++
 .../unit/graphbinary/BigIntegerSerializer-test.js  | 200 +++
 .../unit/graphbinary/BooleanSerializer-test.js | 141 +
 .../unit/graphbinary/BulkSetSerializer-test.js | 158 ++
 .../unit/graphbinary/ByteBufferSerializer-test.js  | 155 ++
 .../test/unit/graphbinary/ByteSerializer-test.js   | 118 
 .../unit/graphbinary/BytecodeSerializer-test.js| 213 
 .../test/unit/graphbinary/ClassSerializer-test.js  |  25 +
 .../test/unit/graphbinary/DateSerializer-test.js   |  25 +
 .../unit/graphbinary/DateSerializerTestTemplate.js | 149 ++
 .../test/unit/graphbinary/DoubleSerializer-test.js | 152 ++
 .../test/unit/graphbinary/EdgeSerializer-test.js   | 170 ++
 .../test/unit/graphbinary/EnumSerializer-test.js   | 172 ++
 .../test/unit/graphbinary/FloatSerializer-test.js  | 123 +
 .../unit/graphbinary/GraphBinaryReader-test.js | 129 +
 .../unit/graphbinary/GraphBinaryWriter-test.js |  86 +++
 .../test/unit/graphbinary/IntSerializer-test.js| 148 +
 .../test/unit/graphbinary/LambdaSerializer-test.js |  95 
 .../test/unit/graphbinary/ListSerializer-test.js   |  25 +
 .../test/unit/graphbinary/LongSerializer-test.js   | 145 +
 .../test/unit/graphbinary/LongSerializerNg-test.js | 133 +
 .../test/unit/graphbinary/MapSerializer-test.js| 215 
 .../test/unit/graphbinary/PSerializer-test.js  | 197 +++
 .../test/unit/graphbinary/PathSerializer-test.js   | 152 ++
 .../unit/graphbinary/PropertySerializer-test.js| 141 +
 .../test/unit/graphbinary/SetSerializer-test.js|  25 +
 .../test/unit/graphbinary/ShortSerializer-test.js  | 122 +
 .../test/unit/graphbinary/StringSerializer-test.js |  25 +
 .../graphbinary/StringSerializerTestTemplate.js| 169 ++
 .../test/uni

[tinkerpop] branch 3.6-dev updated (f94e203a7b -> 6b8c1f38b2)

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a change to branch 3.6-dev
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from f94e203a7b Merge '3.5-dev' into 3.6-dev and solving conflicts
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling
 add bdca983795 Impl DateSerializer
 add 908583911a Impl VertexSerializer
 add 17e124c5cb Impl EdgeSerializer
 add 1ca2bbf130 CI: include gremlin-javascript GraphBinary unit tests
 add 20a8d85f49 Impl TimestampSerializer
 add 39695d997f Normalize style of GraphBinary unit tests
 add 85202c0cfa Impl ClassSerializer
 add 76be2bc178 Impl DoubleSerializer and FloatSerializer
 add d3296b3182 Impl SetSerializer
 add 5c1750e666 Impl PathSerializer
 add 525a3c8a71 Impl PropertySerializer
 add df55dfec6f Impl VertexPropertySerializer
 add 5c13ce8fd3 Impl LambdaSerializer
 add 2de6335fc6 Ignore Graph and Binding types
 add ad4a411d2a Impl ByteSerializer
 add fb3f18c942 Impl ByteBufferSerializer
 add 88759eaacd Impl TextPSerializer
 add fba5041d00 Impl TraversalStrategySerializer
 add 15084ad4e7 Impl BulkSetSerializer
 add b922456c79 Backup bigint based Long implementation as LongSerializerNg
 add 0b31ff973e LongSerializer: move away from bigint and align with 
GraphSON implementation
 add df8596b11b Impl NumberSerializationStrategy
 add 61b41f6328 AnySerializer-test: mention that Set is the same as List
 add a84f6b1d9e Turn off LongSerializerNg
 add cca51d5f46 BytecodeSerializer: fix sources serialization
 add 2faf27fc1e Make TraversalStrategySerializer use FQCN
 add 33c326d249 AnySerializer: fix LONG deserialization test
 add 668e7a94d5 Fix ReservedKeysVerificationStrategy.fqcn field value
 add 142d04c0e3 gremlin-javascript: run tests for two protocols 
(application/vnd.gremlin-v3.0+json

[tinkerpop] branch 3.6-dev-temp created (now 6b8c1f38b2)

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a change to branch 3.6-dev-temp
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


  at 6b8c1f38b2 Merge branch '3.5-dev' into 3.6-dev

This branch includes the following new commits:

 new 6b8c1f38b2 Merge branch '3.5-dev' into 3.6-dev

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[tinkerpop] 01/01: Merge branch '3.5-dev' into 3.6-dev

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a commit to branch 3.6-dev-temp
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 6b8c1f38b22728b18bf1c194088e843f0d6c275c
Merge: f94e203a7b 18a1ed7e04
Author: Jorge Bay 
AuthorDate: Thu Jun 2 09:51:10 2022 +0200

Merge branch '3.5-dev' into 3.6-dev

 .gitignore |   1 +
 .../gremlin-javascript/lib/driver/connection.js|  14 +-
 .../lib/process/traversal-strategy.js  |   2 +-
 .../lib/structure/io/binary/GraphBinary.js | 109 
 .../structure/io/binary/internals/AnySerializer.js |  95 
 .../io/binary/internals/ArraySerializer.js | 135 +
 .../io/binary/internals/BigIntegerSerializer.js| 151 ++
 .../io/binary/internals/BooleanSerializer.js   | 103 
 .../io/binary/internals/BulkSetSerializer.js   | 124 +
 .../io/binary/internals/ByteBufferSerializer.js| 124 +
 .../io/binary/internals/ByteSerializer.js  |  95 
 .../io/binary/internals/BytecodeSerializer.js  | 249 +
 .../lib/structure/io/binary/internals/DataType.js  |  97 
 .../io/binary/internals/DateSerializer.js  | 103 
 .../io/binary/internals/DoubleSerializer.js|  99 
 .../io/binary/internals/EdgeSerializer.js  | 215 
 .../io/binary/internals/EnumSerializer.js  | 148 +
 .../io/binary/internals/FloatSerializer.js |  99 
 .../io/binary/internals/GraphBinaryReader.js   |  80 +++
 .../io/binary/internals/GraphBinaryWriter.js   |  83 +++
 .../structure/io/binary/internals/IntSerializer.js | 112 
 .../io/binary/internals/LambdaSerializer.js|  70 +++
 .../io/binary/internals/LongSerializer.js  | 107 
 .../io/binary/internals/LongSerializerNg.js| 102 
 .../structure/io/binary/internals/MapSerializer.js | 153 ++
 .../internals/NumberSerializationStrategy.js   |  68 +++
 .../structure/io/binary/internals/PSerializer.js   | 151 ++
 .../io/binary/internals/PathSerializer.js  | 123 +
 .../io/binary/internals/PropertySerializer.js  | 141 +
 .../io/binary/internals/ShortSerializer.js |  99 
 .../io/binary/internals/StringSerializer.js| 114 
 .../io/binary/internals/TextPSerializer.js | 145 +
 .../internals/TraversalStrategySerializer.js   |  72 +++
 .../io/binary/internals/TraverserSerializer.js | 123 +
 .../binary/internals/UnspecifiedNullSerializer.js  |  75 +++
 .../io/binary/internals/UuidSerializer.js  | 119 +
 .../binary/internals/VertexPropertySerializer.js   | 172 ++
 .../io/binary/internals/VertexSerializer.js| 135 +
 .../lib/structure/io/binary/internals/utils.js |  59 ++
 .../javascript/gremlin-javascript/package.json |  12 +-
 .../javascript/gremlin-javascript/test/helper.js   |  17 +-
 .../test/integration/sasl-authentication-tests.js  |   4 +-
 .../test/integration/traversal-test.js |   2 +-
 .../test/unit/graphbinary/AnySerializer-test.js| 594 +
 .../graphbinary/ArraySerializerTestTemplate.js | 151 ++
 .../unit/graphbinary/BigIntegerSerializer-test.js  | 200 +++
 .../unit/graphbinary/BooleanSerializer-test.js | 141 +
 .../unit/graphbinary/BulkSetSerializer-test.js | 158 ++
 .../unit/graphbinary/ByteBufferSerializer-test.js  | 155 ++
 .../test/unit/graphbinary/ByteSerializer-test.js   | 118 
 .../unit/graphbinary/BytecodeSerializer-test.js| 213 
 .../test/unit/graphbinary/ClassSerializer-test.js  |  25 +
 .../test/unit/graphbinary/DateSerializer-test.js   |  25 +
 .../unit/graphbinary/DateSerializerTestTemplate.js | 149 ++
 .../test/unit/graphbinary/DoubleSerializer-test.js | 152 ++
 .../test/unit/graphbinary/EdgeSerializer-test.js   | 170 ++
 .../test/unit/graphbinary/EnumSerializer-test.js   | 172 ++
 .../test/unit/graphbinary/FloatSerializer-test.js  | 123 +
 .../unit/graphbinary/GraphBinaryReader-test.js | 129 +
 .../unit/graphbinary/GraphBinaryWriter-test.js |  86 +++
 .../test/unit/graphbinary/IntSerializer-test.js| 148 +
 .../test/unit/graphbinary/LambdaSerializer-test.js |  95 
 .../test/unit/graphbinary/ListSerializer-test.js   |  25 +
 .../test/unit/graphbinary/LongSerializer-test.js   | 145 +
 .../test/unit/graphbinary/LongSerializerNg-test.js | 133 +
 .../test/unit/graphbinary/MapSerializer-test.js| 215 
 .../test/unit/graphbinary/PSerializer-test.js  | 197 +++
 .../test/unit/graphbinary/PathSerializer-test.js   | 152 ++
 .../unit/graphbinary/PropertySerializer-test.js| 141 +
 .../test/unit/graphbinary/SetSerializer-test.js|  25 +
 .../test/unit/graphbinary/ShortSerializer-test.js  | 122 +
 .../test/unit/graphbinary/StringSerializer-test.js |  25 +
 .../graphbinary/StringSerializerTestTemplate.js| 169 +

[GitHub] [tinkerpop] codecov-commenter commented on pull request #1670: Bump ws from 6.2.2 to 8.7.0 in /gremlin-javascript/src/main/javascript/gremlin-javascript

2022-06-02 Thread GitBox


codecov-commenter commented on PR #1670:
URL: https://github.com/apache/tinkerpop/pull/1670#issuecomment-1144555157

   # 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1670?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#1670](https://codecov.io/gh/apache/tinkerpop/pull/1670?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (517a9d1) into 
[3.5-dev](https://codecov.io/gh/apache/tinkerpop/commit/18a1ed7e046c273f5d3b17aada731ed5619e3848?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (18a1ed7) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@   Coverage Diff@@
   ##   3.5-dev#1670   +/-   ##
   
 Coverage63.35%   63.35%   
   
 Files   23   23   
 Lines 3553 3553   
   
 Hits  2251 2251   
 Misses1129 1129   
 Partials   173  173   
   ```
   
   
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1670?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1670?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Last update 
[18a1ed7...517a9d1](https://codecov.io/gh/apache/tinkerpop/pull/1670?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Read the [comment 
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] codecov-commenter commented on pull request #1655: Bump cucumber from 4.2.1 to 6.0.7 in /gremlin-javascript/src/main/javascript/gremlin-javascript

2022-06-02 Thread GitBox


codecov-commenter commented on PR #1655:
URL: https://github.com/apache/tinkerpop/pull/1655#issuecomment-1144554990

   # 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1655?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#1655](https://codecov.io/gh/apache/tinkerpop/pull/1655?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (2250cf4) into 
[3.5-dev](https://codecov.io/gh/apache/tinkerpop/commit/18a1ed7e046c273f5d3b17aada731ed5619e3848?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (18a1ed7) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@   Coverage Diff@@
   ##   3.5-dev#1655   +/-   ##
   
 Coverage63.35%   63.35%   
   
 Files   23   23   
 Lines 3553 3553   
   
 Hits  2251 2251   
 Misses1129 1129   
 Partials   173  173   
   ```
   
   
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1655?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1655?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Last update 
[18a1ed7...2250cf4](https://codecov.io/gh/apache/tinkerpop/pull/1655?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Read the [comment 
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] jorgebay commented on pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


jorgebay commented on PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547#issuecomment-1144543532

   I usually advocate for squash & merge commit GH workflow but given that it's 
a large changeset (10K LOC) I think there's value in keeping the history. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[tinkerpop] branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/eslint-8.16.0 updated (3d33351dd6 -> 360f4dfb0e)

2022-06-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch 
dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/eslint-8.16.0
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


 discard 3d33351dd6 Bump eslint
 add e2bba8f3a5 Bump Moq from 4.17.2 to 4.18.1 in /gremlin-dotnet
 add ce9499467a Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/Moq-4.18.1' into 3.5-dev
 add 104510a12e Fix deadlock on non response-specific read errors (#1669)
 add e991b66064 Invoking CTR as this is simple documentation and Bytecode 
exporting
 add c0fe76ba53 adding v3 back to go.mod
 add 4927388e0c updating pathing for cucumber versions
 add 6b0169e37b Gremlin-Go: Updated Docker Testing Set Up (#1665)
 add 5fb7888f4e Added getters to high and low properties in RangeLocalStep 
(#1667)
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling
 add bdca983795 Impl DateSerializer
 add 908583911a Impl VertexSerializer
 add 17e124c5cb Impl EdgeSerializer
 add 1ca2bbf130 CI: include gremlin-javascript GraphBinary unit tests
 add 20a8d85f49 Impl TimestampSerializer
 add 39695d997f Normalize style of GraphBinary unit tests
 add 85202c0cfa Impl ClassSerializer
 add 76be2bc178 Impl DoubleSerializer and FloatSerializer
 add d3296b3182 Impl SetSerializer
 add 5c1750e666 Impl PathSerializer
 add 525a3c8a71 Impl PropertySerializer
 add df55dfec6f Impl VertexPropertySerializer
 add 5c13ce8fd3 Impl LambdaSerializer
 add 2de6335fc6 Ignore Graph and Binding types
 add ad4a411d2a Impl ByteSerializer
 add fb3f18c942 Impl ByteBufferSerializer
 add 88759eaacd Impl TextPSerializer
 add fba5041d00 Impl TraversalStrategySerializer
 add 15084ad4e7 Impl BulkSetSerializer
 add b922456c79 Backup bigint based Long implementation as L

[tinkerpop] branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/mocha-10.0.0 updated (52c68c1262 -> ec946eb1d6)

2022-06-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch 
dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/mocha-10.0.0
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


 discard 52c68c1262 Bump mocha in 
/gremlin-javascript/src/main/javascript/gremlin-javascript
 add 3436b3f80a Serialization refactoring (#1648)
 add 60dd5f966a Fix for boolean writes
 add cf653e303d Bump Microsoft.NET.Test.Sdk from 17.1.0 to 17.2.0 in 
/gremlin-dotnet
 add 1a6618f5db Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/Microsoft.NET.Test.Sdk-17.2.0' into 
3.5-dev
 add ede36bb254 Bump System.Text.Json from 6.0.3 to 6.0.4 in /gremlin-dotnet
 add d90a2e0e04 Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/System.Text.Json-6.0.4' into 3.5-dev
 add b2c050fd33 Adjust export of enum traversal operations (#1650)
 add f59c47cab9 refactor ResultSet.One() to handle nil result (#1653)
 add ad6b3b7d4d Add named parameters for strategies in Gremlin-Go and 
updated translator (#1664)
 add c8db98c5f9 Gremlin-Go: Added support for additional types (#1663)
 add aa2820bc4c Adding strategy support to cucumber world for gremlin-go
 add e2bba8f3a5 Bump Moq from 4.17.2 to 4.18.1 in /gremlin-dotnet
 add ce9499467a Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/Moq-4.18.1' into 3.5-dev
 add 104510a12e Fix deadlock on non response-specific read errors (#1669)
 add e991b66064 Invoking CTR as this is simple documentation and Bytecode 
exporting
 add c0fe76ba53 adding v3 back to go.mod
 add 4927388e0c updating pathing for cucumber versions
 add 6b0169e37b Gremlin-Go: Updated Docker Testing Set Up (#1665)
 add 5fb7888f4e Added getters to high and low properties in RangeLocalStep 
(#1667)
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling

[tinkerpop] branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/ws-8.7.0 updated (a84943beca -> 517a9d195a)

2022-06-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch 
dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/ws-8.7.0
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


 discard a84943beca Bump ws in 
/gremlin-javascript/src/main/javascript/gremlin-javascript
 add 104510a12e Fix deadlock on non response-specific read errors (#1669)
 add e991b66064 Invoking CTR as this is simple documentation and Bytecode 
exporting
 add c0fe76ba53 adding v3 back to go.mod
 add 4927388e0c updating pathing for cucumber versions
 add 6b0169e37b Gremlin-Go: Updated Docker Testing Set Up (#1665)
 add 5fb7888f4e Added getters to high and low properties in RangeLocalStep 
(#1667)
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling
 add bdca983795 Impl DateSerializer
 add 908583911a Impl VertexSerializer
 add 17e124c5cb Impl EdgeSerializer
 add 1ca2bbf130 CI: include gremlin-javascript GraphBinary unit tests
 add 20a8d85f49 Impl TimestampSerializer
 add 39695d997f Normalize style of GraphBinary unit tests
 add 85202c0cfa Impl ClassSerializer
 add 76be2bc178 Impl DoubleSerializer and FloatSerializer
 add d3296b3182 Impl SetSerializer
 add 5c1750e666 Impl PathSerializer
 add 525a3c8a71 Impl PropertySerializer
 add df55dfec6f Impl VertexPropertySerializer
 add 5c13ce8fd3 Impl LambdaSerializer
 add 2de6335fc6 Ignore Graph and Binding types
 add ad4a411d2a Impl ByteSerializer
 add fb3f18c942 Impl ByteBufferSerializer
 add 88759eaacd Impl TextPSerializer
 add fba5041d00 Impl TraversalStrategySerializer
 add 15084ad4e7 Impl BulkSetSerializer
 add b922456c79 Backup bigint based Long implementation as LongSerializerNg
 add 0b31ff973e LongSerializer: move away from bigint and align with 
GraphSON implementation
  

[tinkerpop] branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/cucumber-6.0.7 updated (9f084d0392 -> 2250cf41bd)

2022-06-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch 
dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.5-dev/cucumber-6.0.7
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


omit 9f084d0392 Bump cucumber
 add 3436b3f80a Serialization refactoring (#1648)
 add 60dd5f966a Fix for boolean writes
 add cf653e303d Bump Microsoft.NET.Test.Sdk from 17.1.0 to 17.2.0 in 
/gremlin-dotnet
 add 1a6618f5db Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/Microsoft.NET.Test.Sdk-17.2.0' into 
3.5-dev
 add ede36bb254 Bump System.Text.Json from 6.0.3 to 6.0.4 in /gremlin-dotnet
 add d90a2e0e04 Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/System.Text.Json-6.0.4' into 3.5-dev
 add b2c050fd33 Adjust export of enum traversal operations (#1650)
 add f59c47cab9 refactor ResultSet.One() to handle nil result (#1653)
 add ad6b3b7d4d Add named parameters for strategies in Gremlin-Go and 
updated translator (#1664)
 add c8db98c5f9 Gremlin-Go: Added support for additional types (#1663)
 add aa2820bc4c Adding strategy support to cucumber world for gremlin-go
 add e2bba8f3a5 Bump Moq from 4.17.2 to 4.18.1 in /gremlin-dotnet
 add ce9499467a Merge branch 
'dependabot/nuget/gremlin-dotnet/3.5-dev/Moq-4.18.1' into 3.5-dev
 add 104510a12e Fix deadlock on non response-specific read errors (#1669)
 add e991b66064 Invoking CTR as this is simple documentation and Bytecode 
exporting
 add c0fe76ba53 adding v3 back to go.mod
 add 4927388e0c updating pathing for cucumber versions
 add 6b0169e37b Gremlin-Go: Updated Docker Testing Set Up (#1665)
 add 5fb7888f4e Added getters to high and low properties in RangeLocalStep 
(#1667)
 add 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 add 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 add 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 add 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 add 8862c6f0c4 Impl UuidSerializer.deserialize()
 add 826d86f09e Impl IntSerializer.deserialize()
 add 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 add 08a09bbcec Impl StringSerializer.deserialize()
 add 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 add 431bda940e Re-structure to multiple js files
 add ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 add e9d609f19b Avoid static class fields to support old js/node
 add 23b35b968f Avoid public class fields to support old js/node
 add 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 add 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 add 79791009cd Impl ListSerializer.deserialize()
 add 0947021717 Impl TraverserSerializer.deserialize()
 add c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 add 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 add 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 add ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 add 44818e8507 Add more tests for ListSerializer
 add e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 add edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 add fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 add 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 add a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 add 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 add 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 add ec8f0a4041 Test AnySerializer.serialize() with a string item
 add f0575172b1 Test AnySerializer.serialize() with an Int32 param
 add 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 add a4ba6e481f Impl BooleanSerializer
 add 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 add 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 add 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 add 76f191ed01 Impl PSerializer
 add 9e05abc898 Fix EnumSerializer null handling
 add bdca983795 Impl DateSerializer
 add 90858391

[tinkerpop] branch 3.5-dev updated (5fb7888f4e -> 18a1ed7e04)

2022-06-02 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a change to branch 3.5-dev
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


from 5fb7888f4e Added getters to high and low properties in RangeLocalStep 
(#1667)
 new 89fe194516 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 new e07e4b2131 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 new 23a3adcc60 (WIP) gremlin-javascript: Add GraphBinary serialization 
support
 new 3971a1ea61 Impl BytecodeSerializer and GraphBinaryWriter.writeRequest
 new 8ffc09c523 GraphBinaryReader.readResponse: start with the very first 
fail cases
 new 38b0984331 Switch from String.replaceAll to String.replace for support 
of older JS/node
 new 8862c6f0c4 Impl UuidSerializer.deserialize()
 new 826d86f09e Impl IntSerializer.deserialize()
 new 44f0b2651d UuidSerializer.deserialize: add test case with nullable 
format and null value
 new 08a09bbcec Impl StringSerializer.deserialize()
 new 3d5c5bcacc graphbinary-test.js: use terminology of official docs in 
error messages
 new 431bda940e Re-structure to multiple js files
 new ff84c5ae60 Impl MapSerializer.deserialize() and 
AnySerializer.deserialize()
 new e9d609f19b Avoid static class fields to support old js/node
 new 23b35b968f Avoid public class fields to support old js/node
 new 5a6434c3d2 Impl BytecodeSerializer.deserialize()
 new 303726de51 AnySerializer test as a testable "register" of supported 
(de)serializers
 new 79791009cd Impl ListSerializer.deserialize()
 new 0947021717 Impl TraverserSerializer.deserialize()
 new c54a2ef3bf GraphBinaryReader: fix a typo: response.request_id -> 
response.requestId
 new 87f7774f4a Impl LongSerializer.serialize() and deserialize()
 new 2524085ee0 TraverserSerializer.deserialize: change {bulk} from Int to 
Long type
 new ec12769e65 Use IntSerializer.deserialize() instead of direct 
Buffer.readInt32BE()
 new 44818e8507 Add more tests for ListSerializer
 new e1848f457d Add at least one complete test for 
GraphBinaryReader.readResponse()
 new edf45799e4 Impl UnspecifiedNullSerializer.serialize() and deserialize()
 new fb24d2694e Add basic structure of AnySerializer.serialize() impl and 
testing
 new 2feeec6f1b Impl TraverserSerializer.serialize() and its support by 
AnySerializer.serialize()
 new a0ada4d113 Impl EnumSerializer (Barrier, Cardinality, Column, 
Direction, Operator, Order, Pick, Pop, Scope, T)
 new 27d094720e Impl ListSerializer.serialize() and canBeUsedFor()
 new 5bf31e6cd5 AnySerializer.serialize() errors if value is not supported 
instead of falling back to StringSerializer
 new ec8f0a4041 Test AnySerializer.serialize() with a string item
 new f0575172b1 Test AnySerializer.serialize() with an Int32 param
 new 2c8c8921c5 Switch EnumSerializer to use fully-qualified enum element 
name string
 new a4ba6e481f Impl BooleanSerializer
 new 6027fc3b10 Switch MapSerializer to Map class, but serialize() still 
can handle usual objects
 new 4a05bd14ec Add UnspecifiedNull support to AnySerializer.serialize()
 new 921d7e38ff Make EnumSerializer.deserialize() return existing EnumValue 
instances
 new 76f191ed01 Impl PSerializer
 new 9e05abc898 Fix EnumSerializer null handling
 new bdca983795 Impl DateSerializer
 new 908583911a Impl VertexSerializer
 new 17e124c5cb Impl EdgeSerializer
 new 1ca2bbf130 CI: include gremlin-javascript GraphBinary unit tests
 new 20a8d85f49 Impl TimestampSerializer
 new 39695d997f Normalize style of GraphBinary unit tests
 new 85202c0cfa Impl ClassSerializer
 new 76be2bc178 Impl DoubleSerializer and FloatSerializer
 new d3296b3182 Impl SetSerializer
 new 5c1750e666 Impl PathSerializer
 new 525a3c8a71 Impl PropertySerializer
 new df55dfec6f Impl VertexPropertySerializer
 new 5c13ce8fd3 Impl LambdaSerializer
 new 2de6335fc6 Ignore Graph and Binding types
 new ad4a411d2a Impl ByteSerializer
 new fb3f18c942 Impl ByteBufferSerializer
 new 88759eaacd Impl TextPSerializer
 new fba5041d00 Impl TraversalStrategySerializer
 new 15084ad4e7 Impl BulkSetSerializer
 new b922456c79 Backup bigint based Long implementation as LongSerializerNg
 new 0b31ff973e LongSerializer: move away from bigint and align with 
GraphSON implementation
 new df8596b11b Impl NumberSerializationStrategy
 new 61b41f6328 AnySerializer-test: mention that Set is the same as List
 new a84f6b1d9e Turn off LongSerializerNg
 new cca51d5f46 BytecodeSerializer: fix sources serialization
 new 2faf27fc1e Make TraversalStrategySerializer use FQCN
 new 33c326d249 AnySerializer: fix LONG deserialization test
 new 668e7a94d5 Fix ReservedKeysVerificationStrategy.fqcn field value
 new 142d04c0e3 gremlin-javascript: run tests for two protocols 
(application/vnd.

[GitHub] [tinkerpop] jorgebay merged pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


jorgebay merged PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [tinkerpop] jorgebay commented on pull request #1547: Add GraphBinary serialization support to gremlin-javascript

2022-06-02 Thread GitBox


jorgebay commented on PR #1547:
URL: https://github.com/apache/tinkerpop/pull/1547#issuecomment-1144538202

   I'll merge it and do the downstream merges `3.6-dev` and `master`.
   
   Following those changes, I'll create a small PR for the changelog/updatedocs 
and I'll ping u folks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org