Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Fix the regression for https://github.com/streamnative/kop/pull/1749#2024

Merged
BewareMyPower merged 1 commit intostreamnative:masterfrom
BewareMyPower:bewaremypower/fix-leak-purgatory
Oct 31, 2023
Merged

Fix the regression for https://github.com/streamnative/kop/pull/1749#2024
BewareMyPower merged 1 commit intostreamnative:masterfrom
BewareMyPower:bewaremypower/fix-leak-purgatory

Conversation

@BewareMyPower
Copy link
Collaborator

Motivation

#1923 removes the completion for the produce purgatory in #1749.

Modification

Added back the logic to check and complete the produce purgatory.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

### Motivation

streamnative#1923 removes the completion for
the produce purgatory in streamnative#1749.

### Modification

Added back the logic to check and complete the produce purgatory.
@BewareMyPower BewareMyPower self-assigned this Oct 31, 2023
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Oct 31, 2023
@codecov
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #2024 (c9a5f8a) into master (db4a9ad) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2024      +/-   ##
============================================
- Coverage     16.98%   16.96%   -0.02%     
+ Complexity      730      729       -1     
============================================
  Files           191      191              
  Lines         14309    14310       +1     
  Branches       1339     1339              
============================================
- Hits           2430     2428       -2     
- Misses        11703    11705       +2     
- Partials        176      177       +1     
Files Coverage Δ
...ve/pulsar/handlers/kop/storage/ReplicaManager.java 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

@BewareMyPower BewareMyPower merged commit aa2f171 into streamnative:master Oct 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-need-doc This pr does not need any document type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants