Skip to content

fix(api): expose upsert API#14

Merged
utay merged 2 commits into
altertable-ai:mainfrom
albert20260301:fix/upsert-endpoint
Jun 10, 2026
Merged

fix(api): expose upsert API#14
utay merged 2 commits into
altertable-ai:mainfrom
albert20260301:fix/upsert-endpoint

Conversation

@albert20260301

@albert20260301 albert20260301 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rename the public file write API from client.upload to client.upsert.
  • Rename UploadMode to UpsertMode.
  • Remove the upsert content_type argument and stop sending a Content-Type header for raw upsert bodies.
  • Keep the /upsert endpoint and optional mode query parameter.

Impact analysis

  • Public API changes: callers now use client.upsert(...); mode enum import changes to UpsertMode.
  • Tests/docs changed: tests and README use upsert naming and no content-type option.
  • Cross-SDK scope: same feedback is applied to Go, JS, Ruby, Rust, and PHP.

Validation

  • python3 -m compileall src tests
  • Not run locally: pytest; Python exists but pip/test dependencies are not installed in this container.

@albert20260301

Copy link
Copy Markdown
Contributor Author

@leo-altertable could you review this when you have a slot? CI is green and this is the Lakehouse /upload to /upsert endpoint update for Python.

@albert20260301 albert20260301 changed the title fix(api): use upsert endpoint fix(api): expose upsert API Jun 10, 2026
@utay utay merged commit ae07019 into altertable-ai:main Jun 10, 2026
9 checks passed
@albert20260301 albert20260301 deleted the fix/upsert-endpoint branch June 10, 2026 20:22
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.

2 participants