[ https://issues.apache.org/jira/browse/DRILL-5230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15851804#comment-15851804 ]
ASF GitHub Bot commented on DRILL-5230: --------------------------------------- Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/739#discussion_r99385541 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/TableBuilder.java --- @@ -56,6 +47,30 @@ public TableBuilder(final String[] columns) { sb.append("</tr>\n"); } + /** + * Representation of a millisecond duration in a short readable text + * @param millis Duration in milliseconds + * @return Human-Readable Duration Time + */ + public String shortDurationFormat(long millis) { --- End diff -- Used elsewhere? If so, consider making static (so we don't need an instance.) If only used here, consider making private. In fact, maybe move this out to a utility class. This logic can be used anywhere we want to display a duration such as in logs, in tests, etc. > Translation of millisecond duration into hours is incorrect > ----------------------------------------------------------- > > Key: DRILL-5230 > URL: https://issues.apache.org/jira/browse/DRILL-5230 > Project: Apache Drill > Issue Type: Bug > Components: Web Server > Affects Versions: 1.9.0 > Reporter: Kunal Khatua > Labels: easyfix, ready-to-commit > Fix For: 1.10.0 > > Original Estimate: 24h > Remaining Estimate: 24h > > The method > {code:JAVA}org.apache.drill.exec.server.rest.profile.TableBuilder.appendMillis(long, > String){code} > has a bug where the human readable translation of a 1+ hr duration in > milliseconds is reported incorrectly. > This has to do with the {code:JAVA}SimpleDateFormat.format() {code} method > incorectly translating it. > For e.g. > {code:JAVA} > long x = 4545342L; //1 hour 15 min 45.342 sec > public void appendMillis(x, null); > {code} > This formats the value as {noformat}17h15m{noformat} instead of > {noformat}1h15m{noformat} -- This message was sent by Atlassian JIRA (v6.3.15#6346)