Skip to content

[python] Remove FutureWarning#3559

Closed
changhiskhan wants to merge 1 commit intoapache:masterfrom
changhiskhan:future-warning
Closed

[python] Remove FutureWarning#3559
changhiskhan wants to merge 1 commit intoapache:masterfrom
changhiskhan:future-warning

Conversation

@changhiskhan
Copy link
Copy Markdown

Currently RecordBatch::to_pyarrow passes the Schema in the second arg position which causes a FutureWarning. Instead we change it to use the schema kwarg.

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Currently RecordBatch::to_pyarrow passes the Schema
in the second arg position which causes a FutureWarning.
Instead we change it to use the `schema` kwarg.
@github-actions github-actions bot added the arrow Changes to the arrow crate label Jan 18, 2023
Copy link
Copy Markdown
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

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

I think it is already fixed by #3544.

@viirya
Copy link
Copy Markdown
Member

viirya commented Jan 18, 2023

Thank you @changhiskhan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow Changes to the arrow crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants