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

[skip ci] pip(deps): Bump pyarrow from 8.0.0 to 10.0.0 #1731

Merged
merged 4 commits into from
Nov 10, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 31, 2022

Bumps pyarrow from 8.0.0 to 10.0.0.

Commits
  • 89f9a09 MINOR: [Release] Update versions for 10.0.0
  • cb64545 MINOR: [Release] Update .deb/.rpm changelogs for 10.0.0
  • 6776681 MINOR: [Release] Update CHANGELOG.md for 10.0.0
  • c81588e ARROW-17954: [R] Update news for 10.0 (#14337)
  • a12fce7 ARROW-17317: [Release][Docs] Normalize previous document version directory (#...
  • 7d4bc9b ARROW-18103: [Packaging][deb][RPM] Fix upload artifacts patterns (#14462)
  • 6278176 ARROW-17891: [Docs][Python] Update and sync Win section of the developers/pyt...
  • 8f03259 ARROW-17353: [Release][R] Validate binaries version (#14396)
  • a735316 ARROW-18083: [C++] Bump vendored zlib version (#14446)
  • eea4a54 ARROW-17524: [C++] Correction for fields included when reading an ORC table (...
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [pyarrow](https://github.com/apache/arrow) from 8.0.0 to 10.0.0.
- [Release notes](https://github.com/apache/arrow/releases)
- [Commits](apache/arrow@go/v8.0.0...go/v10.0.0)

---
updated-dependencies:
- dependency-name: pyarrow
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Oct 31, 2022
@l-jhon
Copy link

l-jhon commented Nov 3, 2022

Hi everyone, how are you? I'm depends on this PR, do you on when it will be possible to merge? @kukushking @jaidisido @cnfait @LeonLuttenberger Thank you! 🙏

Copy link

@ismailsimsek ismailsimsek left a comment

Choose a reason for hiding this comment

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

LGTM

@kukushking
Copy link
Contributor

Hi @l-jhon - thanks for opening this!

We're currently working on sdk for pandas 3.0 that supports distributed data frames using ray and modin. In that version at this point we're limited to pyarrow < 8 (by ray).

We'll have to support both based on if the user intends to use ray or not. For this PR, I am ok with the upgrade to 10 in main but we'll have to revise this when 3.0 is out.

@kukushking
Copy link
Contributor

FYI kicked off the tests.

@kukushking kukushking self-assigned this Nov 8, 2022
@malachi-constant

This comment was marked as outdated.

@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
  • Commit ID: 1b87bdf
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@kukushking
Copy link
Contributor

Looks like uint32 is overflowing now in tests/test_athena_parquet.py::test_unsigned_parquet when we read the results back with Athena and cast with Athena types - looking into that.

@@ -52,6 +52,9 @@ def _new_writer(
pyarrow_additional_kwargs["coerce_timestamps"] = "ms"
if "flavor" not in pyarrow_additional_kwargs:
pyarrow_additional_kwargs["flavor"] = "spark"
if "version" not in pyarrow_additional_kwargs:
# By default, use version 1.0 logical type set to maximize compatibility
pyarrow_additional_kwargs["version"] = "1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

We may revise this in 3.0.0

@malachi-constant

This comment was marked as outdated.

@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
  • Commit ID: a4f64cd
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Copy link
Contributor

@malachi-constant malachi-constant left a comment

Choose a reason for hiding this comment

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

Looks good, seems like those timestreams tests are failing fairly often now.

@kukushking
Copy link
Contributor

@malachi-constant yes those two timestream tests have been very flaky lately. We should be all right with this PR though.

@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
  • Commit ID: fac03a8
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@kukushking kukushking merged commit 8ecebec into main Nov 10, 2022
@dependabot dependabot bot deleted the dependabot/pip/pyarrow-10.0.0 branch November 10, 2022 14:34
@l-jhon
Copy link

l-jhon commented Nov 10, 2022

Thank you so much, everyone! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants