kkhatua commented on a change in pull request #1779: DRILL-7222: Visualize
estimated and actual row counts for a query
URL: https://github.com/apache/drill/pull/1779#discussion_r280902167
##########
File path: exec/java-exec/src/main/resources/rest/profile/profile.ftl
##########
@@ -587,6 +622,49 @@
if (e.target.form)
<#if
model.isOnlyImpersonationEnabled()>doSubmitQueryWithUserName()<#else>doSubmitQueryWithAutoLimit()</#if>;
});
+
+ // Convert scientific to Decimal [Ref:
https://gist.github.com/jiggzson/b5f489af9ad931e3d186]
+ function scientificToDecimal(num) {
Review comment:
Ideally, yes.
But the way we construct the `Operators Overview` table is by only injecting
values (which then get converted to human readable formats based on data
types). That makes it difficult for me to inject these `<div>` elements during
construction.
Also, with Javascript, I reduce the processing and construction overhead on
the WebServer and have the client browsers do this formatting.
In theory, the entire profile can be constructed by Javascript from the JSON
profile, but basic summaries are easier to implement (and faster to calculate)
in Java.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
