Skip to content

Add Integration test for Upsert Preload#11160

Merged
KKcorps merged 3 commits intoapache:masterfrom
KKcorps:preload_integ_test
Jul 26, 2023
Merged

Add Integration test for Upsert Preload#11160
KKcorps merged 3 commits intoapache:masterfrom
KKcorps:preload_integ_test

Conversation

@KKcorps
Copy link
Contributor

@KKcorps KKcorps commented Jul 24, 2023

Integration test for #11020

@KKcorps KKcorps requested a review from Jackie-Jiang July 24, 2023 12:59
@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2023

Codecov Report

Merging #11160 (4fcf4c0) into master (5eafe76) will decrease coverage by 0.01%.
Report is 19 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11160      +/-   ##
==========================================
- Coverage    0.11%    0.11%   -0.01%     
==========================================
  Files        2208     2214       +6     
  Lines      118583   118822     +239     
  Branches    17944    17978      +34     
==========================================
  Hits          137      137              
- Misses     118426   118665     +239     
  Partials       20       20              
Flag Coverage Δ
integration1temurin11 0.00% <ø> (ø)
integration1temurin17 0.00% <ø> (ø)
integration1temurin20 0.00% <ø> (ø)
integration2temurin11 0.00% <ø> (?)
integration2temurin17 0.00% <ø> (ø)
integration2temurin20 0.00% <ø> (ø)
unittests1temurin11 0.00% <ø> (ø)
unittests1temurin17 0.00% <ø> (ø)
unittests1temurin20 0.00% <ø> (ø)
unittests2temurin11 0.11% <ø> (-0.01%) ⬇️
unittests2temurin17 0.11% <ø> (-0.01%) ⬇️
unittests2temurin20 0.11% <ø> (-0.01%) ⬇️

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

see 34 files with indirect coverage changes

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

@Jackie-Jiang
Copy link
Contributor

Please take a look at the failed test

@KKcorps KKcorps force-pushed the preload_integ_test branch from 007f617 to 6d0c5d1 Compare July 25, 2023 13:19
@KKcorps KKcorps merged commit d9c64f3 into apache:master Jul 26, 2023
s0nskar pushed a commit to s0nskar/pinot that referenced this pull request Aug 10, 2023
* Add Integration test for Upsert Preload

* Fix tests

* Reverting Base method changes as it breaks compatibility tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants