Skip to content

Remove incubator from pinot and superset #8223

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
xiangfu0 merged 1 commit into from
Feb 18, 2022
Merged

Remove incubator from pinot and superset #8223

xiangfu0 merged 1 commit into from
Feb 18, 2022

Conversation

@Jackie-Jiang
Copy link
Contributor

@Jackie-Jiang Jackie-Jiang commented Feb 18, 2022

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2022 •
edited
Loading

Codecov Report

Merging #8223 (f90ff68) into master (582d621) will decrease coverage by 35.48%.
The diff coverage is n/a.

❗ Current head f90ff68 differs from pull request most recent head cf8dddc. Consider uploading reports for the commit cf8dddc to get more accurate results

Impacted file tree graph

@@              Coverage Diff              @@
##             master    #8223       +/-   ##
=============================================
- Coverage     64.28%   28.79%   -35.49%     
=============================================
  Files          1581     1614       +33     
  Lines         83122    84650     +1528     
  Branches      12587    12753      +166     
=============================================
- Hits          53436    24377    -29059     
- Misses        25852    58053    +32201     
+ Partials       3834     2220     -1614
Flag Coverage Δ
integration1 28.79% <ø> (?)
unittests1 ?
unittests2 ?

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

Impacted Files Coverage Δ
.../java/org/apache/pinot/spi/utils/BooleanUtils.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/data/MetricFieldSpec.java 0.00% <0.00%> (-100.00%) ⬇️
...va/org/apache/pinot/spi/utils/BigDecimalUtils.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%) ⬇️
.../org/apache/pinot/spi/data/DimensionFieldSpec.java 0.00% <0.00%> (-100.00%) ⬇️
.../org/apache/pinot/spi/data/readers/FileFormat.java 0.00% <0.00%> (-100.00%) ⬇️
...org/apache/pinot/spi/config/table/QuotaConfig.java 0.00% <0.00%> (-100.00%) ⬇️
...org/apache/pinot/spi/config/tenant/TenantRole.java 0.00% <0.00%> (-100.00%) ⬇️
... and 1323 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 582d621...cf8dddc. Read the comment docs.

@xiangfu0 xiangfu0 merged commit de1cdf2 into apache:master Feb 18, 2022
@Jackie-Jiang Jackie-Jiang deleted the remove_incubator branch February 18, 2022 19:13
xiangfu0 pushed a commit to xiangfu0/pinot that referenced this pull request Feb 23, 2022
@Jackie-Jiang @xiangfu0
Remove incubator from pinot and superset (apache#8223)
05f009a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xiangfu0 xiangfu0 xiangfu0 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Jackie-Jiang @codecov-commenter @xiangfu0