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-1687: [C++] Enforce naming style. #1888

Closed ffacs closed 2 months ago

ffacs commented 2 months ago

What changes were proposed in this pull request?

  1. Add make check-format, make check-clang-tidy,make format and make fix-clang-tidy
  2. Add a github action that runs check-clang-tidy
  3. Fix naming style errors in c++/src/OrcHdfsFile.cc

Why are the changes needed?

To enforce identifiers' naming style on c++ side.

How was this patch tested?

UT passed

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

NO

dongjoon-hyun commented 2 months ago

Thanks!