Skip to content

Protobuf Dependency Management #1737

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

Closed
bakjos opened this issue Aug 2, 2021 · 4 comments · Fixed by #2272
Closed

Protobuf Dependency Management #1737

bakjos opened this issue Aug 2, 2021 · 4 comments · Fixed by #2272
Assignees
Labels
type/bug Something isn't working

Comments

@bakjos
Copy link

bakjos commented Aug 2, 2021

I was trying to use a proto file with additional dependencies, the way this is being handled by the Confluent schema registry is to create a subject per external dependency and resolve it when it's being queried. If I try to do the same using the Apicurio schema registry, the write works well, but when I try to deserialize the data, it fails because it can not resolve one type. I couldn't find a way to do write the external dependencies to the schema registry and use them when reading. This was also working fine with the previous version (1.3.1.Final) using the PROTOBUF_FD

@EricWittmann EricWittmann added the type/bug Something isn't working label Aug 16, 2021
@EricWittmann
Copy link
Member

Dependencies/references are not currently supported in Apicurio Registry. This is on our backlog but just not implemented yet.

@einarjohnson
Copy link

einarjohnson commented Sep 6, 2021

@EricWittmann , do you have a rough ETA on when this may be implemented? I am currently working on a large project where we use Protobuf to define events posted to our Kafka infrastructure. All the Protobuf schemas are using package and import for code re-usability and we have been looking into adopting the Apicurio Registry. This is a must-have feature for us to proceed.

@EricWittmann
Copy link
Member

I'm sorry to say I don't have a timeframe for this - it's high on our backlog but not yet addressed. Other priorities are preventing this.

@carlesarnal this is a part of the ref work. :)

@einarjohnson
Copy link

ok, thanks for getting back to me @EricWittmann

@carlesarnal carlesarnal self-assigned this Sep 21, 2021
@carlesarnal carlesarnal linked a pull request Feb 22, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants