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

Bug/778 #2

Open
wants to merge 139 commits into
base: bug/776
Choose a base branch
from
Open

Bug/778 #2

wants to merge 139 commits into from

Conversation

dcsobral
Copy link

@dcsobral dcsobral commented May 1, 2018

This had to be based on top of bug/776 or there would have been conflicts merging both. I'll submit a single PR to Confluent fixing both bugs at the same time, I suppose.

Alex Ayars and others added 30 commits February 13, 2018 11:34
* CLIENTS-986 Forward Auth headers
* Add link from Schema Registry docs to cp-demo

* DOCS-385: Addressing comments from Joel
mageshn and others added 25 commits June 7, 2018 20:22
…figs and allow unprefixed versions (confluentinc#837)

* Deprecate schema.registry. prefixed resource extension class config and provide unprefixed version

* Deprecate schema.registry. prefixed inter instance protocol config and provide unprefixed version

* Deprecate schema.registry. prefixed basic.auth.user.info config and provide unprefixed version

* fix tests with Java 10
…nt change took away the by subject cache. This will do that again when we need schema versions. (confluentinc#839)
@dcsobral dcsobral force-pushed the bug/778 branch 2 times, most recently from 0e30415 to 781c2e6 Compare July 6, 2018 17:15
dcsobral added 2 commits July 6, 2018 11:29
Because fixed can appear multiple times in an enum, being a named type,
enough information must be preserved to correctly identify the union
index, which is mostly easily managed by simply reifying fixed types
with their name and size.
When setting default values for unions, encapsulate the value inside
a struct so that it matches the schema to which the default value is
being assigned.
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.