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

ASF GitHub Bot commented on KAFKA-7005:
---------------------------------------

big-andy-coates opened a new pull request #5184: KAFKA-7005: Remove duplicate 
resource class.
URL: https://github.com/apache/kafka/pull/5184
 
 
   Fix for [KAFKA-7005](https://issues.apache.org/jira/browse/KAFKA-7005).
   
   This is a follow-on change requested as part of the initial PR for KIP-290 
#5117.  @cmccabe requested that the `resource.Resource` class be factored out 
in favour of `ConfigResource` to avoid confusion between all the `Resource` 
implementations.
   
   cc @cmccabe, @junrao 
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Remove duplicate Java Resource class.
> -------------------------------------
>
>                 Key: KAFKA-7005
>                 URL: https://issues.apache.org/jira/browse/KAFKA-7005
>             Project: Kafka
>          Issue Type: Sub-task
>          Components: core, security
>            Reporter: Andy Coates
>            Assignee: Andy Coates
>            Priority: Major
>             Fix For: 2.0.0
>
>
> Relating to one of the outstanding work items in PR 
> [#5117|[https://github.com/apache/kafka/pull/5117]...]
> TheĀ o.a.k.c.request.Resource class could be dropped in favour of 
> o.a.k.c..config.ConfigResource.
> This will remove the duplication of `Resource` classes.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to