[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15290034#comment-15290034
]
Duo Xu commented on CALCITE-1242:
-
or avoid confusion, maxRowCount should only be translated to “number
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15290030#comment-15290030
]
Duo Xu commented on CALCITE-1242:
-
I would suggest we expose two parameters instead of a single
[
https://issues.apache.org/jira/browse/CALCITE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289966#comment-15289966
]
Josh Elser commented on CALCITE-1243:
-
Turns out that the Avatica (java) client is sending negative
[
https://issues.apache.org/jira/browse/CALCITE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1243:
Fix Version/s: (was: avatica-1.9.0)
avatica-1.8.0
> max_row_count in Proto
[
https://issues.apache.org/jira/browse/CALCITE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1243:
Priority: Blocker (was: Major)
> max_row_count in Proto file should be signed int
>
[
https://issues.apache.org/jira/browse/CALCITE-1244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289931#comment-15289931
]
Francis Chuang commented on CALCITE-1244:
-
This looks very similar to (if not duplicates)
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289896#comment-15289896
]
Julian Hyde commented on CALCITE-1241:
--
Yes, please add a test to SqlParserTest as part of the
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289879#comment-15289879
]
Venki Korukanti commented on CALCITE-1241:
--
I think it makes sense to have a test in Calcite to
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289838#comment-15289838
]
Josh Elser commented on CALCITE-1242:
-
bq. Can you remind me why there is a maximum maxRowCount?
[
https://issues.apache.org/jira/browse/CALCITE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1243:
Fix Version/s: avatica-1.9.0
> max_row_count in Proto file should be signed int
>
[
https://issues.apache.org/jira/browse/CALCITE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Julian Hyde updated CALCITE-1243:
-
Assignee: Josh Elser (was: Julian Hyde)
> max_row_count in Proto file should be signed int
>
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289805#comment-15289805
]
Julian Hyde commented on CALCITE-1241:
--
The list of reserved keywords doesn't change very often
Duo Xu created CALCITE-1243:
---
Summary: max_row_count in Proto file should be signed int
Key: CALCITE-1243
URL: https://issues.apache.org/jira/browse/CALCITE-1243
Project: Calcite
Issue Type: Bug
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289793#comment-15289793
]
Julian Hyde commented on CALCITE-1242:
--
[~elserj] Can you remind me why there is a maximum
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1242:
Summary: Allow configuration of maximum allowed value for maxRowCount
(was: Make maxRowCount
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289779#comment-15289779
]
Julian Hyde commented on CALCITE-1241:
--
SQL syntax unfortunately allows expressions and queries in
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser reassigned CALCITE-1242:
---
Assignee: Josh Elser (was: Julian Hyde)
> Make maxRowCount configurable
>
[
https://issues.apache.org/jira/browse/CALCITE-1242?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1242:
Fix Version/s: avatica-1.9.0
> Make maxRowCount configurable
> -
>
>
Duo Xu created CALCITE-1242:
---
Summary: Make maxRowCount configurable
Key: CALCITE-1242
URL: https://issues.apache.org/jira/browse/CALCITE-1242
Project: Calcite
Issue Type: Bug
[
https://issues.apache.org/jira/browse/CALCITE-1237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289718#comment-15289718
]
Julian Hyde edited comment on CALCITE-1237 at 5/18/16 8:17 PM:
---
bq. Q3: I
[
https://issues.apache.org/jira/browse/CALCITE-1237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289718#comment-15289718
]
Julian Hyde commented on CALCITE-1237:
--
bq. Q3: I think supporting session in the GROUP BY clause
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289584#comment-15289584
]
Venki Korukanti commented on CALCITE-1241:
--
This is the most common issue, I have seen. One
[
https://issues.apache.org/jira/browse/CALCITE-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289546#comment-15289546
]
Julian Hyde commented on CALCITE-1241:
--
It makes sense, and the patch looks good. But I wonder,
[
https://issues.apache.org/jira/browse/CALCITE-1208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15289511#comment-15289511
]
Maryann Xue commented on CALCITE-1208:
--
Hi [~jni], please check this file for a (hopefully
Venki Korukanti created CALCITE-1241:
Summary: Add a freemarker variable for adding non reserved
keywords to Parser.jj template
Key: CALCITE-1241
URL: https://issues.apache.org/jira/browse/CALCITE-1241
[
https://issues.apache.org/jira/browse/CALCITE-1237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15288687#comment-15288687
]
Fabian Hueske commented on CALCITE-1237:
Hi [~julianhyde], thanks for the proposal! I like it.
26 matches
Mail list logo