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.

Sign up for GitHub

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

Jump to bottom

[Python] ParquetWriter("path", None) segfaults #35858

Closed
jonashaag opened this issue May 31, 2023 · 0 comments · Fixed by #36011 or #36046
Closed

[Python] ParquetWriter("path", None) segfaults #35858

jonashaag opened this issue May 31, 2023 · 0 comments · Fixed by #36011 or #36046
Assignees
@westonpace
Labels
Component: Python Type: bug
Milestone
13.0.0

Comments

@jonashaag
Copy link
Contributor

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

This segfaults:

python -c 'import pyarrow.parquet as pa; pa.ParquetWriter("x", None)'

Component(s)

Python
@github-actions github-actions bot mentioned this issue Jun 9, 2023
Merged
@jorisvandenbossche jorisvandenbossche changed the title ParquetWriter("path", None) segfaults [Python] ParquetWriter("path", None) segfaults Jun 13, 2023
@jorisvandenbossche jorisvandenbossche added this to the 13.0.0 milestone Jun 13, 2023
jorisvandenbossche pushed a commit that referenced this issue Jun 13, 2023
@westonpace
GH-35858: [Python] disallow none schema parquet writer (#36011)
1ddeaab 
### Rationale for this change

Previously, passing in None for the schema would cause a segmentation fault.

### What changes are included in this PR?

Now a TypeError is raised instead

### Are these changes tested?

Yes, a new unit test is created

### Are there any user-facing changes?

No
* Closes: #35858

Authored-by: Weston Pace <weston.pace@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
jorisvandenbossche added a commit to jorisvandenbossche/arrow that referenced this issue Jun 13, 2023
@jorisvandenbossche
apacheGH-35858: fixup linting from PR apacheGH-36011
6de5539
@github-actions github-actions bot mentioned this issue Jun 13, 2023
Merged
kou pushed a commit that referenced this issue Jun 13, 2023
@jorisvandenbossche
GH-35858: [Python] Fixup linting from PR GH-36011 (#36046)
8b84cb0 
Fixing up linting errors from PR #36011 that I merged.
* Closes: #35858

Authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@westonpace westonpace

Labels
Component: Python Type: bug
Projects
None yet
Milestone
13.0.0
3 participants
@jonashaag @jorisvandenbossche @westonpace