Skip to content

Backport TINKERPOP-3017 Added AnonymousTraversalSource.with() to 3.8#3073

Merged
Cole-Greer merged 2 commits intoapache:3.8-devfrom
andreachild:3.8-backport-TINKERPOP-3017
Mar 17, 2025
Merged

Backport TINKERPOP-3017 Added AnonymousTraversalSource.with() to 3.8#3073
Cole-Greer merged 2 commits intoapache:3.8-devfrom
andreachild:3.8-backport-TINKERPOP-3017

Conversation

@andreachild
Copy link
Copy Markdown
Contributor

https://issues.apache.org/jira/browse/TINKERPOP-3017

Replaces withEmbedded() and withRemote() and removes the long deprecated withGraph(). A single unified with() removes the need to check the object being given to the AnonymousTraversalSource to decide which method to use. Can't remember why that split was used in the first place. This should make it easier to explain how to use the AnonymousTraversalSource in the first place, requires a bit less typing and simply streamlines the API.

This is a back port of #2366 from master (4.0).

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 14, 2025

Codecov Report

Attention: Patch coverage is 25.00000% with 9 lines in your changes missing coverage. Please review.

Please upload report for BASE (3.8-dev@a986e98). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...in/process/traversal/AnonymousTraversalSource.java 25.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             3.8-dev    #3073   +/-   ##
==========================================
  Coverage           ?   76.37%           
  Complexity         ?    13244           
==========================================
  Files              ?     1062           
  Lines              ?    61577           
  Branches           ?     7347           
==========================================
  Hits               ?    47031           
  Misses             ?    12033           
  Partials           ?     2513           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Replaces withEmbedded() and withRemote() and removes the long deprecated withGraph(). A single unified with() removes the need to check the object being given to the AnonymousTraversalSource to decide which method to use. Can't remember why that split was used in the first place. This should make it easier to explain how to use the AnonymousTraversalSource in the first place, requires a bit less typing and simply streamlines the API.
@andreachild andreachild force-pushed the 3.8-backport-TINKERPOP-3017 branch from 6c980d9 to e60ccd2 Compare March 14, 2025 16:55
@Cole-Greer
Copy link
Copy Markdown
Contributor

VOTE +1

@kenhuuu
Copy link
Copy Markdown
Contributor

kenhuuu commented Mar 15, 2025

VOTE +1.

@kenhuuu
Copy link
Copy Markdown
Contributor

kenhuuu commented Mar 15, 2025

Merge note: when this gets merged up to master, remove the CHANGELOG and UPGRADE entries from master as they will now be included in the 3.8 sections.

@Cole-Greer
Copy link
Copy Markdown
Contributor

Merging as CTR

@Cole-Greer Cole-Greer merged commit 8c6f908 into apache:3.8-dev Mar 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants