[ARROW-7332] [C++][Parquet] Explicitly catch status exceptions in PARQUET_CATCH_NOT_OK

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.15.1
• Fix Version/s: 0.17.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/23614
• Assignee: Antoine Pitrou
• Reporter: Ben Kietzman
• Created: 05/Dec/19 19:54
• Updated: 11/Jan/23 07:53
• Resolved: 17/Mar/20 19:41

Description

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:

  } catch (const ::parquet::ParquetStatusException& e) { \
    return e.status(); \
  } catch (const ::parquet::ParquetException& e) { \
    return Status::IOError(e.what()) \

This will retain the original StatusCode rather than overwriting it with IOError.

Issue Links

• links to: GitHub Pull Request #6642