Skip to content
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.

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

Disallow peer download when replication is < 2 #11469

Merged
merged 3 commits into from
Sep 12, 2023

Conversation

saurabhd336
Copy link
Contributor

Disallow peer download when replication is < 2

@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2023

Codecov Report

Merging #11469 (450ac31) into master (f6b29aa) will decrease coverage by 63.08%.
Report is 5 commits behind head on master.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11469       +/-   ##
=============================================
- Coverage     63.07%    0.00%   -63.08%     
=============================================
  Files          2325     2249       -76     
  Lines        124772   121043     -3729     
  Branches      19052    18497      -555     
=============================================
- Hits          78696        0    -78696     
- Misses        40475   121043    +80568     
+ Partials       5601        0     -5601     
Flag Coverage Δ
integration 0.00% <0.00%> (-0.01%) ⬇️
integration1 ?
integration2 0.00% <0.00%> (ø)
java-11 0.00% <0.00%> (-63.04%) ⬇️
java-17 ?
java-20 ?
temurin 0.00% <0.00%> (-63.08%) ⬇️
unittests ?
unittests1 ?
unittests2 ?

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

Files Changed Coverage Δ
...he/pinot/segment/local/utils/TableConfigUtils.java 0.00% <0.00%> (-71.26%) ⬇️

... and 1981 files with indirect coverage changes

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

@xiangfu0
Copy link
Contributor

xiangfu0 commented Sep 7, 2023

Can you please rebase and fix the test?

@saurabhd336
Copy link
Contributor Author

@xiangfu0 thanks for reviewing. Tests have been fixed.

@Jackie-Jiang Jackie-Jiang added the user-experience Related to user experience label Sep 12, 2023
@Jackie-Jiang Jackie-Jiang merged commit 0929b4c into apache:master Sep 12, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user-experience Related to user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants