Skip to content

GH-39732: [Python][CI] Fix test failures with latest/nightly pandas#39760

Merged
jorisvandenbossche merged 2 commits intoapache:mainfrom
AlenkaF:gh-39732-pandas-table-shuffle
Jan 25, 2024
Merged

GH-39732: [Python][CI] Fix test failures with latest/nightly pandas#39760
jorisvandenbossche merged 2 commits intoapache:mainfrom
AlenkaF:gh-39732-pandas-table-shuffle

Conversation

@AlenkaF
Copy link
Copy Markdown
Member

@AlenkaF AlenkaF commented Jan 23, 2024

This PR rearranges if-else blocks in the table function (table.pxi) so that pandas dataframe object comes before checking for __arrow_c_stream__ and __arrow_c_array__.

@jorisvandenbossche
Copy link
Copy Markdown
Member

Alternatively, we might also want to do a similar check of the names and metadata keywords in case of objects that implement the protocol. Because also in that case we are currently ignoring those keywords.

@jorisvandenbossche
Copy link
Copy Markdown
Member

(this is somewhat similar as the discussion in #39394 (comment) for pa.array(..), while here it is for pa.table(..))

@AlenkaF
Copy link
Copy Markdown
Member Author

AlenkaF commented Jan 23, 2024

Did I understand correctly? -> 20be425

@AlenkaF
Copy link
Copy Markdown
Member Author

AlenkaF commented Jan 23, 2024

@github-actions crossbow submit test-conda-python-3.10-pandas-*

@github-actions
Copy link
Copy Markdown

Revision: 20be425

Submitted crossbow builds: ursacomputing/crossbow @ actions-dcc5b97ac0

Task Status
test-conda-python-3.10-pandas-latest GitHub Actions
test-conda-python-3.10-pandas-nightly GitHub Actions

@AlenkaF AlenkaF marked this pull request as ready for review January 23, 2024 15:00
Copy link
Copy Markdown
Member

@jorisvandenbossche jorisvandenbossche left a comment

Choose a reason for hiding this comment

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

Thanks!

@jorisvandenbossche jorisvandenbossche merged commit c67d026 into apache:main Jan 25, 2024
@jorisvandenbossche jorisvandenbossche removed the awaiting review Awaiting review label Jan 25, 2024
@github-actions github-actions bot added the awaiting merge Awaiting merge label Jan 25, 2024
@AlenkaF AlenkaF deleted the gh-39732-pandas-table-shuffle branch January 25, 2024 11:16
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit c67d026.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
…ndas (apache#39760)

This PR rearranges if-else blocks in the `table` function (`table.pxi`) so that pandas dataframe object comes before checking for `__arrow_c_stream__` and `__arrow_c_array__`.
* Closes: apache#39732

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
raulcd pushed a commit that referenced this pull request Feb 20, 2024
…39760)

This PR rearranges if-else blocks in the `table` function (`table.pxi`) so that pandas dataframe object comes before checking for `__arrow_c_stream__` and `__arrow_c_array__`.
* Closes: #39732

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Python][CI] Failures on latest/nightly pandas

2 participants