You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by amansinha100 <gi...@git.apache.org> on 2018/02/09 17:10:22 UTC

[GitHub] drill pull request #1116: DRILL-6140: Correctly list Operators in a Query Pr...

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.  


---