8000 Adding support for defaults from EnumSymbols by cbrentharris · Pull Request #177 · linkedin/avro-util · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Adding support for defaults from EnumSymbols #177

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

Conversation

cbrentharris
Copy link
Contributor

When creating fields with the FieldBuilder and supplying a default
value to the field builder by using the adapter's genericDefaultValue
from an existing field, it will return a GenericData.EnumSymbol.

This changeset makes Jackson1Utils GenericData.EnumSymbol aware.

When creating fields with the FieldBuilder and supplying a default
value to the field builder by using the adapter's genericDefaultValue
from an existing field, it will return a `GenericData.EnumSymbol`.

This changeset makes Jackson1Utils `GenericData.EnumSymbol` aware.
@codecov-commenter
Copy link

Codecov Report

Merging #177 (730e0b6) into master (17f362f) will increase coverage by 0.01%.
The diff coverage is 66.66%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #177      +/-   ##
============================================
+ Coverage     25.10%   25.12%   +0.01%     
- Complexity     1144     1145       +1     
============================================
  Files           188      188              
  Lines         14839    14842       +3     
  Branches       2276     2277       +1     
============================================
+ Hits           3726     3729       +3     
  Misses        10672    10672              
  Partials        441      441              
Impacted Files Coverage Δ
...inkedin/avroutil1/compatibility/Jackson1Utils.java 41.37% <66.66%> (+6.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17f362f...730e0b6. Read the comment docs.

@radai-rosenblatt radai-rosenblatt merged commit 498f9b1 into linkedin:master Aug 2, 2021
@srramach srramach added the helper related to the compatibility helper label Oct 2, 2021
flowenol pushed a commit to RTBHOUSE/avro-util that referenced this pull request Aug 23, 2023
When creating fields with the FieldBuilder and supplying a default
value to the field builder by using the adapter's genericDefaultValue
from an existing field, it will return a `GenericData.EnumSymbol`.

This changeset makes Jackson1Utils `GenericData.EnumSymbol` aware.

Co-authored-by: Christopher Harris <chharris@chharris-mn2.linkedin.biz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helper related to the compatibility helper
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0