jinhwan2 / kafka-admin-server

카프카 어드민 서버
MIT License
2 stars 1 forks source link

기능 분석 #7

Open jinhwan2 opened 2 years ago

jinhwan2 commented 2 years ago

개요

스펙은 kafka-admin-client의 스펙에 종속된다.

논의 필요 내용

어디까지 구현해볼까?

kafka-admin-client 인터페이스

3.3.1 기준

cluster #13

describeCluster

  • [x] clusterId
  • [x] Nodes
  • [x] Controller
  • [x] AclOperation

topic #6

config #10

consumer #12

acl

describeAcls createAcls deleteAcls

file alterReplicaLogDirs describeLogDirs describeReplicaLogDirs

token createDelegationToken renewDelegationToken expireDelegationToken describeDelegationToken

etc.. createPartitions deleteRecords electLeaders alterPartitionReassignments listPartitionReassignments listOffsets describeClientQuotas alterClientQuotas describeUserScramCredentials alterUserScramCredentials describeFeatures updateFeatures describeMetadataQuorum unregisterBroker describeProducers describeTransactions abortTransaction

fenceProducers listTransactions

monitoring

metrics

lsh110600 commented 2 years ago

TopicController 코드 중, PUT request 보내는 부분 변경이 필요합니다.

  1. method 이름 변경
  2. 두 번째 인자에 @RequestBody 어노테이션 추가

[As-is] @PutMapping("/topics/{topicName}")

boolean updateBrokerConfig(@PathVariable String topicName, ConfigModifyRequest configModifyRequest) throws ExecutionException, InterruptedException {
    return this.topicService.updateConfig(topicName, configModifyRequest);
}

[To-be] @PutMapping("/topics/{topicName}")

boolean updateTopicConfig(@PathVariable String topicName, @RequestBody ConfigModifyRequest configModifyRequest) throws ExecutionException, InterruptedException {
    return this.topicService.updateConfig(topicName, configModifyRequest);
}
jinhwan2 commented 2 years ago

TopicController 코드 중, PUT request 보내는 부분 변경이 필요합니다.

  1. method 이름 변경
  2. 두 번째 인자에 @requestbody 어노테이션 추가

[As-is] @PutMapping("/topics/{topicName}")

boolean updateBrokerConfig(@PathVariable String topicName, ConfigModifyRequest configModifyRequest) throws ExecutionException, InterruptedException {
    return this.topicService.updateConfig(topicName, configModifyRequest);
}

[To-be] @PutMapping("/topics/{topicName}")

boolean updateTopicConfig(@PathVariable String topicName, @RequestBody ConfigModifyRequest configModifyRequest) throws ExecutionException, InterruptedException {
    return this.topicService.updateConfig(topicName, configModifyRequest);
}

에고 제가 실수를 했네요 ㅎㅎ 버그제보는 댓글보다 이슈하나따서 버그로 리포팅 해주셔도 돼요

이건 집가서 수정할게요~ 감사합니다 💯