Skip to content

Commit

Permalink
[ISSUE #1279]⚡️Optimize name server DefaultRequestProcessor#get_all_t…
Browse files Browse the repository at this point in the history
…opic_list_from_nameserver
  • Loading branch information
mxsm committed Nov 24, 2024
1 parent a40e982 commit 217a8f8
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion rocketmq-namesrv/src/route/route_info_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -721,12 +721,13 @@ impl RouteInfoManager {
}

pub(crate) fn get_all_topic_list(&self) -> TopicList {
let lock = self.lock.read();

Check warning on line 724 in rocketmq-namesrv/src/route/route_info_manager.rs

View check run for this annotation

Codecov / codecov/patch

rocketmq-namesrv/src/route/route_info_manager.rs#L724

Added line #L724 was not covered by tests
let topics = self
.topic_queue_table
.keys()
.cloned()
.collect::<Vec<CheetahString>>();

drop(lock);

Check warning on line 730 in rocketmq-namesrv/src/route/route_info_manager.rs

View check run for this annotation

Codecov / codecov/patch

rocketmq-namesrv/src/route/route_info_manager.rs#L730

Added line #L730 was not covered by tests
TopicList {
topic_list: topics,
broker_addr: None,
Expand Down

0 comments on commit 217a8f8

Please sign in to comment.