Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1197#discussion_r179403081
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java
 ---
    @@ -38,7 +39,16 @@
      * Wrapper class for profiles of ALL operator instances of the same 
operator type within a major fragment.
      */
     public class OperatorWrapper {
    +  //private static final org.slf4j.Logger logger = 
org.slf4j.LoggerFactory.getLogger(OperatorWrapper.class);
    +
    +  private static final String HTML_ATTRIB_SPILLS = "spills";
    +  private static final String HTML_ATTRIB_CLASS = "class";
    +  private static final String HTML_ATTRIB_STYLE = "style";
    +  private static final String HTML_ATTRIB_TITLE = "title";
    +  private static final DecimalFormat DECIMAL_FORMATTER = new 
DecimalFormat("#.##");
       private static final String UNKNOWN_OPERATOR = "UNKNOWN_OPERATOR";
    +  //-ve value to indicate absence of metric
    --- End diff --
    
    `-ve value`?


---

Reply via email to