[IMPALA-4880] Clarify synchronization policy for 'done_' in KuduScanNode

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Impala 2.8.0
• Fix Version/s: Impala 2.9.0
• Component/s: Backend
• Labels: kudu
• Assignee: Sailesh Mukil
• Reporter: Sailesh Mukil
• Created: 03/Feb/17 22:48
• Updated: 18/Mar/17 00:30
• Resolved: 18/Mar/17 00:30

Description

We need to clarify that it's safe to optimistically read 'done_' in some places in KuduScanNode. This is to avoid some confusion as the code synchronizes access to the variable in certain places but not in others.