[KAFKA-5911] Avoid creation of extra Map for futures in KafkaAdminClient

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Later
• Labels: client
• Assignee: Unassigned
• Reporter: Ted Yu
• Created: 15/Sep/17 19:30
• Updated: 18/Oct/17 23:18
• Resolved: 18/Oct/17 23:18

Description

In various methods from KafkaAdminClient, there is extra Map created when constructing XXResult instance.
e.g.

        return new DescribeReplicaLogDirResult(new HashMap<TopicPartitionReplica, KafkaFuture<ReplicaLogDirInfo>>(futures));

Prior to returning, futures Map is already filled.
Calling get() and values() does not involve the internals of HashMap when we consider thread-safety.

The extra Map doesn't need to be created.

Issue Links

• links to: GitHub Pull Request #3925