[
https://issues.apache.org/jira/browse/DRILL-7222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16900166#comment-16900166
]
ASF GitHub Bot commented on DRILL-7222:
---------------------------------------
agozhiy commented on pull request #1779: DRILL-7222: Visualize estimated and
actual row counts for a query
URL: https://github.com/apache/drill/pull/1779#discussion_r310666533
##########
File path: exec/java-exec/src/main/resources/rest/profile/profile.ftl
##########
@@ -84,6 +84,29 @@
document.getElementById(warningElemId).style.display="none";
}
+ //Injects Estimated Rows
+ function injectEstimatedRows() {
+ Object.keys(opRowCountMap).forEach(key => {
+ var tgtElem = $("td.estRowsAnchor[key='"+key+"']");
+ var status = tgtElem.append( "<div class='estRows'
title='Estimated'>("+opRowCountMap[key]+")</div>" );
+ });
+ }
+
+ //Toggle Estimates' visibility
+ function toggleEstimates(tgtColumn) {
+ var colClass = '.est'+tgtColumn
Review comment:
```suggestion
var colClass = '.est' + tgtColumn
```
----------------------------------------------------------------
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]
> Visualize estimated and actual row counts for a query
> -----------------------------------------------------
>
> Key: DRILL-7222
> URL: https://issues.apache.org/jira/browse/DRILL-7222
> Project: Apache Drill
> Issue Type: Improvement
> Components: Web Server
> Affects Versions: 1.16.0
> Reporter: Kunal Khatua
> Assignee: Kunal Khatua
> Priority: Major
> Labels: doc-impacting, user-experience
> Fix For: 1.17.0
>
>
> With statistics in place, it would be useful to have the *estimated* rowcount
> along side the *actual* rowcount query profile's operator overview.
> We can extract this from the Physical Plan section of the profile.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
