Skip to content
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

[arrow] Update to 7.0.0 #23032

Merged
merged 9 commits into from
Feb 18, 2022
Merged

[arrow] Update to 7.0.0 #23032

merged 9 commits into from
Feb 18, 2022

Conversation

ianmcook
Copy link
Contributor

Updates the arrow port to version 7.0.0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.
If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/arrow/vcpkg.json

Valid values for the license field are listed at https://spdx.org/licenses/

@JonLiu1993 JonLiu1993 self-assigned this Feb 11, 2022
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Feb 11, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/arrow/vcpkg.json

Valid values for the license field can be found in the documentation

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/arrow/vcpkg.json

Valid values for the license field can be found in the documentation

@ianmcook
Copy link
Contributor Author

I tested the features locally with triplets x64-osx, x64-windows, and x64-windows-static.

Feature orc fails because Arrow now requires version 1.7.2 of ORC (apache/arrow#12006) and the orc vcpkg port is at version 1.6.4. I opened #23118 to ask for an update of the orc port.

Besides that, I observed no feature regressions from the current version (6.0.1).

@ianmcook ianmcook marked this pull request as ready for review February 16, 2022 00:46
@ianmcook
Copy link
Contributor Author

@GPSnoopy @jgiannuzzi here's the Arrow 7.0.0 update PR. Let me know if you encounter any issues when trying this with ParquetSharp.

@JonLiu1993 JonLiu1993 added info:reviewed Pull Request changes follow basic guidelines and removed info:reviewed Pull Request changes follow basic guidelines labels Feb 18, 2022
@vicroms vicroms merged commit fc87706 into microsoft:master Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants