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

Finish the Avro codec #309

Closed
vigoo opened this issue Aug 18, 2022 · 7 comments
Closed

Finish the Avro codec #309

vigoo opened this issue Aug 18, 2022 · 7 comments

Comments

@vigoo
Copy link
Contributor

vigoo commented Aug 18, 2022

Problems:

  • Some of its tests are failing
  • It is not implementing the Codec trait
@mjabari
Copy link
Contributor

mjabari commented Oct 10, 2022

I've looked at the code. I think the main issue here is Union in Avro schema

Unions, as mentioned above, are represented using JSON arrays. For example, ["null", "string"] declares a schema which may be either a null or string.

so we'll loose structure in a roundtrip.
although there are other open issues and a WIP for it

@rajcspsg
Copy link

@vigoo @mjabari I would like to work on this issue

@jdegoes
Copy link
Member

jdegoes commented Apr 25, 2023

/bounty $250

@algora-pbc
Copy link

algora-pbc bot commented Apr 25, 2023

💎 $250 bounty • ZIO

Steps to solve:

  1. Start working: Comment /attempt #309 with your implementation plan
  2. Submit work: Create a pull request including /claim #309 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Additional opportunities:

Thank you for contributing to zio/zio-schema!

Add a bountyShare on socials

@vidyasankarv
Copy link

vidyasankarv commented May 10, 2023

@rajcspsg if you are not working on this, I would like to try

@rajcspsg
Copy link

@vidyasankarv please go ahead!!!

@vidyasankarv
Copy link

#563 @devsprint Looks like your PR for AVRO codec on ZIO Schema might be eligible for this bounty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

6 participants