Re: Some questions

2018-02-11 Thread Julian Hyde
Some context, for the benefit of others on the thread. The JIRA case is https://issues.apache.org/jira/browse/CALCITE-2128 , and Jonathan’s branch is https://github.com/shmushkis/calcite/tree/branch-1.15

Re: copy method in RelSubset class

2018-02-11 Thread Julian Hyde
Can you tell me why you want to copy a RelSubset? A RelSubset is an equivalence class - a set of relational expressions that always return the same results. So if you made a copy you’d be creating another equivalent relational expression - that by definition should be in the original RelSubset.

copy method in RelSubset class

2018-02-11 Thread Alessandro Solimando
Hello community, I am adding a SparkAdapter test with the following query: > select * > from *(values (1, 'a'), (2, 'b'), (3, 'b'), (4, 'c'), (2, 'c')) as t(x, y)* > where x between 3 and 4 > > When executed, an exception is thrown (the full stack trace is at the end of the email) in *copy* metho

Re: Some questions

2018-02-11 Thread Jonathan Doron
Hi, Following our thread I have updated my branch according to your(Julian) comments, I haven't touched the failing tests as they have already been fixed by you, If its fine by you please push my branch to calcite. Thanks On Thu, Feb 8, 2018 at 10:54 PM, Julian Hyde wrote: > > Dev list is the