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

    https://github.com/apache/drill/pull/1066#discussion_r158374197
  
    --- Diff: exec/java-exec/src/main/codegen/includes/parserImpls.ftl ---
    @@ -351,4 +351,23 @@ SqlNode SqlDropFunction() :
        {
            return new SqlDropFunction(pos, jar);
        }
    -}
    \ No newline at end of file
    +}
    +
    +<#if !parser.includeCompoundIdentifier >
    --- End diff --
    
    Actually, it does not cause a new behaviour or functionality, it just helps 
to preserve old one after changes in Calcite `Parser.jj`. Therefore existing 
unit tests cover this change.


---

Reply via email to