[GitHub] [tinkerpop] bechbd opened a new pull request #1348: TINKERPOP-2446 - Add Looping Recipe for Optional Depth

2020-10-13 Thread GitBox
bechbd opened a new pull request #1348: URL: https://github.com/apache/tinkerpop/pull/1348 Add another recipe to the looping section demonstrating how to accomplish a loop with optional depth This is an automated message

[GitHub] [tinkerpop] spmallette merged pull request #1339: TINKERPOP-2420 Per Request Options in .NET

2020-10-13 Thread GitBox
spmallette merged pull request #1339: URL: https://github.com/apache/tinkerpop/pull/1339 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

[tinkerpop] branch 3.4-dev updated (a290d35 -> 8156a51)

2020-10-13 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch 3.4-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. from a290d35 Add Gremlin Snippets link to compendium page CTR add ec5a420 TINKERPOP-2420 Added with()

[tinkerpop] branch master updated (8548860 -> 959af07)

2020-10-13 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 8548860 Merge branch '3.4-dev' add ec5a420 TINKERPOP-2420 Added with() support for request options in

[tinkerpop] 01/02: Merge branch 'TINKERPOP-2420' into 3.4-dev

2020-10-13 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a commit to branch travis-fix in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit 8156a51496f563b9a3c5c7c782490dd7fc105b53 Merge: a290d35 28456f5 Author: Stephen Mallette AuthorDate: Tue Oct

[tinkerpop] 02/02: Merge branch '3.4-dev' into master

2020-10-13 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a commit to branch travis-fix in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit 959af077c7028795eafc186592ee43786ce4 Merge: 8548860 8156a51 Author: Stephen Mallette AuthorDate: Tue Oct

[tinkerpop] branch travis-fix updated (7d1770c -> 959af07)

2020-10-13 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch travis-fix in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. discard 7d1770c merge 3.4-dev add ec5a420 TINKERPOP-2420 Added with() support for request options in C#

[tinkerpop] branch travis-fix updated: merge 3.4-dev

2020-10-13 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a commit to branch travis-fix in repository https://gitbox.apache.org/repos/asf/tinkerpop.git The following commit(s) were added to refs/heads/travis-fix by this push: new 7d1770c merge 3.4-dev 7d1770c

[GitHub] [tinkerpop] divijvaidya commented on pull request #1344: TINKERPOP-2441 Add support for WebSocket compression

2020-10-13 Thread GitBox
divijvaidya commented on pull request #1344: URL: https://github.com/apache/tinkerpop/pull/1344#issuecomment-707903925 > a small benchmark to demonstrate might be neat too. i can work on that sort of thing if you want to move on from this enhancement - just let me know. Thank you

[GitHub] [tinkerpop] spmallette commented on pull request #1341: TINKERPOP-2432 + TINKERPOP-2439: fix bytecode toString() and fix P and TextP toString()

2020-10-13 Thread GitBox
spmallette commented on pull request #1341: URL: https://github.com/apache/tinkerpop/pull/1341#issuecomment-707658949 A CHANGELOG conflict is not a big deal - easy for either of us to resolve (we just don't get a travis run to look at when there is a conflict on the PR, but that's fine, i

[GitHub] [tinkerpop] spmallette commented on pull request #1344: TINKERPOP-2441 Add support for WebSocket compression

2020-10-13 Thread GitBox
spmallette commented on pull request #1344: URL: https://github.com/apache/tinkerpop/pull/1344#issuecomment-707657089 cool - the test changes that show the difference in payload sizes are pretty compelling. this might be a smart change to promote in the upgrade documentation. a small

[GitHub] [tinkerpop] spmallette commented on pull request #1343: TINKERPOP-2440 Simplify driver by delegating keepAlive logic to Netty

2020-10-13 Thread GitBox
spmallette commented on pull request #1343: URL: https://github.com/apache/tinkerpop/pull/1343#issuecomment-707653519 Good change - Glad the `shouldKeepAliveForWebSockets()` test is back. I suppose there is nothing now testing `scheduleKeepAlive()` logic - which remains fairly complex. I