-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EXP: [C++] Require C++20 #45445
base: main
Are you sure you want to change the base?
EXP: [C++] Require C++20 #45445
Conversation
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?
or
See also: |
394fc03
to
68d69e2
Compare
@github-actions crossbow submit -g cpp -g python |
This comment was marked as outdated.
This comment was marked as outdated.
68d69e2
to
948c651
Compare
948c651
to
318b69a
Compare
@github-actions crossbow submit -g cpp -g python -g wheel |
Revision: 318b69a Submitted crossbow builds: ursacomputing/crossbow @ actions-433ce83702 |
@github-actions crossbow submit -g linux -g r |
Revision: 318b69a Submitted crossbow builds: ursacomputing/crossbow @ actions-e3ccd41a98 |
So, it seems two main issues may be blocking this:
Edit: a couple R builds also seem to have a non-C++20 compliant stdlib. @kou What is our policy for Almalinux and CentOS support? |
We can use
|
I see, thanks. So there are probably no insurmountable issues against switching to C++20. Of course, this will have to be discussed on the ML at some point (and there's no urgency). |
Right. FYI: There is a related issue: #44792 |
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?