[KYUUBI #6445] Normalize extra name for optional Python distribution dependencies

[camper42]

…pendencies

🔍 Description

Issue References 🔗

This pull request fixes #6445

Describe Your Solution 🔧

use hive-pure-sasl instead of hive_pure_sasl for extra name for optional distribution dependencies.

this avoid potential WARNING: pyhive x.y.z does not provide the extra 'hive-pure-sasl' and missing dependencies when a package depends on pyhive[hive_pure_sasl] and python build system choose to normalize it to pyhive[hive-pure-saal].

Types of changes 🔖

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

a package depends on pyhive[hive_pure_sasl] may complain WARNING: pyhive x.y.z does not provide the extra 'hive-pure-sasl' and missing dependencies to support hive feature.

Behavior With This Pull Request 🎉

based on my test on our internal pypi, users use pyhive[hive_pure_sasl] are not affected by this change. But we should update README when we release a new version of pyhive.

Related Unit Tests

---

Checklist 📝

• This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.44%. Comparing base (ef203e0) to head (d82e5ca).
Report is 1 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #6454      +/-   ##
============================================
- Coverage     58.46%   58.44%   -0.02%     
  Complexity       24       24              
============================================
  Files           656      656              
  Lines         40267    40267              
  Branches       5498     5498              
============================================
- Hits          23544    23536       -8     
- Misses        14219    14227       +8     
  Partials       2504     2504              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[camper42]

A quick look at #6341 shows that it changes hive_pure_sasl to hive-pure-sasl as well, but there's no record of why this was done. I think it's good to keep a seperate PR

[pan3793]

Thanks, merged to master