[DRILL-5230] Translation of millisecond duration into hours is incorrect

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.0
• Fix Version/s: 1.10.0
• Component/s: Web Server
• Labels: easyfix ready-to-commit
• Flags: Important
• Assignee: Kunal Khatua
• Reporter: Kunal Khatua
• Reviewer: Paul Rogers
• Created: 27/Jan/17 22:11
• Updated: 25/Feb/17 00:26
• Resolved: 21/Feb/17 15:36

Description

The method

org.apache.drill.exec.server.rest.profile.TableBuilder.appendMillis(long, String)

has a bug where the human readable translation of a 1+ hr duration in milliseconds is reported incorrectly.
This has to do with the

SimpleDateFormat.format() 

method incorectly translating it.

For e.g.

long x = 4545342L; //1 hour 15 min 45.342 sec
public void appendMillis(x, null);

This formats the value as

17h15m

instead of

1h15m

Issue Links

• is required by: DRILL-5190 Display planning and queued time for a query in its profile page
• links to: GitHub Pull Request #739