[HBASE-24492] ProtobufLogReader.readNext does not need looping

• Type: Task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 1.7.0, 2.4.0
• Component/s: Replication, wal
• Hadoop Flags: Reviewed
• Assignee: Viraj Jasani
• Reporter: Viraj Jasani
• Created: 02/Jun/20 17:08
• Updated: 04/Oct/20 19:48
• Resolved: 03/Jun/20 16:57

Description

ProtobufLogReader.readNext is using infinite while loop for the whole method, however it returns value or throws errors in single execution. while(true) was part of initial code and with better improvements in the method, this loop has not been removed yet.

Issue Links

• links to: GitHub Pull Request #1831
• links to: GitHub Pull Request #2478
• links to: GitHub Pull Request #2479

Sub-Tasks

• Backport HBASE-24492 to all 2.x branch: Resolved