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

Questions about the future scope of this library #3

Closed
gabrielnau opened this issue Oct 16, 2015 · 2 comments
Closed

Questions about the future scope of this library #3

gabrielnau opened this issue Oct 16, 2015 · 2 comments

Comments

@gabrielnau
Copy link

Hi,

I was unfortunately working on a fork of node-avro-io these days to make it work in the browser.

My primary goal was to support long numbers. The best solution I've found to do so is to use https://github.com/dcodeIO/Long.js. Would you (or did you) consider supporting long numbers, using int64-native for node and long.js (or another one) for the browser ?

At a higher level, is the work on this library supported by Linkedin ? If so I would expect this lib to become the de facto standard Avro javascript implementation.

Unrelated question, I'm curious about the use of Avro in the browser at Linkedin: is it already in production ? Are browser generated events used "as it" later on in Samza ?

Thanks

@mtth
Copy link
Owner

mtth commented Oct 17, 2015

Hey!

Adding long support is definitely something I'm considering; I'm still thinking about how best to do so. Using an external library such as the ones you mention is an option, but I'd rather not tie avsc to any particular implementation (especially since it'll likely be two, one for node, and one for browsers). One other possibility would be to provide hooks (as an option to avsc.parse), letting you choose the best library for each use-case.

About your other questions, it's too early to tell :).

@mtth
Copy link
Owner

mtth commented Oct 17, 2015

Created a more specific issue for long support: #4. Feel free to comment there if you have any suggestions or use-cases in mind.

@mtth mtth closed this as completed Oct 17, 2015
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

No branches or pull requests

2 participants