Skip to content

Do not shade jar for pinot native plug-in components#7245

Merged
snleee merged 1 commit intoapache:masterfrom
snleee:reduce-binary-tar
Aug 3, 2021
Merged

Do not shade jar for pinot native plug-in components#7245
snleee merged 1 commit intoapache:masterfrom
snleee:reduce-binary-tar

Conversation

@snleee
Copy link
Contributor

@snleee snleee commented Aug 3, 2021

We create the shaded jars for plug-in modules by default; however,
this results in a side effect of large binary release data size.
This pr don't create shaded jars for pinot native plug-in
components. This reduces the binary release from 848MB to 566MB.

We create the shaded jars for plug-in modules by default; however,
this results in a side effect of large binary release data size.
This pr don't create shaded jars for pinot native plug-in
components. This reduces the binary release from `848MB` to `566MB`.
@codecov-commenter
Copy link

codecov-commenter commented Aug 3, 2021

Codecov Report

Merging #7245 (d38390e) into master (29a6ba9) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #7245      +/-   ##
============================================
- Coverage     73.63%   73.53%   -0.10%     
  Complexity       92       92              
============================================
  Files          1508     1508              
  Lines         73970    73970              
  Branches      10686    10686              
============================================
- Hits          54465    54396      -69     
- Misses        15936    16012      +76     
+ Partials       3569     3562       -7     
Flag Coverage Δ
integration 41.83% <ø> (-0.03%) ⬇️
unittests 65.33% <ø> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
...data/manager/realtime/SegmentCommitterFactory.java 64.70% <0.00%> (-35.30%) ⬇️
...ller/helix/core/minion/TaskTypeMetricsUpdater.java 80.00% <0.00%> (-20.00%) ⬇️
...e/impl/dictionary/LongOnHeapMutableDictionary.java 63.85% <0.00%> (-12.05%) ⬇️
...er/api/resources/LLCSegmentCompletionHandlers.java 52.40% <0.00%> (-8.66%) ⬇️
...altime/ServerSegmentCompletionProtocolHandler.java 49.52% <0.00%> (-8.58%) ⬇️
...e/data/manager/realtime/SplitSegmentCommitter.java 53.84% <0.00%> (-7.70%) ⬇️
...impl/dictionary/DoubleOnHeapMutableDictionary.java 46.98% <0.00%> (-7.23%) ⬇️
...r/helix/SegmentOnlineOfflineStateModelFactory.java 58.87% <0.00%> (-6.55%) ⬇️
...nction/DistinctCountBitmapAggregationFunction.java 47.42% <0.00%> (-6.19%) ⬇️
...mpl/dictionary/DoubleOffHeapMutableDictionary.java 57.44% <0.00%> (-5.32%) ⬇️
... and 21 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29a6ba9...d38390e. Read the comment docs.

@xiangfu0 xiangfu0 requested review from npawar and xiangfu0 August 3, 2021 17:24
Copy link
Contributor

@xiangfu0 xiangfu0 left a comment

Choose a reason for hiding this comment

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

LGTM!

@snleee snleee merged commit c1ac8a1 into apache:master Aug 3, 2021
@snleee snleee deleted the reduce-binary-tar branch August 3, 2021 18:39
snleee pushed a commit that referenced this pull request Aug 3, 2021
We create the shaded jars for plug-in modules by default; however,
this results in a side effect of large binary release data size.
This pr don't create shaded jars for pinot native plug-in
components. This reduces the binary release from `848MB` to `566MB`.
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