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

refactor: Bump arrow to 52 #15943

Merged
merged 16 commits into from
Jul 2, 2024
Merged

refactor: Bump arrow to 52 #15943

merged 16 commits into from
Jul 2, 2024

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jul 1, 2024

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Part of #15809

This PR is a blocker of iceberg migration since iceberg-rust is depending on arrow 52.

bendsql needs extra PR to bump arrow 52, but it can be processed later.

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo requested a review from youngsofun July 1, 2024 13:46
@github-actions github-actions bot added the pr-refactor this PR changes the code base without new features or bugfix label Jul 1, 2024
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@sundy-li
Copy link
Member

sundy-li commented Jul 1, 2024

can we enable bendpy member now ?

@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 1, 2024

can we enable bendpy member now ?

What's the problem of this? I don't have the context.

@Xuanwo Xuanwo enabled auto-merge July 1, 2024 14:22
@Xuanwo
Copy link
Member Author

Xuanwo commented Jul 1, 2024

delta table tests failed, I'm working on it.

Xuanwo added 2 commits July 1, 2024 23:36
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo disabled auto-merge July 1, 2024 16:18
Xuanwo and others added 4 commits July 2, 2024 00:19
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as draft July 2, 2024 04:42
Xuanwo added 2 commits July 2, 2024 13:08
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review July 2, 2024 10:28
@Xuanwo Xuanwo enabled auto-merge July 2, 2024 10:30
@Xuanwo Xuanwo added this pull request to the merge queue Jul 2, 2024
Merged via the queue into databendlabs:main with commit 28dc42e Jul 2, 2024
72 checks passed
@Xuanwo Xuanwo deleted the bump-arrow branch July 2, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-refactor this PR changes the code base without new features or bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants