Skip to content

fix: modify the incorrect Hive configuration in hoodie hive catalog#18365

Merged
wombatu-kun merged 1 commit into
apache:masterfrom
yangxiao0320:fix_hive_configuration
Mar 23, 2026
Merged

fix: modify the incorrect Hive configuration in hoodie hive catalog#18365
wombatu-kun merged 1 commit into
apache:masterfrom
yangxiao0320:fix_hive_configuration

Conversation

@yangxiao0320
Copy link
Copy Markdown
Contributor

@yangxiao0320 yangxiao0320 commented Mar 21, 2026

Describe the issue this Pull Request addresses

#18011 The incorrect Hive configuration is introduced. hive.metastore.kerberos.principle is incorrect, and hive.metastore.kerberos.principal is correct.

Summary and Changelog

To avoid introducing incorrect variables, variable names are directly referenced from HiveConf.

Impact

The Hive synchronization fails in the Kerberos environment due to incorrect Hive configuration name.

Risk Level

none

Documentation Update

none

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions Bot added the size:S PR with lines of changes in (10, 100] label Mar 21, 2026
@hudi-bot
Copy link
Copy Markdown
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.46%. Comparing base (3aef2ca) to head (41e676f).

Files with missing lines Patch % Lines
...g/apache/hudi/table/catalog/HoodieHiveCatalog.java 33.33% 1 Missing and 3 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18365      +/-   ##
============================================
- Coverage     68.46%   68.46%   -0.01%     
  Complexity    27472    27472              
============================================
  Files          2427     2427              
  Lines        132655   132655              
  Branches      15994    15994              
============================================
- Hits          90819    90817       -2     
- Misses        34786    34788       +2     
  Partials       7050     7050              
Flag Coverage Δ
common-and-other-modules 44.43% <33.33%> (-0.01%) ⬇️
hadoop-mr-java-client 45.06% <ø> (+0.06%) ⬆️
spark-client-hadoop-common 48.21% <ø> (-0.01%) ⬇️
spark-java-tests 48.82% <ø> (ø)
spark-scala-tests 44.94% <ø> (-0.01%) ⬇️
utilities 38.64% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...g/apache/hudi/table/catalog/HoodieHiveCatalog.java 48.34% <33.33%> (+0.22%) ⬆️

... and 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wombatu-kun wombatu-kun merged commit 941ae62 into apache:master Mar 23, 2026
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants