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

[avro] Add documentation for the Avro format. #2452

Merged
merged 2 commits into from
Sep 10, 2024
Merged

[avro] Add documentation for the Avro format. #2452

merged 2 commits into from
Sep 10, 2024

Conversation

ryzhyk
Copy link
Contributor

@ryzhyk ryzhyk commented Sep 9, 2024

No description provided.

/// Custom headers that will be added to every call to the schema registry.
///
/// This property is only applicable to output connectors.
///
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
///

/// nullability of columns.
/// When this property is set, the connector uses the provided schema instead of
/// retrieving the schema from the schema registry. This setting is mutually exclusive
/// with `registry_urls`.
pub schema: Option<String>,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't we usually use Object for json stuff?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what's an object. We could try parsing this into serde_json::Value, but this is an escaped string, not a regular piece of JSON, so not sure how to do this exactly, plus the library function for deserializing schemas works with strings.

docs/formats/avro.md Outdated Show resolved Hide resolved
docs/formats/avro.md Outdated Show resolved Hide resolved

* **Raw** - every message contains a single Avro-encoded record that represents a row in a SQL
table or view. This format does not carry any additional metadata that can be used to distinguish
inserts and deletes. It is therefore only suitable for representing inserts and upserts, but not
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add link to where we explain how upserts work in feldera (if we have one?)

docs/formats/avro.md Outdated Show resolved Hide resolved
The Avro encoder generates an Avro schema, which the consumer can use to decode messages produced
by the encoder. For message formats that include key and value components with different schemas,
e.g., the Confluent JDBC connector format, the encoder generates both key and value schemas.
If the connector configuration specifies a schema registry, the encoder publishes both
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add link to the relevant example

If the connector configuration specifies a schema registry, the encoder publishes both
key and value schemas in the registry.

The encoder supports an alternative workflow where users provide the schema as a JSON string
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another link to example

docs/formats/avro.md Outdated Show resolved Hide resolved
docs/formats/avro.md Outdated Show resolved Hide resolved
Signed-off-by: Leonid Ryzhyk <leonid@feldera.com>
@ryzhyk ryzhyk added this pull request to the merge queue Sep 10, 2024
Merged via the queue into main with commit c687a40 Sep 10, 2024
5 checks passed
@ryzhyk ryzhyk deleted the avro-docs branch September 10, 2024 10:10
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