apache / orc

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

ORC-1601: Reduce get HadoopShims sync block in HadoopShimsFactory #1769

Closed cxzl25 closed 9 months ago

cxzl25 commented 9 months ago

What changes were proposed in this pull request?

Improve calling efficiency of org.apache.orc.impl.HadoopShimsFactory#get.

Why are the changes needed?

org.apache.orc.impl.HadoopShimsFactory#get is a static method with synchronized. It needs to be locked every time it is called. This is unnecessary.

How was this patch tested?

GA

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

No

dongjoon-hyun commented 9 months ago

Thank you, @cxzl25 .