Skip to content

Commit

Permalink
fix: carry on in the face of more arrow-exceptions
Browse files Browse the repository at this point in the history
There are more Parquet-related exceptions that prevent the file format
conversion from progressing.

Instead of continuing to add to the list of exception types, have found the
PyArrow exception base class at
https://github.com/apache/arrow/blob/fb8e8122f623f4548b22ece7485c4570d7ece1a6/python/pyarrow/error.pxi#L33C7-L33C21
  • Loading branch information
michalc committed Dec 4, 2024
1 parent c6570c3 commit 13b9e8b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app_worker.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,7 @@ def get_num_statements_with_rows(query_multi, script):
try:
aws_multipart_upload(signed_s3_request, s3_key,
stream_write_parquet(cols, rows))
except (pa.ArrowNotImplementedError, pa.ArrowTypeError):
except pa.ArrowException:
logger.exception('Unable to convert to parquet')

# ... and as ODS with the results of each statement as a separate sheet
Expand Down

0 comments on commit 13b9e8b

Please sign in to comment.