Closed vmaurin closed 5 months ago
Attention: 7 lines
in your changes are missing coverage. Please review.
Comparison is base (
f8d0d15
) 95.06% compared to head (1395718
) 95.03%.
Files | Patch % | Lines |
---|---|---|
aiokafka/admin/client.py | 75.86% | 4 Missing and 3 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@vmaurin I think it's ready to merge, no objections?
@ods Sure, thank you for fixing my mistake :+1:
When doing stream processing, it is convinient to use "transient" topic :
The java kafka streams client is using the deleteRecords of the admin client to perform this operation. It is lacking in aiokafka
The KIP reference https://cwiki.apache.org/confluence/display/KAFKA/KIP-204+%3A+Adding+records+deletion+operation+to+the+new+Admin+Client+API
refs #967
Changes
Fixes #967
Checklist
CHANGES
folder<issue_id>.<type>
(e.g.588.bugfix
)issue_id
change it to the pr id after creating the PR.feature
: Signifying a new feature..bugfix
: Signifying a bug fix..doc
: Signifying a documentation improvement..removal
: Signifying a deprecation or removal of public API..misc
: A ticket has been closed, but it is not of interest to users.Fix issue with non-ascii contents in doctest text files.