Integrate Python driver examples into automated build process (master)#3280
Merged
andreachild merged 5 commits intoapache:masterfrom Nov 21, 2025
Merged
Conversation
…ate for TinkerPop 4 Changes Made: - Added example execution to gremlin-python-integration-tests container in docker-compose.yml - Made server URLs configurable via environment variables - Added configurable vertex labels via VERTEX_LABEL environment variable - Updated protocol from WebSocket (ws://) to HTTP for TinkerPop 4 compatibility - Updated serializer from GraphBinarySerializersV1 to GraphBinarySerializersV4 - Fixed edge creation syntax using __.V() wrapper for TinkerPop 4 from()/to() compatibility - Migrated example authentication from parameters to auth functions (basic(), sigv4()) - Added AWS SigV4 authentication examples for permanent and temporary credentials
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3280 +/- ##
============================================
+ Coverage 77.87% 78.37% +0.50%
- Complexity 13578 14598 +1020
============================================
Files 1015 1068 +53
Lines 59308 61893 +2585
Branches 6835 7384 +549
============================================
+ Hits 46184 48509 +2325
+ Misses 10817 10799 -18
- Partials 2307 2585 +278 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
andreachild
reviewed
Nov 17, 2025
andreachild
reviewed
Nov 18, 2025
Contributor
|
Please add a changelog entry |
andreachild
reviewed
Nov 18, 2025
| rc = DriverRemoteConnection( | ||
| 'ws://localhost:8182/gremlin', 'g', | ||
| username="", password="", kerberized_service='', | ||
| message_serializer=GraphBinarySerializersV1(), graphson_reader=None, |
Contributor
There was a problem hiding this comment.
Can this example be retained which is demonstrating how the graph binary serializer can be configured?
Contributor
Author
There was a problem hiding this comment.
Added with_configs() example in 61c0e77, showcasing graph binary serializer configuration. Are there any other configurations that should be showcased here?
Contributor
|
VOTE +1 |
2 similar comments
Contributor
|
VOTE +1 |
Contributor
|
VOTE +1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a follow-up to #3231 and includes the integration of Python driver examples into the automated build process, as well as the updates needed to make the examples work against the
mastercode.Changes Made:
Updates for
master: