Skip to content

Conversation

@xiangfu0
Copy link
Contributor

@xiangfu0 xiangfu0 commented Nov 8, 2023

fixing the regression introduced from: #11839

Copy link
Contributor

@snleee snleee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow, thanks for the quick fix! LGTM

@xiangfu0 xiangfu0 force-pushed the fixing-dateTimeConvert branch from 34cd639 to 5c57c33 Compare November 8, 2023 06:14
@xiangfu0 xiangfu0 requested a review from Jackie-Jiang November 8, 2023 06:45
@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2023

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.41%. Comparing base (b5e9823) to head (5ea37e8).
⚠️ Report is 2944 commits behind head on master.

Files with missing lines Patch % Lines
...a/org/apache/pinot/common/utils/PinotDataType.java 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11971      +/-   ##
============================================
- Coverage     61.43%   61.41%   -0.02%     
  Complexity      207      207              
============================================
  Files          2385     2385              
  Lines        129149   129150       +1     
  Branches      19994    19994              
============================================
- Hits          79342    79318      -24     
- Misses        44055    44075      +20     
- Partials       5752     5757       +5     
Flag Coverage Δ
custom-integration1 <0.01% <0.00%> (ø)
integration <0.01% <0.00%> (ø)
integration1 <0.01% <0.00%> (ø)
integration2 0.00% <0.00%> (ø)
java-11 61.37% <20.00%> (-0.02%) ⬇️
java-21 61.28% <20.00%> (+0.01%) ⬆️
skip-bytebuffers-false 61.41% <20.00%> (+0.01%) ⬆️
skip-bytebuffers-true 27.59% <0.00%> (-33.68%) ⬇️
temurin 61.41% <20.00%> (-0.02%) ⬇️
unittests 61.41% <20.00%> (-0.02%) ⬇️
unittests1 46.64% <20.00%> (-0.05%) ⬇️
unittests2 27.61% <0.00%> (+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 force-pushed the fixing-dateTimeConvert branch from 5c57c33 to 5ea37e8 Compare November 8, 2023 07:18
@xiangfu0 xiangfu0 merged commit 972b555 into apache:master Nov 8, 2023
@xiangfu0 xiangfu0 deleted the fixing-dateTimeConvert branch November 8, 2023 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants