Closed
Description
We had the archboard meeting on 9/14 and you can find the recording in Azure/azure-sdk#3303.
Feedback:
Schema Registry
- remove caching
- let's throw instead of returning an undefined in
getSchema
andgetSchemaProperties
, since the avro package will throw anyway and to be consistent with other languages - rename
endpoint
tofullyQualifiedNamespace
- rename
content
toschemaDefinition
- rename
KnownSerializationType
toKnownSchemaFormat
- rename
serializationType
toformat
Wont fix
see #17697 (comment) for details
- investigate using
ArrayBuffer
instead of string as the type for thecontent
parameter and property type inSchema
- investigate perf/API implications of adding a new SchemaId type to allow the ID to be stored in bytes rather than converting to string. We can expose a
toString
method for debugging, and implement comparison based on the ID
Schema Registry Avro
- implement an LRU eviction policy for the existing cache to limit max entries to 128. Send an email to the archboard team with the details of the implementation once it is done.
- make
groupName
optional in the constructor. For context, group name is not needed to deserialize. Since you have the schema Id when deserializing, this should be enough to get the schema. - investigate adding a
serialize
overload that take a schema ID as input
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment