search

apache / orc

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

ORC-1602: [C++] limit compression block size #1774

Closed ffacs closed 7 months ago

ffacs commented 7 months ago

What changes were proposed in this pull request?

limit compression block size on c++ side.

Why are the changes needed?

to fix https://github.com/apache/orc/issues/1727

How was this patch tested?

UT passed

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

NO

cxzl25 commented 7 months ago

Why are the changes needed?[

Remove [

deshanxiao commented 7 months ago

LGTM, thanks @ffacs !

dongjoon-hyun commented 7 months ago

Merged to main/2.0/1.9.

To @ffacs , could you make a backporting PR to branch-1.8 and branch-1.7 ?