[ 
https://issues.apache.org/jira/browse/KAFKA-13716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17527730#comment-17527730
 ] 

Richard Joerger commented on KAFKA-13716:
-----------------------------------------

Hi Team,

 

I'd like to work on this particular issue but it seems that the requisite code 
we would need in the MockAdminClient class is missing, specifically the 
"MockAdminClient.deleteRecords()" method is not implemented. Would it make 
sense to create another Jira and then link it to this one as a blocker? I'm 
also not sure how involved the changes in the MockAdminClient would be to 
implement such a feature or if that would be a similar level of difficulty to 
this particular Jira. 

> add tests for `DeleteRecordsCommand` class
> ------------------------------------------
>
>                 Key: KAFKA-13716
>                 URL: https://issues.apache.org/jira/browse/KAFKA-13716
>             Project: Kafka
>          Issue Type: Test
>          Components: tools
>            Reporter: Luke Chen
>            Assignee: Shivanjal Arora
>            Priority: Major
>              Labels: Newbie, newbie
>
> Found there's no tests for `DeleteRecordsCommand` class, which is used in 
> `kafka-delete-records.sh`. We should add it.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to