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] Create a base class for Table and RecordBatch #34979

Closed
danepitkin opened this issue Apr 7, 2023 · 1 comment · Fixed by #34980
Closed

[Python] Create a base class for Table and RecordBatch #34979

danepitkin opened this issue Apr 7, 2023 · 1 comment · Fixed by #34980

Comments

@danepitkin
Copy link
Contributor

Describe the enhancement requested

Create a base class to share dataframe-like method implementations.

Component(s)

Python

@danepitkin
Copy link
Contributor Author

This is part of 30559

@jorisvandenbossche jorisvandenbossche added this to the 13.0.0 milestone Apr 27, 2023
jorisvandenbossche pushed a commit that referenced this issue Apr 27, 2023
)

### Rationale for this change

This is an incremental first step towards #30559

### What changes are included in this PR?

Introduce `class _Table` in `table.pxi`.

### Are these changes tested?

Existing pytests will check for regression.

### Are there any user-facing changes?

No
* Closes: #34979

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
liujiacheng777 pushed a commit to LoongArch-Python/arrow that referenced this issue May 11, 2023
apache#34980)

### Rationale for this change

This is an incremental first step towards apache#30559

### What changes are included in this PR?

Introduce `class _Table` in `table.pxi`.

### Are these changes tested?

Existing pytests will check for regression.

### Are there any user-facing changes?

No
* Closes: apache#34979

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
apache#34980)

### Rationale for this change

This is an incremental first step towards apache#30559

### What changes are included in this PR?

Introduce `class _Table` in `table.pxi`.

### Are these changes tested?

Existing pytests will check for regression.

### Are there any user-facing changes?

No
* Closes: apache#34979

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
apache#34980)

### Rationale for this change

This is an incremental first step towards apache#30559

### What changes are included in this PR?

Introduce `class _Table` in `table.pxi`.

### Are these changes tested?

Existing pytests will check for regression.

### Are there any user-facing changes?

No
* Closes: apache#34979

Authored-by: Dane Pitkin <dane@voltrondata.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