GH-49227: [Python] Deprecate pyarrow.gandiva#49637
GH-49227: [Python] Deprecate pyarrow.gandiva#49637AlenkaF wants to merge 4 commits intoapache:mainfrom
pyarrow.gandiva#49637Conversation
raulcd
left a comment
There was a problem hiding this comment.
@pitrou what are your thoughts here? Should we merge this as part of 24.0.0 given no response to the ML or should we push it to 25.0.0 and "give time" to users? Maybe we should bump the ML thread and potentially share in user@arrow.apache.org before merging?
|
Well, there really is no need to rush, so I would agree with waiting till 25.0.0 and adding more attention to the ML thread before merging. (Edit: sorry, I am sure Antoine has not been tagged when I was first reading this =) ) |
|
This is a deprecation, so we want this to be merged for 24.0.0. Waiting for another version wouldn't do any good, as people would not be informed before the deprecation appears. |
Well, it depends. I was asking because I was thinking whether we want to bump the ML thread to give more visibility for any concern from users before deprecating. If we have decided, thanks to lazy consensus, that we are deprecating, then, yes I do agree we should merge it for 24.0.0. |
raulcd
left a comment
There was a problem hiding this comment.
Based on the discussion on the issue and the ML. I agree we seem to be in agreement and no-one answered the ML. So by the power of lazy consensus I approve :)
|
We can always undo the deprecation later, so there is really no downside to doing this in 24.0.0 IMHO. |
Rationale for this change
See: #49227 (comment)
What changes are included in this PR?
Deprecation warnings for
pyarrow.gandivaadded for version 24.0.0Are these changes tested?
Yes, tested locally that warnings are raised on import. CI tests verify the warnings are filtered in out test suite.
Are there any user-facing changes?
Yes,
pyarrow.gandivais deprecated and will be removed in a feature PyArrow version.pyarrow.gandiva#49227