[KAFKA-13512] topicIdsToNames and topicNamesToIds allocate unnecessary maps

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.1.0
• Fix Version/s: 3.1.0
• Assignee: Justine Olshan
• Reporter: Justine Olshan
• Created: 07/Dec/21 17:24
• Updated: 09/Dec/21 21:28
• Resolved: 09/Dec/21 21:28

Description

Currently we write the methods as follows:

def topicNamesToIds(): util.Map[String, Uuid] = {
    new util.HashMap(metadataSnapshot.topicIds.asJava)
}

We do not need to allocate a new map however, we can simply use

Collections.unmodifiableMap(metadataSnapshot.topicIds.asJava)

We can do something similar for the topicIdsToNames implementation.

Issue Links

• links to: GitHub Pull Request #11576