Skip to content

Commit

Permalink
Fix formatting cumulative memory in Web UI
Browse files Browse the repository at this point in the history
Value for cumulative memory of a query was previously reported using
following format:

"4.33T seconds". It is not obviouls that it represents integer of memory
coinsumption over time.
This commit changes format to:
"4.33TB*seconds"
  • Loading branch information
losipiuk committed May 31, 2022
1 parent aff1d64 commit 954125e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion core/trino-main/src/main/resources/webapp/dist/query.js

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -1502,11 +1502,11 @@ export class QueryDetail extends React.Component {
Cumulative User Memory
</td>
<td className="info-text">
{formatDataSizeBytes(query.queryStats.cumulativeUserMemory / 1000.0) + " seconds"}
{formatDataSize(query.queryStats.cumulativeUserMemory / 1000.0) + "*seconds"}
</td>
{taskRetriesEnabled &&
<td className="info-failed">
{formatDataSizeBytes(query.queryStats.failedCumulativeUserMemory / 1000.0) + " seconds"}
{formatDataSize(query.queryStats.failedCumulativeUserMemory / 1000.0) + "*seconds"}
</td>
}
</tr>
Expand Down

0 comments on commit 954125e

Please sign in to comment.