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

Open to package overhaul? #201

Closed
quinnj opened this issue Aug 12, 2022 · 0 comments · Fixed by #202
Closed

Open to package overhaul? #201

quinnj opened this issue Aug 12, 2022 · 0 comments · Fixed by #202

Comments

@quinnj
Copy link
Member

quinnj commented Aug 12, 2022

At RelationalAI, @Drvi and I have undertaken doing a native-julia protoc implementation (called protojl). It parses proto files and can generate the appropriate modules/structs/serialize/deserialize for full protobuf integration. It solves some of the long-standing issues people have had with protobuf in Julia: weight of the protoc binary dependency on runtime applications, performance/allocations of serialize/deserialize, and improves coverage over the entire protobuf format specification.

We've started using it internally at RAI, but our goal since the beginning was to contribute back to the community, so robustness of the code can be accelerated and everyone can benefit. In lieu of starting a 2nd package, we'd rather just overhaul this existing package, as long as current maintainers agree. We're happy to assume maintainership of the new code going forward and are eager to support other applications/industry use.

If there are major concerns, let's discuss here; we're planing on putting up a PR sometime next week.

cc: @tanmaykm @aviks

@Drvi Drvi mentioned this issue Aug 16, 2022
@Drvi Drvi closed this as completed in #202 Aug 17, 2022
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 a pull request may close this issue.

1 participant