Skip to content

Conversation

@navina
Copy link
Contributor

@navina navina commented Feb 10, 2022

Updated the existing stream data providers like AirlineDataStream and PullRequestMergedEventsStream to accept stream producers so that we can have more examples with non-kafka data source.

@codecov-commenter
Copy link

codecov-commenter commented Feb 10, 2022

Codecov Report

Merging #8190 (6a68580) into master (58f74ce) will increase coverage by 0.03%.
The diff coverage is 90.00%.

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8190      +/-   ##
============================================
+ Coverage     71.36%   71.39%   +0.03%     
  Complexity     4308     4308              
============================================
  Files          1623     1623              
  Lines         84356    84365       +9     
  Branches      12654    12657       +3     
============================================
+ Hits          60197    60234      +37     
+ Misses        20023    20001      -22     
+ Partials       4136     4130       -6     
Flag Coverage Δ
integration1 28.92% <0.00%> (+0.06%) ⬆️
integration2 27.54% <0.00%> (-0.15%) ⬇️
unittests1 67.89% <ø> (+<0.01%) ⬆️
unittests2 14.18% <90.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
...ntroller/helix/core/PinotHelixResourceManager.java 66.12% <90.00%> (+0.35%) ⬆️
...pinot/common/utils/fetcher/HttpSegmentFetcher.java 61.53% <0.00%> (-10.26%) ⬇️
...a/org/apache/pinot/common/utils/ServiceStatus.java 60.00% <0.00%> (-7.15%) ⬇️
...core/startree/operator/StarTreeFilterOperator.java 85.31% <0.00%> (-2.10%) ⬇️
.../pinot/server/starter/helix/BaseServerStarter.java 57.98% <0.00%> (-1.97%) ⬇️
...r/helix/SegmentOnlineOfflineStateModelFactory.java 63.20% <0.00%> (-1.89%) ⬇️
...ces/PinotSegmentUploadDownloadRestletResource.java 57.51% <0.00%> (-1.29%) ⬇️
...e/pinot/broker/broker/helix/BaseBrokerStarter.java 75.67% <0.00%> (-1.09%) ⬇️
.../pinot/core/data/manager/BaseTableDataManager.java 86.80% <0.00%> (-0.86%) ⬇️
...e/pinot/common/utils/FileUploadDownloadClient.java 63.00% <0.00%> (-0.74%) ⬇️
... and 16 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 58f74ce...ffc3140. Read the comment docs.

@navina navina force-pushed the update-realtime-stream branch from 6d28d8c to ffc3140 Compare February 11, 2022 05:18
@mayankshriv mayankshriv merged commit cf58d37 into apache:master Feb 11, 2022
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