[GitHub] [tinkerpop] simonz-bq opened a new pull request, #1709: TINKERPOP-2617 Clean up Java connection pool logic

2022-06-15 Thread GitBox
simonz-bq opened a new pull request, #1709: URL: https://github.com/apache/tinkerpop/pull/1709 Removed duplication while fetching a connection form the ConnectionPool. Also, handled some potential edge cases where a user specified a minimum connections size of 0, where there could potential

[GitHub] [tinkerpop] codecov-commenter commented on pull request #1708: updated logback to 1.2.11

2022-06-15 Thread GitBox
codecov-commenter commented on PR #1708: URL: https://github.com/apache/tinkerpop/pull/1708#issuecomment-1157092335 # [Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1708?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache

[tinkerpop] branch master updated (ceb6b06c92 -> 8e8d19d231)

2022-06-15 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 ceb6b06c92 Merge branch '3.6-dev' new 4675011d96 Gremlin Dotnet Windows Build fixes (#113) new 3f9c49

[GitHub] [tinkerpop] vkagamlyk opened a new pull request, #1708: updated logback to 1.2.11

2022-06-15 Thread GitBox
vkagamlyk opened a new pull request, #1708: URL: https://github.com/apache/tinkerpop/pull/1708 https://issues.apache.org/jira/browse/TINKERPOP-2746 updated logback to latest 1.2.11 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tinkerpop] spmallette commented on pull request #1690: TINKERPOP-2749: Gremlin Dotnet Windows Build fixes

2022-06-15 Thread GitBox
spmallette commented on PR #1690: URL: https://github.com/apache/tinkerpop/pull/1690#issuecomment-1157084467 Cherry-picked as 4675011d960c60b499b1c6300591fdd2968ac8c6 to get rid of the merge commit. I had a conflict merging to 3.6-dev and wasn't sure how to proceed so I kept what was in 3.6

[GitHub] [tinkerpop] spmallette closed pull request #1690: TINKERPOP-2749: Gremlin Dotnet Windows Build fixes

2022-06-15 Thread GitBox
spmallette closed pull request #1690: TINKERPOP-2749: Gremlin Dotnet Windows Build fixes URL: https://github.com/apache/tinkerpop/pull/1690 -- 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 specif

[tinkerpop] branch 3.6-dev updated (01663308b8 -> 3f9c492d13)

2022-06-15 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 01663308b8 Minor bumps to hadoop/spark versions add 4675011d96 Gremlin Dotnet Windows Build fixes (#113)

[tinkerpop] branch 3.5-dev updated (86919292a5 -> 4675011d96)

2022-06-15 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch 3.5-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from 86919292a5 Fix non-determinism in test shouldSerializeToTreeJson() add 4675011d96 Gremlin Dotnet Windows

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

2022-06-15 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 3f9c492d13c055dd4e8594692c5970d36b2b714f Merge: 01663308b8 4675011d96 Author: Stephen Mallette AuthorDate: Wed Ju

[tinkerpop] 01/03: Gremlin Dotnet Windows Build fixes (#113)

2022-06-15 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 4675011d960c60b499b1c6300591fdd2968ac8c6 Author: Rithin Kumar <36722714+l0lma...@users.noreply.github.com> AuthorD

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

2022-06-15 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 8e8d19d231fc3cd25659f46adabe95ce47973598 Merge: ceb6b06c92 3f9c492d13 Author: Stephen Mallette AuthorDate: Wed Ju

[GitHub] [tinkerpop] jroimartin commented on a diff in pull request #1700: gremlin-go: support per-request arguments in bytecode

2022-06-15 Thread GitBox
jroimartin commented on code in PR #1700: URL: https://github.com/apache/tinkerpop/pull/1700#discussion_r898489406 ## gremlin-go/driver/request.go: ## @@ -83,6 +88,85 @@ func makeBytecodeRequest(bytecodeGremlin *Bytecode, traversalSource string, sess } } +// allowedR

[GitHub] [tinkerpop] simonz-bq commented on pull request #1700: gremlin-go: support per-request arguments in bytecode

2022-06-15 Thread GitBox
simonz-bq commented on PR #1700: URL: https://github.com/apache/tinkerpop/pull/1700#issuecomment-1156985391 Overall, from a code perspective LGTM. Will let someone more familiar with the feature itself review as well. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [tinkerpop] simonz-bq commented on a diff in pull request #1700: gremlin-go: support per-request arguments in bytecode

2022-06-15 Thread GitBox
simonz-bq commented on code in PR #1700: URL: https://github.com/apache/tinkerpop/pull/1700#discussion_r898466000 ## gremlin-go/driver/request.go: ## @@ -83,6 +88,85 @@ func makeBytecodeRequest(bytecodeGremlin *Bytecode, traversalSource string, sess } } +// allowedRe

[GitHub] [tinkerpop] li-boxuan commented on a diff in pull request #1657: TINKERPOP-2742 Give graph providers a hint of property cardinality

2022-06-15 Thread GitBox
li-boxuan commented on code in PR #1657: URL: https://github.com/apache/tinkerpop/pull/1657#discussion_r898463715 ## tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java: ## @@ -411,6 +404,15 @@ public boolean willAllowId(final Ob

[GitHub] [tinkerpop] spmallette commented on pull request #1698: Complete ResponseStatusCodeEnum in .NET

2022-06-15 Thread GitBox
spmallette commented on PR #1698: URL: https://github.com/apache/tinkerpop/pull/1698#issuecomment-1156955093 weird that these were missing. i guess we dont have tests for those. VOTE +1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [tinkerpop] spmallette commented on a diff in pull request #1657: TINKERPOP-2742 Give graph providers a hint of property cardinality

2022-06-15 Thread GitBox
spmallette commented on code in PR #1657: URL: https://github.com/apache/tinkerpop/pull/1657#discussion_r898437742 ## tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java: ## @@ -411,6 +404,15 @@ public boolean willAllowId(final O

[GitHub] [tinkerpop] dependabot[bot] closed pull request #1612: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /spark-gremlin

2022-06-15 Thread GitBox
dependabot[bot] closed pull request #1612: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /spark-gremlin URL: https://github.com/apache/tinkerpop/pull/1612 -- 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 t

[GitHub] [tinkerpop] dependabot[bot] commented on pull request #1612: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /spark-gremlin

2022-06-15 Thread GitBox
dependabot[bot] commented on PR #1612: URL: https://github.com/apache/tinkerpop/pull/1612#issuecomment-1156789766 Looks like com.fasterxml.jackson.core:jackson-databind is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [tinkerpop] dependabot[bot] closed pull request #1613: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /hadoop-gremlin

2022-06-15 Thread GitBox
dependabot[bot] closed pull request #1613: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /hadoop-gremlin URL: https://github.com/apache/tinkerpop/pull/1613 -- 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

[GitHub] [tinkerpop] dependabot[bot] commented on pull request #1613: Bump jackson-databind from 2.12.3 to 2.12.6.1 in /hadoop-gremlin

2022-06-15 Thread GitBox
dependabot[bot] commented on PR #1613: URL: https://github.com/apache/tinkerpop/pull/1613#issuecomment-1156789656 Looks like com.fasterxml.jackson.core:jackson-databind is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond

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

2022-06-15 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 ceb6b06c92d4486ba54dd29bcf529624f81410c8 Merge: 6a392489b5 01663308b8 Author: Stephen Mallette AuthorDate: Wed Ju

[tinkerpop] branch 3.6-dev updated (2817f1f4be -> 01663308b8)

2022-06-15 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 2817f1f4be More dev docs CTR add 01663308b8 Minor bumps to hadoop/spark versions No new revisions were a

[tinkerpop] branch master updated (6a392489b5 -> ceb6b06c92)

2022-06-15 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 6a392489b5 Merge branch '3.6-dev' add 01663308b8 Minor bumps to hadoop/spark versions new ceb6b06c92

[GitHub] [tinkerpop] xiazcy commented on pull request #1700: gremlin-go: support per-request arguments in bytecode

2022-06-15 Thread GitBox
xiazcy commented on PR #1700: URL: https://github.com/apache/tinkerpop/pull/1700#issuecomment-1156785816 Thank you so much for submitting the PR @jroimartin, we have enabled the GitHub workflows and will be taking a closer look into this. -- This is an automated message from the Apache G

[GitHub] [tinkerpop] codecov-commenter commented on pull request #1700: gremlin-go: support per-request arguments in bytecode

2022-06-15 Thread GitBox
codecov-commenter commented on PR #1700: URL: https://github.com/apache/tinkerpop/pull/1700#issuecomment-1156773779 # [Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1700?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache

[tinkerpop] branch dependabot/nuget/gremlin-dotnet/3.5-dev/System.Text.Json-6.0.5 created (now 9f3b82f224)

2022-06-15 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/nuget/gremlin-dotnet/3.5-dev/System.Text.Json-6.0.5 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git at 9f3b82f224 Bump System.Text.Json from 6.0.4 to 6.0.5 i

[GitHub] [tinkerpop] dependabot[bot] opened a new pull request, #1707: Bump System.Text.Json from 6.0.4 to 6.0.5 in /gremlin-dotnet

2022-06-15 Thread GitBox
dependabot[bot] opened a new pull request, #1707: URL: https://github.com/apache/tinkerpop/pull/1707 Bumps [System.Text.Json](https://github.com/dotnet/runtime) from 6.0.4 to 6.0.5. Release notes Sourced from https://github.com/dotnet/runtime/releases";>System.Text.Json's releases

[GitHub] [tinkerpop] spmallette closed pull request #1658: Fix non determinism in test shouldSerializeToTreeJson()

2022-06-15 Thread GitBox
spmallette closed pull request #1658: Fix non determinism in test shouldSerializeToTreeJson() URL: https://github.com/apache/tinkerpop/pull/1658 -- 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 s

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

2022-06-15 Thread GitBox
spmallette commented on PR #1658: URL: https://github.com/apache/tinkerpop/pull/1658#issuecomment-1156301991 Seems like there was some confusion on who was going to actually perform the merge. I just quickly did it - cherry-picked back to 3.5-dev and merged forward with: 86919292a5ea2d79931

[tinkerpop] branch 3.5-dev updated (4d42ffd247 -> 86919292a5)

2022-06-15 Thread spmallette
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch 3.5-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from 4d42ffd247 Bumped prettier, types/jest, and typescript for gremlint CTR add a6ab09c8bb Minor addition to

[tinkerpop] branch 3.6-dev updated (ea5e5ce6c0 -> 2817f1f4be)

2022-06-15 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 ea5e5ce6c0 Updated package-lock.json after version bumps to gremlint CTR add a6ab09c8bb Minor addition t

[tinkerpop] branch master updated (97a7ffff2e -> 6a392489b5)

2022-06-15 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 97a72e Merge branch '3.6-dev' add a6ab09c8bb Minor addition to dev docs CTR add 86919292a5 Fix no

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

2022-06-15 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 6a392489b5c41a017d65ece60723958f35a44e95 Merge: 97a72e 2817f1f4be Author: Stephen Mallette AuthorDate: Wed Ju