feat(api)!: remove legacy API v1/VGraph support #865
+47
−361
Conversation
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
BREAKING CHANGE: Remove legacy VGraph/protobuf API v1 in favor of Arrow API v3. Removed: - `_etl1()`, `_etl_url()`, `_check_url()` methods from pygraphistry.py - API v1 dispatch path from PlotterBase.py - `TestPlotterBindings_API_1` and `TestPlotterStylesJSON` test classes Changed: - `register(api=...)` type from `Literal[1, 3]` to `Literal[3]` - `client_session.py` type from `Literal["arrow", "vgraph"]` to `Literal["arrow"]` Migration: Users calling `graphistry.register(api=1)` must switch to `graphistry.register(api=3)` or omit the parameter (defaults to v3). 5 files changed, 47 insertions(+), 361 deletions(-)
lmeyerov
reviewed
Dec 6, 2025
| raise ValueError("Expected API version to be 1, 3, instead got (likely from API_VERSION): %s" % env_api_version) | ||
| env_api_version = 3 | ||
| elif env_api_version != 3: | ||
| raise ValueError("Expected API version to be 3 (Arrow format). Legacy API versions 1 and 2 are no longer supported. Got: %s" % env_api_version) |
Contributor
There was a problem hiding this comment.
Api equal 3. Supports arrow, parquet, CSV, json and orc via file api. Basically anything that cudf takes. I do think we do happen to do arrow uploads for DFs.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
BREAKING CHANGE: Remove legacy VGraph/protobuf API v1 in favor of Arrow API v3.
Removed:
_etl1(),_etl_url(),_check_url()methods from pygraphistry.pyTestPlotterBindings_API_1andTestPlotterStylesJSONtest classesChanged:
register(api=...)type fromLiteral[1, 3]toLiteral[3]client_session.pytype fromLiteral["arrow", "vgraph"]toLiteral["arrow"]Migration: Users calling
graphistry.register(api=1)must switch tographistry.register(api=3)or omit the parameter (defaults to v3).5 files changed, 47 insertions(+), 361 deletions(-)