Skip to content

[Doc] Add Schema Evolution and Compatibility section#4841

Merged
sijie merged 2 commits intoapache:masterfrom
Anonymitaet:schema_e_c
Aug 1, 2019
Merged

[Doc] Add Schema Evolution and Compatibility section#4841
sijie merged 2 commits intoapache:masterfrom
Anonymitaet:schema_e_c

Conversation

@Anonymitaet
Copy link
Member

@Anonymitaet Anonymitaet commented Jul 29, 2019

Structure of Schema Chapter: #4789

Copy link
Member

@sijie sijie left a comment

Choose a reason for hiding this comment

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

@Anonymitaet looks good to me. +1

| Compatibility check strategy | Description |
|---|---|
`ALWAYS_COMPATIBLE` | The compatibility check is disabled. <br>Consequently, you can upgrade the producers and consumers in **any order**. |
`ALWAYS_INCOMPATIBLE` | / |
Copy link
Member

Choose a reason for hiding this comment

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

"/" doesn't provide useful information. And it sometimes causes confusion. I would suggest adding at least one sentence for it.

Copy link
Member Author

Choose a reason for hiding this comment

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

@sijie Thank you, I've replaced "/" with "contents".

@sijie sijie added doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. component/schemaregistry labels Aug 1, 2019
@sijie sijie added this to the 2.5.0 milestone Aug 1, 2019
@sijie
Copy link
Member

sijie commented Aug 1, 2019

run integration tests

@Anonymitaet
Copy link
Member Author

run java8 tests

1 similar comment
@sijie
Copy link
Member

sijie commented Aug 1, 2019

run java8 tests

@sijie sijie merged commit 4acb838 into apache:master Aug 1, 2019
@Anonymitaet Anonymitaet deleted the schema_e_c branch August 4, 2019 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants