Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tsid extension #225

Merged
merged 2 commits into from
Dec 9, 2024
Merged

Add tsid extension #225

merged 2 commits into from
Dec 9, 2024

Conversation

lmangani
Copy link
Contributor

@lmangani lmangani commented Dec 9, 2024

No description provided.

@carlopi
Copy link
Collaborator

carlopi commented Dec 9, 2024

Hi! This seems to me somehow similar to https://duckdb.org/community_extensions/extensions/ulid from @Maxxen, unsure if you were aware.

Obv there is space for everyone, but wanted to point to connected work in the area.

@lmangani
Copy link
Contributor Author

lmangani commented Dec 9, 2024

@carlopi its similar indeed (as most *IDs in general) but ULID seems barely documented and I'm not sure what the plan is. Our TSID extension uses a larger size ID and exposes functionality to extract timestamps and seed generation. I guess they can co-exist?

@lmangani lmangani marked this pull request as ready for review December 9, 2024 18:40
@carlopi carlopi merged commit c1cafd7 into duckdb:main Dec 9, 2024
37 checks passed
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