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

[REST API + KafkaSQL] Significant updates to the REST API to change how artifact and version metadata behave #4410

Merged
merged 16 commits into from
Mar 13, 2024

Conversation

EricWittmann
Copy link
Member

  • Artifacts no longer have state, only versions have state
  • Removed the /state endpoints and moved state into the metadata - changing a version state is now done by updating the version metadata
  • Artifact metadata is now its own thing, not inherited from the "latest" version metadata
  • Artifacts no longer have content - only metadata and a collection of versions
  • /groups/:groupId/artifacts/:artifactId now GETs and PUTs metadata, not content
  • Removed the /groups/:groupId/artifacts/:artifactId/owner endpoint - owner is now just another metadata property

All of these changes required multiple storage layer updates which are relatively easy in SQL but annoying in KafkaSQL. As a result, I have changed the KafkaSQL layer to use more of a 1-1 RPC style approach rather than the existing approach that is based on log compaction. This means we are now committed to implementing snapshotting.

@apicurio-bot
Copy link

apicurio-bot bot commented Mar 5, 2024

Thank you for creating a pull request!

Pinging @jsenko, @carlesarnal to respond or triage.

@EricWittmann EricWittmann marked this pull request as ready for review March 13, 2024 13:35
@EricWittmann EricWittmann merged commit e057544 into main Mar 13, 2024
19 checks passed
@EricWittmann EricWittmann deleted the rest-api/artifact-version-state branch March 13, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant