Re: Spatial indexes

I will log a JIRA case for this, and we can continue discussions there. On Jun 28, 2017 10:43 AM, "Jacques Nadeau" wrote: > I don't know how much I can contribute but it sounds like a great way to > expand Calcite. Good idea Julian! > > On Jun 27, 2017 7:06 PM, "Atri Sharma"

Re: Spatial indexes

I don't know how much I can contribute but it sounds like a great way to expand Calcite. Good idea Julian! On Jun 27, 2017 7:06 PM, "Atri Sharma" wrote: > I am all for it. This sounds really interesting. > > On Jun 28, 2017 7:02 AM, "Julian Hyde" wrote: >

Re: Allow Adapter to Participate in the SQL Validation Phase

I'm curious why you want to validate relational expressions (RelNode). Most projects that use Calcite use SQL as the front end, and they validate the SQL parse tree (SqlNode). If the parse tree is valid then the relational algebra will also be valid. But anyway, I could see a use for

Re: Spatial indexes

I am all for it. This sounds really interesting. On Jun 28, 2017 7:02 AM, "Julian Hyde" wrote: > Is anyone looking for a neat project in Calcite that would have a big > impact? I'm thinking that we could add support for spatial indexes to > Calcite in such a way that

Spatial indexes

Is anyone looking for a neat project in Calcite that would have a big impact? I'm thinking that we could add support for spatial indexes to Calcite in such a way that downstream projects such as Phoenix and Flink could easily benefit from it. GIS (Geographic Information Systems, aka Spatial

Re: Rule to eliminate a redundant LogicalSort

Atri, Julian, Thank you very much for your responses. Is there a chance that this rule will make it into the upcoming 1.13.0 release? Mike On Tue, Jun 27, 2017 at 4:17 PM, Julian Hyde wrote: > > Atri Sharma wrote: > > > > but have hit a bug in HepPlanner which Julian is

Re: [JENKINS] [IMPORTANT] - Jenkins Migration and Upgrade (And JDK7 deprecation)

I believe the answer is "maybe" at this point. The phrasing of the alert seems to be saying "jdk7 might not work", but non-committal in that warning. On 6/27/17 4:24 PM, Julian Hyde wrote: Does this mean that we will lose coverage for JDK 7? If so, it shouldn't be a huge problem. I run tests

Re: [JENKINS] [IMPORTANT] - Jenkins Migration and Upgrade (And JDK7 deprecation)

Does this mean that we will lose coverage for JDK 7? If so, it shouldn't be a huge problem. I run tests nightly on my home Linux box and it should detect if anything is amiss. On Tue, Jun 27, 2017 at 8:57 AM, Josh Elser wrote: > tl;dr Infra is upgrading Jenkins in 2 weeks and

Re: call.transformTo not returning infinitely

Yes please file a JIRA. With a test case if you have one. On Tue, Jun 27, 2017 at 7:51 AM, Atri Sharma wrote: > I can help fixing that bug, should I file a JIRA? > > BTW, I will be posting an updated PR for this today , with latest merged > upstream and added tests > > On

Re: Rule to eliminate a redundant LogicalSort

> Atri Sharma wrote: > > but have hit a bug in HepPlanner which Julian is looking into. For the record, I am not actively looking into the bug. > Michael Alexeev wrote: > > Is SortRemoveRule supposed to address this - Planner rule to remove Sort if > its input is already sorted. > > Looking at

Re: PR for Calcite 873

Thanks for the PR! See my review comments in https://issues.apache.org/jira/browse/CALCITE-873. On Tue, Jun 27, 2017 at 11:18 AM, Atri Sharma wrote: > Just noticed that my editor decided to fold the imports automatically. Will > fix that tomorrow > > On Jun 27, 2017 11:33

Re: PR for Calcite 873

Just noticed that my editor decided to fold the imports automatically. Will fix that tomorrow On Jun 27, 2017 11:33 PM, "Atri Sharma" wrote: > Hi All, > > Please find PR for Calcite 873 at: > > https://github.com/apache/calcite/pull/485 > > Please let me know if there are

Fwd: [JENKINS] [IMPORTANT] - Jenkins Migration and Upgrade (And JDK7 deprecation)

tl;dr Infra is upgrading Jenkins in 2 weeks and Java7 Maven jobs may/may-not work after this. See explanation below from [1]: Users with jobs configured with the "Maven project" type may not be able to use Java 7 for their Maven jobs. The correct behavior is not guaranteed so proceed at your

[jira] [Created] (CALCITE-1860) Fix NPE in RexUtil

Ruidong Li created CALCITE-1860: --- Summary: Fix NPE in RexUtil Key: CALCITE-1860 URL: https://issues.apache.org/jira/browse/CALCITE-1860 Project: Calcite Issue Type: Bug Reporter:

[jira] [Created] (CALCITE-1859) NPE in validate method of VolcanoPlanner

Jesus Camacho Rodriguez created CALCITE-1859: Summary: NPE in validate method of VolcanoPlanner Key: CALCITE-1859 URL: https://issues.apache.org/jira/browse/CALCITE-1859 Project: Calcite

Re: Rule to eliminate a redundant LogicalSort

Hi Michael, I am currently working on 873 which address this exact use case. I have the patch ready but have hit a bug in HepPlanner which Julian is looking into. Post that, I should be able to push the JIRA and get this rule in. Will be glad for your review on it On Jun 27, 2017 8:22 PM,