[jira] [Resolved] (CALCITE-1654) Serialized messages are generated even though they are not logged

2017-02-22 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Josh Elser resolved CALCITE-1654. - Resolution: Fixed Fixed in

[jira] [Commented] (CALCITE-1654) Serialized messages are generated even though they are not logged

2017-02-22 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1654?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879235#comment-15879235 ] Josh Elser commented on CALCITE-1654: - Put up a pull-request with the trivial change..

[jira] [Commented] (CALCITE-1069) In Aggregate, combine GROUPING columns into one GROUP_ID column

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1069?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879141#comment-15879141 ] Julian Hyde commented on CALCITE-1069: -- I'm thinking of an alternative solution. Currently, as you

[jira] [Commented] (CALCITE-1515) Support TableFunctionScan in RelBuilder

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879107#comment-15879107 ] Julian Hyde commented on CALCITE-1515: -- Regarding 1. Note that {{RelBuilder.scan}} calls

[jira] [Resolved] (CALCITE-1652) Allow GROUPING to have multiple arguments, like GROUPING_ID

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1652?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-1652. -- Resolution: Fixed Fix Version/s: 1.12.0 Fixed in

[jira] [Commented] (CALCITE-1515) Support TableFunctionScan in RelBuilder

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879056#comment-15879056 ] Julian Hyde commented on CALCITE-1515: -- Regarding 2: use {{ImmutableList.copyOf(inputs)}}. >

[jira] [Commented] (CALCITE-1641) Base functions support for MATCH_RECOGNIZE

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1641?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879048#comment-15879048 ] Julian Hyde commented on CALCITE-1641: -- The Oracle documentation is copyright. I don't think you

[jira] [Commented] (CALCITE-1652) Allow GROUPING to have multiple arguments, like GROUPING_ID

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879012#comment-15879012 ] Julian Hyde commented on CALCITE-1652: -- You're right. I was mistaken. GROUPING (and GROUPING_ID)

[jira] [Commented] (CALCITE-1439) Handling errors during constant reduction

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15879010#comment-15879010 ] Julian Hyde commented on CALCITE-1439: -- Yes, let's do that. ReduceExpressionsRule compares the

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878997#comment-15878997 ] Julian Hyde commented on CALCITE-1653: -- [~rusanu], One more thing. The rex package should not

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878987#comment-15878987 ] Julian Hyde commented on CALCITE-1653: -- [~jcamachorodriguez], See my comments on CALCITE-1417 about

[jira] [Created] (CALCITE-1654) Serialized messages are generated even though they are not logged

2017-02-22 Thread Josh Elser (JIRA)
Josh Elser created CALCITE-1654: --- Summary: Serialized messages are generated even though they are not logged Key: CALCITE-1654 URL: https://issues.apache.org/jira/browse/CALCITE-1654 Project: Calcite

[jira] [Assigned] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde reassigned CALCITE-1653: Assignee: Remus Rusanu (was: Julian Hyde) > Allow for custom RelOptPlanner.Executor in

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878939#comment-15878939 ] Julian Hyde commented on CALCITE-1653: -- The change looks basically good. Can you please make the

[jira] [Commented] (CALCITE-1620) CAST('' AS BINARY) is supported by Hive, but not by Calcite

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1620?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878907#comment-15878907 ] Remus Rusanu commented on CALCITE-1620: --- CALCITE-1653 makes this obsolete. Using the Hive executor

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878893#comment-15878893 ] Remus Rusanu commented on CALCITE-1653: --- https://github.com/apache/calcite/pull/379 > Allow for

[jira] [Comment Edited] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878479#comment-15878479 ] Remus Rusanu edited comment on CALCITE-1653 at 2/22/17 5:31 PM: In some

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878802#comment-15878802 ] Remus Rusanu commented on CALCITE-1653: --- Turns out traversing that stack is not hard at all.

[jira] [Commented] (CALCITE-1651) Difference in DATE cast between Hive and Calcite

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878799#comment-15878799 ] Remus Rusanu commented on CALCITE-1651: --- CALCITE-1653 makes this problem irrelevant as using the

[jira] [Commented] (CALCITE-1650) Difference in BOOLEAN cast between Hive and Calcite

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878789#comment-15878789 ] Remus Rusanu commented on CALCITE-1650: --- In any case, CALCITE-1653 makes this a moot point because

[jira] [Commented] (CALCITE-1650) Difference in BOOLEAN cast between Hive and Calcite

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878781#comment-15878781 ] Remus Rusanu commented on CALCITE-1650: --- I think the relevant snippet is 6.13 20) (SQL2011) which

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Jesus Camacho Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878569#comment-15878569 ] Jesus Camacho Rodriguez commented on CALCITE-1653: -- Simplifying cast on constant was a

[jira] [Commented] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Remus Rusanu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15878479#comment-15878479 ] Remus Rusanu commented on CALCITE-1653: --- In some cases the executor has to percolate through quite

[jira] [Created] (CALCITE-1653) Allow for custom RelOptPlanner.Executor in RexUtil.simplify

2017-02-22 Thread Remus Rusanu (JIRA)
Remus Rusanu created CALCITE-1653: - Summary: Allow for custom RelOptPlanner.Executor in RexUtil.simplify Key: CALCITE-1653 URL: https://issues.apache.org/jira/browse/CALCITE-1653 Project: Calcite

[jira] [Commented] (CALCITE-1439) Handling errors during constant reduction

2017-02-22 Thread Jesus Camacho Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15877968#comment-15877968 ] Jesus Camacho Rodriguez commented on CALCITE-1439: -- {quote} The only error that should

[jira] [Updated] (CALCITE-1652) Allow GROUPING to have multiple arguments, like GROUPING_ID

2017-02-22 Thread Jesus Camacho Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1652?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jesus Camacho Rodriguez updated CALCITE-1652: - Description: When initially added to the SQL standard {{GROUPING}} took

[jira] [Commented] (CALCITE-1652) Allow GROUPING to have multiple arguments, like GROUPING_ID

2017-02-22 Thread Jesus Camacho Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15877960#comment-15877960 ] Jesus Camacho Rodriguez commented on CALCITE-1652: -- Changes LGTM [~julianhyde], +1. I