ARROW-6199: [Java] Avro adapter avoid potential resource leak.

[tianchen92]

Related to ARROW-6199.

Currently, avro consumer interface has no close API, which may cause resource leak like AvroBytesConsumer#cacheBuffer.
To resolve this, make consumer extends AutoCloseable and create CompositeAvroConsumer to encompasses consume and close logic.

[codecov-io]

Codecov Report

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

@@            Coverage Diff             @@
##           master    #5059      +/-   ##
==========================================
+ Coverage   87.63%   89.75%   +2.11%     
==========================================
  Files        1014      674     -340     
  Lines      144947   100382   -44565     
  Branches     1437        0    -1437     
==========================================
- Hits       127030    90093   -36937     
+ Misses      17555    10289    -7266     
+ Partials      362        0     -362

Impacted Files	Coverage Δ
r/src/recordbatch.cpp
r/R/Table.R
js/src/util/fn.ts
go/arrow/array/bufferbuilder.go
r/src/symbols.cpp
rust/datafusion/src/execution/projection.rs
rust/datafusion/src/execution/filter.rs
rust/arrow/src/csv/writer.rs
rust/datafusion/src/bin/main.rs
go/arrow/ipc/file_reader.go
... and 332 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 3cc12ab...d60d94c. Read the comment docs.

[tianchen92]

@emkornfield Hi Micah, Please help take a look on this. About avro adapter there are Array, Map, Fixed type left, I would like to start follow-ups(support all types and iterator) as soon as possible to merge it before 0.15 branch cut together with Jdbc iterator. Many thanks :)

[tianchen92]

@emkornfield Please help take a look at this one :)

[tianchen92]

@emkornfield build passed, and I am curious about which time zone you are in?

[emkornfield]

+1 thank you. US/Pacific.

[emkornfield]

lets fix in one of the follow-ups. but I think looping should be done outside of the consumer interfaces.

[tianchen92]

Thank you Micah, will fix in next PR. I plan to start two PRs one by one(support all type, iterator API), then I think i could be called a initial version and hope to catch up 0.15 branch cut.