Implement ChunkReader for Bytes, deprecate SliceableCursor

[tustvold]

Deprecate SliceableCursor

Which issue does this PR close?

Part of #1163
Part of #1474

Rationale for this change

Better ecosystem interoperability, removes custom code, and may eventually allow eliding copies on read.

What changes are included in this PR?

Implement ChunkReader for Bytes and deprecates SliceableCursor

Are there any user-facing changes?

Deprecates an API

[codecov-commenter]

Codecov Report

Merging #1775 (6f8b935) into master (c1a91dc) will decrease coverage by 0.01%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##           master    #1775      +/-   ##
==========================================
- Coverage   83.39%   83.38%   -0.02%     
==========================================
  Files         198      198              
  Lines       56142    56148       +6     
==========================================
- Hits        46821    46818       -3     
- Misses       9321     9330       +9     

Impacted Files	Coverage Δ
parquet/src/util/cursor.rs	62.18% <ø> (-1.69%)	⬇️
parquet/src/file/serialized_reader.rs	94.46% <75.00%> (-1.17%)	⬇️
parquet/src/arrow/arrow_reader.rs	96.13% <100.00%> (ø)
parquet/src/arrow/arrow_writer.rs	97.77% <100.00%> (ø)
parquet/src/file/footer.rs	95.37% <100.00%> (ø)
parquet/src/file/writer.rs	92.84% <100.00%> (-0.02%)	⬇️
parquet_derive/src/parquet_field.rs	65.75% <0.00%> (-0.46%)	⬇️
arrow/src/datatypes/datatype.rs	65.79% <0.00%> (+0.37%)	⬆️

---

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 c1a91dc...6f8b935. Read the comment docs.

[alamb]

I like where this is heading (less custom stuff in parquet-rs) 👍

[alamb]

👍