[
https://issues.apache.org/jira/browse/CALCITE-7599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chris Dennis updated CALCITE-7599:
----------------------------------
Description:
There is a small inconsistency remaining between the reference documentation
[https://calcite.apache.org/docs/reference.html#syntax] and the parser syntax.
The documentation example shows unquoted numeric literals being used as values
in a key-value hint. The reference documentation use this as an example query:
{code:sql}
SELECT /*+ hint1, hint2(a=1, b=2) */
...
FROM
tableName /*+ hint3(5, 'x') */
JOIN
tableName /*+ hint4(c=id), hint5 */
...
{code}
The unquoted literals in {{hint2(a=1, b=2)}} do not parse using the current
grammar however, and are not present in copy of the documentation query used in
SqlHintsConverterTest#testDocumentationExample}}. I think it should be pretty
simple to expand the grammar to support the same set of literals as is
currently supported by {{ParenthesizedLiteralOptionCommaList}}, that is
{{StringLiteral | NumericLiteral}} (in addition to the {{SimpleIdentifier}}
that is currently also supported for the value of a key-value pair). At the
same time we should update the test so that the query exactly matches the
documentation example by not quoting the two numeric constants. To be clear
that means:
[-]
was:
There is a small inconsistency remaining between the reference documentation
[https://calcite.apache.org/docs/reference.html#syntax] and the parser syntax.
The documentation example shows unquoted numeric literals being used as values
in a key-value hint. The reference documentation use this example query:
```
This isn't reflected in the grammar that follows in the document, the parser
grammar itself, or in {{SqlHintsConverterTest#testDocumentationExample}}. I
think it should be pretty simple to expand the grammar to support either a
string or numeric literal, and to update the test to exactly match the
documentation example by not quoting the two numeric constants.
> Enhance parser to allow numeric literals as values in k/v hints as suggested
> by documentation
> ---------------------------------------------------------------------------------------------
>
> Key: CALCITE-7599
> URL: https://issues.apache.org/jira/browse/CALCITE-7599
> Project: Calcite
> Issue Type: Bug
> Components: core
> Affects Versions: 1.42.0
> Reporter: Chris Dennis
> Priority: Minor
> Labels: pull-request-available
>
> There is a small inconsistency remaining between the reference documentation
> [https://calcite.apache.org/docs/reference.html#syntax] and the parser
> syntax. The documentation example shows unquoted numeric literals being used
> as values in a key-value hint. The reference documentation use this as an
> example query:
> {code:sql}
> SELECT /*+ hint1, hint2(a=1, b=2) */
> ...
> FROM
> tableName /*+ hint3(5, 'x') */
> JOIN
> tableName /*+ hint4(c=id), hint5 */
> ...
> {code}
> The unquoted literals in {{hint2(a=1, b=2)}} do not parse using the current
> grammar however, and are not present in copy of the documentation query used
> in SqlHintsConverterTest#testDocumentationExample}}. I think it should be
> pretty simple to expand the grammar to support the same set of literals as is
> currently supported by {{ParenthesizedLiteralOptionCommaList}}, that is
> {{StringLiteral | NumericLiteral}} (in addition to the {{SimpleIdentifier}}
> that is currently also supported for the value of a key-value pair). At the
> same time we should update the test so that the query exactly matches the
> documentation example by not quoting the two numeric constants. To be clear
> that means:
> [-]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
