[
https://issues.apache.org/jira/browse/ARROW-7332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Wes McKinney resolved ARROW-7332.
---------------------------------
Resolution: Fixed
Issue resolved by pull request 6642
[https://github.com/apache/arrow/pull/6642]
> [C++][Parquet] Explicitly catch status exceptions in PARQUET_CATCH_NOT_OK
> -------------------------------------------------------------------------
>
> Key: ARROW-7332
> URL: https://issues.apache.org/jira/browse/ARROW-7332
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Affects Versions: 0.15.1
> Reporter: Ben Kietzman
> Assignee: Antoine Pitrou
> Priority: Minor
> Labels: pull-request-available
> Fix For: 0.17.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> PARQUET_THROW_NOT_OK throws a ParquetStatusException, which contains a full
> Status rather than just an error string. These could be caught explicitly in
> PARQUET_CATCH_NOT_OK and the original status returned rather than creating a
> new status:
> {code}
> } catch (const ::parquet::ParquetStatusException& e) { \
> return e.status(); \
> } catch (const ::parquet::ParquetException& e) { \
> return Status::IOError(e.what()) \
> {code}
> This will retain the original StatusCode rather than overwriting it with
> IOError.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
