Skip to content

Add delta_version function to get a table's current version#316

Closed
djouallah wants to merge 5 commits into
duckdb:v1.5-variegatafrom
djouallah:delta-version-1.5
Closed

Add delta_version function to get a table's current version#316
djouallah wants to merge 5 commits into
duckdb:v1.5-variegatafrom
djouallah:delta-version-1.5

Conversation

@djouallah

@djouallah djouallah commented May 30, 2026

Copy link
Copy Markdown
Contributor

Adds a delta_version('path') table function that returns the current (HEAD) version of a Delta table as a single row.

Closes #234

🤖 Generated with Claude Code

Exposes the current version of a Delta table via
delta_version(path), returning a single row (version UBIGINT).
Reuses the existing DeltaMultiFileList snapshot loading and the
metadata table-function plumbing.

Addresses #234

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@djouallah

Copy link
Copy Markdown
Contributor Author

@benfleis @samansmink any chance you have a look, it is only touching the read path and less dependancy on delta_rs :)

@benfleis

Copy link
Copy Markdown
Member

Hi @djouallah thanks for the ping!

I have 3 adjustments needed, which I'll apply and push:

  1. the version call should be a scalar function (instead of table fn)
  2. delta_version could refer to extension, or protocol, or table version, renamed to delta_table_version to be clear
  3. needs docs (like many other things in here!)

Thanks!!

benfleis added 4 commits June 15, 2026 09:41
- convert to scalar function
- rename: delta_version too ambiguous, use delta_table_version
- document it
@djouallah djouallah closed this by deleting the head repository Jun 19, 2026
@benfleis

Copy link
Copy Markdown
Member

Hey @djouallah did you mean to close this? It got back-burnered by the build issues, but I planned to finish it shortly.

@djouallah

Copy link
Copy Markdown
Contributor Author

Damn sorry, I deleted the repo by accident

@benfleis

Copy link
Copy Markdown
Member

I still have a copy so I can pick it up, was just checking whether I missed a reason to delete :)

@djouallah

Copy link
Copy Markdown
Contributor Author

someone from reddit, took my fork as a new project, i freakout, so i deleted the repo, sorry :)

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