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
ARROW-5846: [Java] Create Avro adapter module and add dependencies #4800
Conversation
java/adapter/avro/pom.xml
Outdated
<dependency> | ||
<groupId>org.apache.avro</groupId> | ||
<artifactId>avro</artifactId> | ||
<version>1.8.2</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think 1.9 has been released?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thanks for reminder, fixed now.
Codecov Report
@@ Coverage Diff @@
## master #4800 +/- ##
=========================================
+ Coverage 83.28% 89.6% +6.32%
=========================================
Files 277 661 +384
Lines 39642 96331 +56689
Branches 1418 0 -1418
=========================================
+ Hits 33015 86318 +53303
- Misses 6265 10013 +3748
+ Partials 362 0 -362 Continue to review full report at Codecov.
|
java/adapter/avro/pom.xml
Outdated
</executions> | ||
</plugin> | ||
</plugins> | ||
</build> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this will be necessary, since we won't be generating source filess?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, fixed.
+1, LGTM |
Related to [ARROW-5846](https://issues.apache.org/jira/browse/ARROW-5846). Author: tianchen <niki.lj@alibaba-inc.com> Closes #4800 from tianchen92/ARROW-5846 and squashes the following commits: 8107847 <tianchen> fix 2 2c5ff66 <tianchen> fix de49b3a <tianchen> add test 79c04c8 <tianchen> update maven-compiler-plugin version d60cc8b <tianchen> update version 2a060fb <tianchen> ARROW-5846: Create Avro adapter module and add dependencies
Related to ARROW-5846.