[IMPALA-1649] Get rid of TCounterType::DOUBLE_VALUE

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Won't Fix
• Affects Version/s: Impala 2.2
• Component/s: Backend
• Labels: ramp-up
• Target Version: Product Backlog
• Assignee: Pranay Singh
• Reporter: Henry Robinson
• Created: 08/Jan/15 19:18
• Updated: 21/Sep/17 18:06
• Resolved: 21/Sep/17 18:06

Description

TCounterType::DOUBLE_VALUE seems to exist only to force PrettyPrinter::Print() to use 2 decimal places, where TCounterType::UNIT would only use 2dp if the value was sufficiently large.

We should either force the caller to provide a required precision to Print(), or replace DOUBLE_VALUE with FIXED_PRECISION (and remove the unnecessary cast) to enforce this behaviour.