search

apache / orc

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

ORC-1623: Use `directOut.put(out)` instead of `directOut.put(out.array())` in `TestZstd` test #1806

Closed cxzl25 closed 6 months ago

cxzl25 commented 6 months ago

What changes were proposed in this pull request?

This PR aims to use directOut.put(out) instead of directOut.put(out.array()) in TestZstd test.

Why are the changes needed?

Improve the readability of test code.

How was this patch tested?

GA

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

No

deshanxiao commented 6 months ago

Thanks @cxzl25 . Pending CI.

dongjoon-hyun commented 6 months ago

To @deshanxiao , please try to merge this to main and branch-2.0 by using script dev/merge_orc_pr.py. 😄

deshanxiao commented 6 months ago

Thanks @cxzl25 @dongjoon-hyun! Merged to main/branch-2.0.