Github user kkhatua commented on a diff in the pull request:
https://github.com/apache/drill/pull/1197#discussion_r179552379
--- 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 --
Using a negative ("-ve") value constant. Will fix the comment
---
