snowplow / iglu-server

A RESTful schema registry
Other
13 stars 9 forks source link

Make it possible to get and set superseding schema info #130

Closed spenes closed 1 year ago

spenes commented 1 year ago

Details are in https://github.com/snowplow/iglu-server/issues/129

spenes commented 1 year ago

I wondered if some of this should be implemented in iglu-scala-core instead of iglu-server. For example, the deserializers and serializers. Maybe the definition of SelfDescribingSchema needs to change? Might we need to use similar serializers/deserializers in igluctl?

I thought about it but I couldn't be sure if we should do it or not since SelfDescribingSchema is used in a lot of places but supersededBy field will be only used by Enrich. I thought it might better to leave it as local as possible. I would vote to leave it like that for now. We can revisit while working on Igluctl for superseding schema feature maybe.