Skip to content

bump datafusion from 43 to 49#13

Merged
dariocurr merged 11 commits into
datafusion-contrib:mainfrom
dariocurr:update-deps
Sep 11, 2025
Merged

bump datafusion from 43 to 49#13
dariocurr merged 11 commits into
datafusion-contrib:mainfrom
dariocurr:update-deps

Conversation

@dariocurr
Copy link
Copy Markdown
Collaborator

@dariocurr dariocurr commented Jul 22, 2025

Hi, I:

  • updated datafusion from 43 to 49
  • added the dependabot configuration
  • updated pre-commit configuration
  • simplified & updated the Cargo.toml
  • simplified the use statements

In order to support coercion in signatures, I'm waiting for Numeric to be implemented

@dariocurr dariocurr changed the title bump datafusion from 43 to 48 bump datafusion from 43 to 49 Aug 5, 2025
@alamb
Copy link
Copy Markdown
Contributor

alamb commented Aug 5, 2025

@dmitriibugakov are you still interested in maintaining this crate?

If not, are you open to others helping out?

@dariocurr
Copy link
Copy Markdown
Collaborator Author

dariocurr commented Aug 20, 2025

meanwhile, I'm going publish this on my own to use it

@alamb
Copy link
Copy Markdown
Contributor

alamb commented Aug 20, 2025

Hi @dariocurr -- I invited you as a committer to this repo. Hopefully that means you can run the tests and help review / merge other PRs

@dariocurr
Copy link
Copy Markdown
Collaborator Author

Hi @dariocurr -- I invited you as a committer to this repo. Hopefully that means you can run the tests and help review / merge other PRs

It's an honor, thank you!
Who should I list as the reviewer in this PR?

@alamb
Copy link
Copy Markdown
Contributor

alamb commented Aug 22, 2025

Hi @dariocurr -- I invited you as a committer to this repo. Hopefully that means you can run the tests and help review / merge other PRs

It's an honor, thank you! Who should I list as the reviewer in this PR?

Anyone you can get the attention of (maybe ask on DataFusion discord?)

Or if you are the only one actively maintaining it, I think you could also just merge it in as well

@dariocurr
Copy link
Copy Markdown
Collaborator Author

I'll try to get attention on discord since "At least 1 approving review is required by reviewers with write access"

@alamb
Copy link
Copy Markdown
Contributor

alamb commented Aug 27, 2025

I'll try to get attention on discord since "At least 1 approving review is required by reviewers with write access"

@dariocurr I updated the branch protection rules and you should now be able to merge this PR

@dariocurr
Copy link
Copy Markdown
Collaborator Author

I'll try to get attention on discord since "At least 1 approving review is required by reviewers with write access"

@dariocurr I updated the branch protection rules and you should now be able to merge this PR

unfortunately not
image

@alamb
Copy link
Copy Markdown
Contributor

alamb commented Sep 6, 2025

I'll try to get attention on discord since "At least 1 approving review is required by reviewers with write access"

@dariocurr I updated the branch protection rules and you should now be able to merge this PR

unfortunately not image

Sorry about that -- I just updated your permissions for this repo to admin -- you should now be able to do whatever is needed

@dariocurr dariocurr merged commit 1a22df3 into datafusion-contrib:main Sep 11, 2025
5 checks passed
@dariocurr dariocurr deleted the update-deps branch September 11, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants