search

apache / orc

Apache ORC - the smallest, fastest columnar storage for Hadoop workloads
https://orc.apache.org/
Apache License 2.0
665 stars 477 forks source link

ORC-1720: [C++] Unified compressor/decompressor exception types #1944

Closed luffy-zh closed 1 month ago

luffy-zh commented 1 month ago

What changes were proposed in this pull request?

Add new exception types to the decompressor/compressor.

Why are the changes needed?

The current implementation of the compressor/decompressor uses a variety of std::exception types, which can perplex users. We can enhance users' debugging efficiency by adopting a unified exception class.

How was this patch tested?

The tests in TestWriter.cc can cover this patch.

Was this patch authored or co-authored using generative AI tooling?

NO

cxzl25 commented 1 month ago

The description of the PR is missing an answer to a question Was this patch authored or co-authored using generative AI tooling?