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

ARROW-5348: [Java][CI] Add missing gandiva javadoc #4327

Closed
wants to merge 2 commits into from

Conversation

emkornfield
Copy link
Contributor

No description provided.

@emkornfield emkornfield changed the title Arrow 5348 ARROW-5348: [Java] Add missing gandiva javadoc May 16, 2019
@emkornfield emkornfield changed the title ARROW-5348: [Java] Add missing gandiva javadoc ARROW-5348: [Java][CI] Add missing gandiva javadoc May 16, 2019
Copy link
Contributor

@praveenbingo praveenbingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@codecov-io
Copy link

Codecov Report

Merging #4327 into master will increase coverage by 23.7%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #4327      +/-   ##
=========================================
+ Coverage   65.49%   89.2%   +23.7%     
=========================================
  Files         527     632     +105     
  Lines       66435   86482   +20047     
  Branches     1251       0    -1251     
=========================================
+ Hits        43513   77144   +33631     
+ Misses      22686    9338   -13348     
+ Partials      236       0     -236
Impacted Files Coverage Δ
cpp/src/arrow/compute/kernel.h 62.76% <0%> (-3.34%) ⬇️
cpp/src/plasma/thirdparty/ae/ae.c 70.75% <0%> (-0.95%) ⬇️
cpp/src/arrow/csv/column-builder.h 100% <0%> (ø) ⬆️
cpp/src/arrow/table_builder.h 100% <0%> (ø) ⬆️
cpp/src/arrow/compute/kernels/cast.h 100% <0%> (ø) ⬆️
go/arrow/ipc/writer.go
go/arrow/math/uint64_amd64.go
go/arrow/memory/memory_avx2_amd64.go
go/arrow/ipc/file_reader.go
js/src/enum.ts
... and 593 more

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 4627232...67198f8. Read the comment docs.

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 this pull request may close these issues.

None yet

3 participants