Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix MissingFormatArgumentException due to log#17801

Merged
swaminathanmanish merged 1 commit intoapache:masterfrom
noob-se7en:fix_empty_index_bug
Mar 3, 2026
Merged

Fix MissingFormatArgumentException due to log#17801
swaminathanmanish merged 1 commit intoapache:masterfrom
noob-se7en:fix_empty_index_bug

Conversation

@noob-se7en
Copy link
Contributor

@noob-se7en noob-se7en commented Mar 3, 2026 

ERROR [ColumnMinMaxValueGenerator] [HelixTaskExecutor-message_handle_thread_10] Caught exception while generating min/max value for column: id in segment: <Redacted>, continuing without persisting them
java.util.MissingFormatArgumentException: Format specifier '%s'
	at java.base/java.util.Formatter.format(Formatter.java:2790)
	at java.base/java.util.Formatter.format(Formatter.java:2728)
	at java.base/java.lang.String.format(String.java:4379)
	at org.apache.pinot.segment.spi.memory.EmptyIndexBuffer.getInt(EmptyIndexBuffer.java:105)
	at org.apache.pinot.segment.spi.memory.PinotDataBuffer.getInt(PinotDataBuffer.java:441)

@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 63.23%. Comparing base (9648c99) to head (62a4671).

Files with missing lines Patch % Lines
...che/pinot/segment/spi/memory/EmptyIndexBuffer.java 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##             master   #17801      +/-   ##
============================================
+ Coverage     63.22%   63.23%   +0.01%     
  Complexity     1454     1454              
============================================
  Files          3185     3185              
  Lines        191548   191526      -22     
  Branches      29301    29301              
============================================
+ Hits         121107   121118      +11     
+ Misses        60972    60931      -41     
- Partials       9469     9477       +8
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.21% <0.00%> (+0.02%) ⬆️
java-21 63.19% <0.00%> (-0.01%) ⬇️
temurin 63.23% <0.00%> (+0.01%) ⬆️
unittests 63.23% <0.00%> (+0.01%) ⬆️
unittests1 55.59% <0.00%> (+0.01%) ⬆️
unittests2 34.14% <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.

swaminathanmanish
swaminathanmanish approved these changes Mar 3, 2026
View reviewed changes
Copy link
Contributor

@swaminathanmanish swaminathanmanish left a comment

Choose a reason for hiding this comment

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

LGTM

@swaminathanmanish swaminathanmanish merged commit 0038c06 into apache:master Mar 3, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swaminathanmanish swaminathanmanish swaminathanmanish approved these changes

Assignees

No one assigned

Labels

bugfix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@noob-se7en @codecov-commenter @swaminathanmanish @Jackie-Jiang