Skip to content

Support registering the table zoo to a local server#11346

Merged
abey79 merged 7 commits into
mainfrom
antoine/register-table-zoo
Sep 29, 2025
Merged

Support registering the table zoo to a local server#11346
abey79 merged 7 commits into
mainfrom
antoine/register-table-zoo

Conversation

@abey79
Copy link
Copy Markdown
Member

@abey79 abey79 commented Sep 26, 2025

Improve the table zoo script to support registering on a local server.

Note:

  • OSS server doesn't support tables yet
  • local only for now (for a remote, we'd need the ability to save the table to some bucket for which we have authorisation, yada)
  • moved to examples (from tests) because there our dependency linting is more lenient, and I didn't want to introduce a "main" dependency on lancedb).

@abey79 abey79 added 🧑‍💻 dev experience developer experience (excluding CI) exclude from changelog PRs with this won't show up in CHANGELOG.md labels Sep 26, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 26, 2025

Web viewer built successfully.

Result Commit Link Manifest
f36c81c https://rerun.io/viewer/pr/11346 +nightly +main

Note: This comment is updated whenever you push a commit.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 26, 2025

Latest documentation preview deployed successfully.

Result Commit Link
f36c81c https://landing-lmvusu4k6-rerun.vercel.app/docs

Note: This comment is updated whenever you push a commit.

@abey79 abey79 marked this pull request as ready for review September 26, 2025 14:02
Copy link
Copy Markdown
Member

@timsaucer timsaucer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this looks great. Moving it out of tests will lower our coverage, though. I think this will potentially be helpful for the OSS server once we implement the register APIs.

@abey79
Copy link
Copy Markdown
Member Author

abey79 commented Sep 29, 2025

Overall this looks great. Moving it out of tests will lower our coverage, though. I think this will potentially be helpful for the OSS server once we implement the register APIs.

Yeah somehow depending on lance in pixi.toml is inevitable I guess. When that happens, we should probably move this back to tests where it really belongs, I agree.

@abey79 abey79 merged commit 6503ef2 into main Sep 29, 2025
71 of 72 checks passed
@abey79 abey79 deleted the antoine/register-table-zoo branch September 29, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧑‍💻 dev experience developer experience (excluding CI) exclude from changelog PRs with this won't show up in CHANGELOG.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants