Skip to content

Clarify and add tests for schema change scenarios - #2669

Merged
sijie merged 1 commit into
apache:masterfrom
ivankelly:consumer-doesnt-update-schema
Sep 30, 2018
Merged

Clarify and add tests for schema change scenarios#2669
sijie merged 1 commit into
apache:masterfrom
ivankelly:consumer-doesnt-update-schema

Conversation

@ivankelly

@ivankelly ivankelly commented Sep 27, 2018

Copy link
Copy Markdown
Contributor

A producer with a schema can attach to a topic if

  • the topic does not exist
  • the topic exists but currently has no schema
  • the topic exists and the schema is compatible with the producer
    schema

A consumer with a schema can attach to a topic if

  • the topic does not exist (the schema from the consumer is added to
    the topic)
  • the topic exists and the schema is compatible with the consumer
    schema

A producer without a schema can attach if

  • the topic does not exist
  • the topic exists and has no schema

A consumer without a schema can attach if

  • the topic exists, either with or without a schema

A producer with a schema can attach to a topic if
- the topic does not exist
- the topic exists but currently has no schema
- the topic exists and the schema is compatible with the producer
  schema

A consumer with a schema can attach to a topic if
- the topic does not exist (the schema from the consumer is added to
  the topic)
- the topic exists and the schema is compactible with the consumer
  schema

A producer without a schema can attach if
- the topic does not exist
- the topic exists and has no schema

A consumer without a schema can attach if
- the topic exists, either with or without a schema
@ivankelly ivankelly self-assigned this Sep 27, 2018
@ivankelly

Copy link
Copy Markdown
Contributor Author

rerun java8 tests
rerun cpp tests

@ivankelly

Copy link
Copy Markdown
Contributor Author

rerun integration tests

1 similar comment
@ivankelly

Copy link
Copy Markdown
Contributor Author

rerun integration tests

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.

3 participants