Skip to content

[Schema Registry] Archboard feedback #17697

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 and getSchemaProperties, since the avro package will throw anyway and to be consistent with other languages
  • rename endpoint to fullyQualifiedNamespace
  • rename content to schemaDefinition
  • rename KnownSerializationType to KnownSchemaFormat
  • rename serializationType to format

Wont fix

see #17697 (comment) for details

  • investigate using ArrayBuffer instead of string as the type for the content parameter and property type in Schema
  • 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

Metadata

Assignees

Labels

ClientThis issue points to a problem in the data-plane of the library.Schema Registry

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions