Skip to content

Conversation

@bagipriyank
Copy link
Contributor

Fixes #9834

@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2022

Codecov Report

Merging #9857 (8421200) into master (2140954) will decrease coverage by 42.07%.
The diff coverage is 14.28%.

@@              Coverage Diff              @@
##             master    #9857       +/-   ##
=============================================
- Coverage     70.34%   28.27%   -42.08%     
+ Complexity     5014       53     -4961     
=============================================
  Files          1972     1960       -12     
  Lines        105692   105418      -274     
  Branches      15993    15993               
=============================================
- Hits          74350    29806    -44544     
- Misses        26137    72691    +46554     
+ Partials       5205     2921     -2284     
Flag Coverage Δ
integration1 25.21% <14.28%> (+<0.01%) ⬆️
integration2 24.76% <0.00%> (+0.12%) ⬆️
unittests1 ?
unittests2 ?

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

Impacted Files Coverage Δ
...e/pinot/common/utils/FileUploadDownloadClient.java 49.56% <14.28%> (-10.71%) ⬇️
...in/java/org/apache/pinot/spi/utils/BytesUtils.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/spi/trace/BaseRecording.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/spi/trace/NoOpRecording.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/config/table/FSTType.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/config/user/RoleType.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/data/MetricFieldSpec.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/stream/StreamMessage.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/common/tier/TierFactory.java 0.00% <0.00%> (-100.00%) ⬇️
...a/org/apache/pinot/spi/config/table/TableType.java 0.00% <0.00%> (-100.00%) ⬇️
... and 1352 more

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

Copy link
Contributor

@Jackie-Jiang Jackie-Jiang left a comment

Choose a reason for hiding this comment

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

LGTM otherwise. I'd be good if we can add a test for them

@Jackie-Jiang Jackie-Jiang added the release-notes Referenced by PRs that need attention when compiling the next release notes label Nov 28, 2022
@Jackie-Jiang
Copy link
Contributor

@Jackie-Jiang Jackie-Jiang merged commit 2dae951 into apache:master Dec 1, 2022
@bagipriyank bagipriyank deleted the issue-9834 branch January 2, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes Referenced by PRs that need attention when compiling the next release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Delete Schema/Table for pinot-admin command

3 participants