apache / hudi-rs

A native Rust library for Apache Hudi, with bindings into Python
https://hudi.apache.org/
Apache License 2.0
142 stars 28 forks source link

feat: support loading hudi global configs #118

Closed zzhpro closed 1 month ago

zzhpro commented 2 months ago

Description

Load hudi-defaults.conf from HUDI_CONF_DIR when creating table instance , for #111 .

How are the changes test-covered

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 93.02326% with 3 lines in your changes missing coverage. Please review.

Project coverage is 87.77%. Comparing base (8209599) to head (42297eb). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/core/src/table/mod.rs 91.66% 2 Missing :warning:
crates/core/src/storage/utils.rs 92.85% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #118 +/- ## ========================================== + Coverage 87.60% 87.77% +0.16% ========================================== Files 14 14 Lines 702 728 +26 ========================================== + Hits 615 639 +24 - Misses 87 89 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

xushiyan commented 1 month ago

@zzhpro thanks. i've made some refactoring based on your version.