[C++][Parquet] Fix backwards compatibility for ParquetV2 data pages written prior to 3.0.0 per ARROW-10353

[asfimport]

As described in https://lists.apache.org/thread/xkrhgfpk9sr1mj74d4chz3r5yp3szt6c,

ef0feb2

Caused some files written prior to 3.0.0 to be unreadable. Given that the patch was small, this will hopefully not be too difficult to fix

Reporter: Will Jones / @wjones127
Assignee: Will Jones / @wjones127

Related issues:

• [C++] Parquet decompresses DataPageV2 pages even if is_compressed==0 (is caused by)

PRs and other links:

• GitHub Pull Request #80
• GitHub Pull Request #13665

_{Note: This issue was originally created as ARROW-17100. Please see the migration documentation for further details.}

[asfimport]

Antoine Pitrou / @pitrou:
That changeset is ARROW-10353, which fixes bugs both in the read and write path for V2 data pages. On the write side, Parquet C++ used not to always set is_compressed = false in the data page, regardless of compression. On the read side, Parquet C++ used to always decompress, regardless of the is_compressed flag.

[asfimport]

Antoine Pitrou / @pitrou:
Note that this issue has existed since 3.0.0, so the blocker classification seems a bit exagerated to me.

[asfimport]

Raúl Cumplido / @raulcd:
This is identified as one of the tickets required to create the first 9.0.0 RC, should this block the release?

[asfimport]

David Li / @lidavidm:
Issue resolved by pull request 13665
#13665