Github user amansinha100 commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1116#discussion_r167269862
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileWrapper.java
 ---
    @@ -322,4 +330,16 @@ public String getOperatorsJSON() {
       public boolean isOnlyImpersonationEnabled() {
         return onlyImpersonationEnabled;
       }
    +
    +  //Generates operator names inferred from physical plan
    +  private void generateOpMap(String plan) {
    +    this.physicalOperatorMap = new HashMap<String,String>();
    +    String[] operatorLine = plan.split("\\n");
    +    for (String line : operatorLine) {
    +      String[] lineToken = line.split("\\s+", 3);
    +      String operatorPath = lineToken[0].trim().replaceFirst("-", "-xx-"); 
//Required format for lookup
    --- End diff --
    
    Can you add (as a comment) the  sample text from the physical plan output 
that this is parsing.  


---

Reply via email to