[KAFKA-5324] AdminClient: add close with timeout, fix some timeout bugs

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.11.0.0
• Fix Version/s: 0.11.0.0
• Component/s: admin
• Assignee: Colin McCabe
• Reporter: Colin McCabe
• Created: 24/May/17 23:04
• Updated: 31/May/17 02:07
• Resolved: 31/May/17 02:07

Description

• Add a close method with a timeout, similar to some other APIs. Close waits for all calls to complete. Once the timeout expires, all calls are aborted.

• Fix a minor bug which made it impossible to have per-call timeouts which were longer than the default timeout.

• Fix a minor bug where we could oversleep short timeouts because we didn't adjust the polling interval based on the extant timeouts.

• Add some unit tests for timeouts

Issue Links

• links to: GitHub Pull Request #3141