GH-38874: [C++][Parquet] Minor: making parquet TypedComparator operation as const method

[mapleFU]

Rationale for this change

parquet::TypedComparator<DType> is not const method, which should be const

What changes are included in this PR?

Change Compare, GetMinMax, GetMinMaxSpaced to const

Are these changes tested?

No

Are there any user-facing changes?

No

• Closes: [C++][Parquet] Minor: making parquet TypedComparator operation as const method #38874

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

In the case of PARQUET issues on JIRA the title also supports:

PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - How to contribute patches

[github-actions]

⚠️ GitHub issue #38874 has been automatically assigned in GitHub to PR creator.

[kou]

+1

[mapleFU]

Will merge it tonight if no further comments

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit c614014.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.