Github user vvysotskyi commented on the pull request:

    
https://github.com/apache/drill/commit/9073aed67d89e8b2188870d6c812706085c9c41b#commitcomment-27970713
  
    In logical/src/main/java/org/apache/drill/common/expression/SchemaPath.java:
    In logical/src/main/java/org/apache/drill/common/expression/SchemaPath.java 
on line 46:
    @gparai, for the case, when `DYNAMIC_STAR` is passed into parseFromString() 
method, the result will be the same, as for the passing "*" string - error. 
`DYNAMIC_STAR` is used in the same places, where usual column names are used. 
But column names may contain some symbols which also breaks Lexer/Parser. 
Therefore column names are wrapped into `SchemaPath` using `getSimplePath()` 
method. `DYNAMIC_STAR` also wrapped into the `SchemaPath`, and its string 
representation may be received using `SchemaPath.STAR_COLUMN.toExpr()` -> 
"\`**\`". So `SchemaPath.parseFromString(SchemaPath.STAR_COLUMN.toExpr())` is 
able to parse it.


---

Reply via email to