[HDDS-4448] Duplicate refreshPipeline in listStatus

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Component/s: Ozone Manager
• Labels: pull-request-available
• Target Version/s: 1.1.0
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 10/Nov/20 12:19
• Updated: 19/Nov/20 08:29
• Resolved: 19/Nov/20 05:21

Description

KeyManagerImpl#listStatus issues duplicate refreshPipeline for each file.

HDDS-3824 moved refreshPipeline outside the bucket lock. But HDDS-3658 added it back, while keeping the one outside the lock, probably as a result of merge conflict resolution.

Also, weichiu pointed out that refreshPipeline supports lists, too, so the calls can be batched to reduce number of RPC.

CC msingh

Issue Links

• links to: GitHub Pull Request #1569