-
-
Notifications
You must be signed in to change notification settings - Fork 90
chore(deps): bump gremlin.version from 3.7.4 to 3.8.0 #2808
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🧪 CI InsightsHere's what we observed from your CI run for 603aa1c. 🟢 All jobs passed!But CI Insights is watching 👀 |
|
@dependabot rebase |
13282ed to
e09cfd5
Compare
This commit fixes compatibility issues with TinkerPop 3.8.0: 1. **GremlinValueComparator.comparable()**: Changed visibility from private to public - TinkerPop 3.8.0 Compare class requires public access to this method - Replaced removed GremlinTypeErrorException with IllegalStateException - Updated throwTypeError() method to use standard exception 2. **GremlinTest.testLongOverflow()**: Updated test expectations - TinkerPop 3.8.0 changed behavior: now throws ArithmeticException on long overflow - Previous versions promoted Long to BigInteger automatically - Test now expects ArithmeticException for overflow scenarios - Added AssertJ assertThatThrownBy import These changes ensure full compatibility with Apache TinkerPop 3.8.0 while maintaining ArcadeDB's performance optimizations for type comparisons. Related: #2808 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
TinkerPop 3.8.0 Compatibility Fixes AppliedI've successfully applied compatibility fixes for the TinkerPop 3.8.0 upgrade. The changes address API modifications in the new version. Changes Made1. GremlinValueComparator.java - Method Visibility Update
2. GremlinTest.java - Test Expectations Update
Test Results✅ Build: Successful (all modules compile) CompatibilityThese changes maintain full compatibility with:
The PR is now ready for review and merge. 🤖 Automated by Claude Code via |
Bumps `gremlin.version` from 3.7.4 to 3.8.0. Updates `org.apache.tinkerpop:gremlin-core` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) Updates `org.apache.tinkerpop:gremlin-server` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) Updates `org.apache.tinkerpop:gremlin-driver` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) Updates `org.apache.tinkerpop:gremlin-util` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) Updates `org.apache.tinkerpop:gremlin-groovy` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) Updates `org.apache.tinkerpop:gremlin-test` from 3.7.4 to 3.8.0 - [Changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc) - [Commits](apache/tinkerpop@3.7.4...3.8.0) --- updated-dependencies: - dependency-name: org.apache.tinkerpop:gremlin-core dependency-version: 3.8.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.tinkerpop:gremlin-server dependency-version: 3.8.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.tinkerpop:gremlin-driver dependency-version: 3.8.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.tinkerpop:gremlin-util dependency-version: 3.8.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.tinkerpop:gremlin-groovy dependency-version: 3.8.0 dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.apache.tinkerpop:gremlin-test dependency-version: 3.8.0 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
This commit fixes compatibility issues with TinkerPop 3.8.0: 1. **GremlinValueComparator.comparable()**: Changed visibility from private to public - TinkerPop 3.8.0 Compare class requires public access to this method - Replaced removed GremlinTypeErrorException with IllegalStateException - Updated throwTypeError() method to use standard exception 2. **GremlinTest.testLongOverflow()**: Updated test expectations - TinkerPop 3.8.0 changed behavior: now throws ArithmeticException on long overflow - Previous versions promoted Long to BigInteger automatically - Test now expects ArithmeticException for overflow scenarios - Added AssertJ assertThatThrownBy import These changes ensure full compatibility with Apache TinkerPop 3.8.0 while maintaining ArcadeDB's performance optimizations for type comparisons. Related: #2808 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
cdc4b5f to
603aa1c
Compare
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesFootnotes
|
This commit fixes compatibility issues with TinkerPop 3.8.0: 1. **GremlinValueComparator.comparable()**: Changed visibility from private to public - TinkerPop 3.8.0 Compare class requires public access to this method - Replaced removed GremlinTypeErrorException with IllegalStateException - Updated throwTypeError() method to use standard exception 2. **GremlinTest.testLongOverflow()**: Updated test expectations - TinkerPop 3.8.0 changed behavior: now throws ArithmeticException on long overflow - Previous versions promoted Long to BigInteger automatically - Test now expects ArithmeticException for overflow scenarios - Added AssertJ assertThatThrownBy import These changes ensure full compatibility with Apache TinkerPop 3.8.0 while maintaining ArcadeDB's performance optimizations for type comparisons. Related: #2808 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit fixes compatibility issues with TinkerPop 3.8.0: 1. **GremlinValueComparator.comparable()**: Changed visibility from private to public - TinkerPop 3.8.0 Compare class requires public access to this method - Replaced removed GremlinTypeErrorException with IllegalStateException - Updated throwTypeError() method to use standard exception 2. **GremlinTest.testLongOverflow()**: Updated test expectations - TinkerPop 3.8.0 changed behavior: now throws ArithmeticException on long overflow - Previous versions promoted Long to BigInteger automatically - Test now expects ArithmeticException for overflow scenarios - Added AssertJ assertThatThrownBy import These changes ensure full compatibility with Apache TinkerPop 3.8.0 while maintaining ArcadeDB's performance optimizations for type comparisons. Related: #2808 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Roberto Franchini <ro.franchini@gmail.com> (cherry picked from commit 18177c7)
Bumps
gremlin.versionfrom 3.7.4 to 3.8.0.Updates
org.apache.tinkerpop:gremlin-corefrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-core's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)Updates
org.apache.tinkerpop:gremlin-serverfrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-server's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)Updates
org.apache.tinkerpop:gremlin-driverfrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-driver's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)Updates
org.apache.tinkerpop:gremlin-utilfrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-util's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)Updates
org.apache.tinkerpop:gremlin-groovyfrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-groovy's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)Updates
org.apache.tinkerpop:gremlin-testfrom 3.7.4 to 3.8.0Changelog
Sourced from org.apache.tinkerpop:gremlin-test's changelog.
... (truncated)
Commits
f9056afTinkerPop 3.8.0 release532d836Add gremlin-mcp exclusions to source-release56422ceMerge branch '3.7-dev' into 3.8-devb05a285TinkerPop 3.7.5 releasea735e41fix source release exclusions876827cVarious upgrade doc fixes for 3.8.0. (#3279)743fd1dFix for JavaTranslator handling of has with null last argument (#3278)337dc22Merge branch '3.7-dev' into 3.8-dev85449b0Deflake tests in GraphSONTypedCompatibilityTest, GraphSONUntypedCompatibility...902ab86Updated changelog and upgrade docs for release. (#3277)You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)