Update default version in parquet.rst

[kylebarron]

According to https://arrow.apache.org/docs/python/generated/pyarrow.parquet.write_table.html, the default Parquet version written is now 2.4, not 1.0

[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

[kylebarron]

Not sure when these docs get published, but if it's not until 13.0, maybe it would be better to update this doc to 2.6 instead of 2.4? ref #36137

[kou]

Could you read the automated comment about our contribution guide?
#37227 (comment)

Could you also use our PR template instead of removing it entirely?
(You must see it when you open this PR.)

You're right. We should use "2.6" instead of "2.4" now.

[kylebarron]

Sorry I figured that this change was small enough to be considered a minor PR. Should I create an issue for this? Or you just mean I should rename the PR title?

[kou]

(In general, it's better that contributors read contribution guide provided by a project.)

If we can use "2.4", this is a MINOR change.
See also https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes that is also linked in #37227 (comment) ?

But we can't use "2.4". We need to use "2.6". If we use "2.6", we need more changes. For example, "the nanoseconds will be cast to microseconds ('us')" isn't done with "2.6". We need to update the description too.
So this is not a MINOR change. We should create an issue for this with "2.6".

[github-actions]

Thank you for your contribution. Unfortunately, this pull request has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this PR will be closed in 14 days. Feel free to re-open this if it has been closed in error. If you do not have repository permissions to reopen the PR, please tag a maintainer.

[kylebarron]

Not stale because the docs on main here still state the default Parquet version as 1.0, but this PR still needs to be updated

[thisisnic]

Thanks @kylebarron, I've removed the label now.

[tadeja]

hi, @kylebarron! I stumbled upon this issue-less PR so I opened the required issue #50228 and suggested newer updates in PR #50228 - Could you check those are good to go? Or do you prefer to continue on this PR instead?

[AlenkaF]

Closing in favour of #50227.