[jira] [Commented] (CALCITE-1242) Allow configuration of maximum allowed value for maxRowCount

2016-05-18 Thread Duo Xu (JIRA)
[ 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

[jira] [Commented] (CALCITE-1242) Allow configuration of maximum allowed value for maxRowCount

2016-05-18 Thread Duo Xu (JIRA)
[ 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

[jira] [Commented] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Josh Elser (JIRA)
[ 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

[jira] [Updated] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Josh Elser (JIRA)
[ 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

[jira] [Updated] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Josh Elser (JIRA)
[ 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 >

[jira] [Commented] (CALCITE-1244) `offset` is ignored in FetchRequest

2016-05-18 Thread Francis Chuang (JIRA)
[ 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)

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Venki Korukanti (JIRA)
[ 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

[jira] [Commented] (CALCITE-1242) Allow configuration of maximum allowed value for maxRowCount

2016-05-18 Thread Josh Elser (JIRA)
[ 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?

[jira] [Updated] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Josh Elser (JIRA)
[ 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 >

[jira] [Updated] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Julian Hyde (JIRA)
[ 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 >

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Created] (CALCITE-1243) max_row_count in Proto file should be signed int

2016-05-18 Thread Duo Xu (JIRA)
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

[jira] [Commented] (CALCITE-1242) Allow configuration of maximum allowed value for maxRowCount

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Updated] (CALCITE-1242) Allow configuration of maximum allowed value for maxRowCount

2016-05-18 Thread Josh Elser (JIRA)
[ 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

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Assigned] (CALCITE-1242) Make maxRowCount configurable

2016-05-18 Thread Josh Elser (JIRA)
[ 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 >

[jira] [Updated] (CALCITE-1242) Make maxRowCount configurable

2016-05-18 Thread Josh Elser (JIRA)
[ 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 > - > >

[jira] [Created] (CALCITE-1242) Make maxRowCount configurable

2016-05-18 Thread Duo Xu (JIRA)
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

[jira] [Comment Edited] (CALCITE-1237) Session windows for streaming SQL

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Commented] (CALCITE-1237) Session windows for streaming SQL

2016-05-18 Thread Julian Hyde (JIRA)
[ 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

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Venki Korukanti (JIRA)
[ 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

[jira] [Commented] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Julian Hyde (JIRA)
[ 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,

[jira] [Commented] (CALCITE-1208) Improve two-level column structure handling

2016-05-18 Thread Maryann Xue (JIRA)
[ 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

[jira] [Created] (CALCITE-1241) Add a freemarker variable for adding non reserved keywords to Parser.jj template

2016-05-18 Thread Venki Korukanti (JIRA)
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

[jira] [Commented] (CALCITE-1237) Session windows for streaming SQL

2016-05-18 Thread Fabian Hueske (JIRA)
[ 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.