Hi all,

I am working with rebasing Drill to new Calcite and I faced a problem with
describe table with column qualifier. For example, Drill supports the next
construction:

DESCRIBE TABLE_NAME 'COLUMN_PATTERN'
(You can see it in TestInfoSchema.describeTableWithColQualifier and
TestInfoSchema.describeTableWithSchemaAndColQualifier tests)

After da875a67 commit, columnQualifier was removed from SqlDescribe and
Drill loses this functionality without it. I tried to override it but my
Describe methods were placed under Calcite SqlDescribe method in SqlStmt()
and I always got ParseException. So I propose to move the next block:

    <#-- Add methods to parse additional statements here -->
    <#list parser.statementParserMethods as method>
            stmt = ${method}
    </#list>

on top of the SqlStmt() method in Parser.jj to allow overriding of Calcite
statements. Could you please tell what do you think about these changes and
do I need to create a Jira for it? Or how can I add column qualifier to
SqlDescribe without these changes?

Best regards,
Roman Kulyk

Reply via email to