Skip to content

Fix class not found exception in GenericQuickstart#13269

Merged
xiangfu0 merged 1 commit into
apache:masterfrom
gortiz:fix-hadoop-on-quickstart
May 30, 2024
Merged

Fix class not found exception in GenericQuickstart#13269
xiangfu0 merged 1 commit into
apache:masterfrom
gortiz:fix-hadoop-on-quickstart

Conversation

@gortiz
Copy link
Copy Markdown
Contributor

@gortiz gortiz commented May 30, 2024

Some recent change has changed the dependency to hadoop-mapreduce-client-core, making it end up as provided when in fact it is not. Unless it is clear who is the one providing the dependency, it should have compile scope

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 30, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.16%. Comparing base (59551e4) to head (d0ca861).
⚠️ Report is 3834 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13269      +/-   ##
============================================
+ Coverage     61.75%   62.16%   +0.41%     
+ Complexity      207      198       -9     
============================================
  Files          2436     2536     +100     
  Lines        133233   139401    +6168     
  Branches      20636    21548     +912     
============================================
+ Hits          82274    86664    +4390     
- Misses        44911    46247    +1336     
- Partials       6048     6490     +442     
Flag Coverage Δ
custom-integration1 <0.01% <ø> (-0.01%) ⬇️
integration <0.01% <ø> (-0.01%) ⬇️
integration1 <0.01% <ø> (-0.01%) ⬇️
integration2 0.00% <ø> (ø)
java-11 62.11% <ø> (+0.40%) ⬆️
java-21 62.05% <ø> (+0.43%) ⬆️
skip-bytebuffers-false 62.15% <ø> (+0.40%) ⬆️
skip-bytebuffers-true 62.01% <ø> (+34.28%) ⬆️
temurin 62.16% <ø> (+0.41%) ⬆️
unittests 62.16% <ø> (+0.41%) ⬆️
unittests1 46.75% <ø> (-0.15%) ⬇️
unittests2 27.74% <ø> (+<0.01%) ⬆️

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

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xiangfu0 xiangfu0 merged commit d4e10e5 into apache:master May 30, 2024
gortiz added a commit to gortiz/pinot that referenced this pull request Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants