Skip to content

Comments

Make BaseDefaultColumnHandler constructor safe for SegmentMetadata initialised with IS#10016

Closed
saurabhd336 wants to merge 1 commit intoapache:masterfrom
saurabhd336:defaultColHandlerProps
Closed

Make BaseDefaultColumnHandler constructor safe for SegmentMetadata initialised with IS#10016
saurabhd336 wants to merge 1 commit intoapache:masterfrom
saurabhd336:defaultColHandlerProps

Conversation

@saurabhd336
Copy link
Contributor

For needPrerocess calls, we don't need metadata files. This changes makes it possible to initialise BaseDefaultColumnHandler with Inputstream based SegmentMetadata implementations.

@codecov-commenter
Copy link

codecov-commenter commented Dec 21, 2022

Codecov Report

Merging #10016 (e59b2e6) into master (f2972c9) will increase coverage by 42.28%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##             master   #10016       +/-   ##
=============================================
+ Coverage     28.05%   70.34%   +42.28%     
- Complexity       53     5682     +5629     
=============================================
  Files          1979     1991       +12     
  Lines        106930   107284      +354     
  Branches      16277    16316       +39     
=============================================
+ Hits          30001    75467    +45466     
+ Misses        73993    26544    -47449     
- Partials       2936     5273     +2337     
Flag Coverage Δ
integration1 25.03% <0.00%> (-0.21%) ⬇️
integration2 24.33% <0.00%> (-0.13%) ⬇️
unittests1 67.88% <50.00%> (?)
unittests2 13.61% <0.00%> (?)

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

Impacted Files Coverage Δ
...loader/defaultcolumn/BaseDefaultColumnHandler.java 58.35% <50.00%> (+58.35%) ⬆️
...data/manager/realtime/DefaultSegmentCommitter.java 0.00% <0.00%> (-80.00%) ⬇️
...er/api/resources/LLCSegmentCompletionHandlers.java 43.56% <0.00%> (-18.82%) ⬇️
...nction/DistinctCountBitmapAggregationFunction.java 41.96% <0.00%> (-16.07%) ⬇️
...data/manager/realtime/SegmentCommitterFactory.java 88.23% <0.00%> (-11.77%) ⬇️
...altime/ServerSegmentCompletionProtocolHandler.java 51.88% <0.00%> (-6.61%) ⬇️
...ion/function/DistinctCountAggregationFunction.java 50.22% <0.00%> (-1.35%) ⬇️
...pache/pinot/core/query/utils/idset/EmptyIdSet.java 25.00% <0.00%> (ø)
...ot/common/function/scalar/ComparisonFunctions.java 42.85% <0.00%> (ø)
...anager/realtime/SegmentBuildTimeLeaseExtender.java 63.23% <0.00%> (ø)
... and 1364 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

2 participants