Commits


Antoine Pitrou authored and GitHub committed c2123b8b90a
GH-40154: [C++][Parquet] Separate encoders and decoder (#43972) ### Rationale for this change `encoding.cc` is quite large nowadays : around 4000 lines of code, which makes code navigation cumbersome. It combines the functionality of encoders and decoders, however, those use distinct infrastructures and do not share any code. Other areas of Parquet tend to separate the reading and writing facilities: for example, `column_reader.cc` vs. `column_writer.cc`. ### What changes are included in this PR? The main change is to move encoders to `encoder.cc`, decoders to `decoder.cc`, and remove `encoding.cc`. A small improvement is also to remove the inclusion of `arrow/util/spaced.h` in `encoding.h` by moving the `TypedDecoder<T>::DecodeSpaced` implementation into `decoder.cc`. Note the massive code shuffle may obscure the git history quite a bit. `git log -C` doesn't seem able to track earlier versions of the encoder and decoder code, but `git blame -C` is. ### Are these changes tested? By existing tests. ### Are there any user-facing changes? No. * GitHub Issue: #40154 Authored-by: Antoine Pitrou <antoine@python.org> Signed-off-by: Antoine Pitrou <antoine@python.org>