You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2019/08/05 15:43:29 UTC

[GitHub] [drill] agozhiy commented on a change in pull request #1779: DRILL-7222: Visualize estimated and actual row counts for a query

agozhiy 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_r310661279
 
 

 ##########
 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>" );
 
 Review comment:
   ```suggestion
           var status = tgtElem.append("<div class='estRows' title='Estimated'>(" + opRowCountMap[key] + ")</div>");
   ```

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services