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

[Python] Remove dead code in _reconstruct_block (pandas_compat.py) #38712

Closed
AlenkaF opened this issue Nov 14, 2023 · 0 comments · Fixed by #38714
Closed

[Python] Remove dead code in _reconstruct_block (pandas_compat.py) #38712

AlenkaF opened this issue Nov 14, 2023 · 0 comments · Fixed by #38714
Assignees
Milestone

Comments

@AlenkaF
Copy link
Member

AlenkaF commented Nov 14, 2023

Describe the bug, including details regarding any error messages, version, and platform.

It seems the object case in _reconstruct_block is a dead code and is not needed anymore so therefore could be removed.

Component(s)

Python

@AlenkaF AlenkaF self-assigned this Nov 14, 2023
@jorisvandenbossche jorisvandenbossche added this to the 15.0.0 milestone Nov 15, 2023
jorisvandenbossche pushed a commit that referenced this issue Nov 15, 2023
### Rationale for this change

It seems the object case in `_reconstruct_block` is a dead code and is not needed anymore so therefore could be removed.

### What changes are included in this PR?

Removal of the object case in `_reconstruct_block` code. Was also looking at the `arrow_to_pandas.cc` code to see if there is any dead code present and I couldn't find any. 

### Are these changes tested?

The change in this PR should not make any of the existing tests fail.

### Are there any user-facing changes?

There shouldn't be.
* Closes: #38712

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…he#38714)

### Rationale for this change

It seems the object case in `_reconstruct_block` is a dead code and is not needed anymore so therefore could be removed.

### What changes are included in this PR?

Removal of the object case in `_reconstruct_block` code. Was also looking at the `arrow_to_pandas.cc` code to see if there is any dead code present and I couldn't find any. 

### Are these changes tested?

The change in this PR should not make any of the existing tests fail.

### Are there any user-facing changes?

There shouldn't be.
* Closes: apache#38712

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 a pull request may close this issue.

2 participants