[jira] [Created] (CALCITE-4633) Enable RelFieldTrimmer to trim fields on a distinct UNION

2021-06-02 Thread Sean Broeder (Jira)
Sean Broeder created CALCITE-4633: - Summary: Enable RelFieldTrimmer to trim fields on a distinct UNION Key: CALCITE-4633 URL: https://issues.apache.org/jira/browse/CALCITE-4633 Project: Calcite

[jira] [Created] (CALCITE-4632) Exception in RelToSqlConverter: "Cannot convert x to DECIMAL(n, m) due to overflow"

2021-06-02 Thread Steven Talbot (Jira)
Steven Talbot created CALCITE-4632: -- Summary: Exception in RelToSqlConverter: "Cannot convert x to DECIMAL(n, m) due to overflow" Key: CALCITE-4632 URL: https://issues.apache.org/jira/browse/CALCITE-4632

Re: RelFieldTrimmer on union distinct

2021-06-02 Thread Julian Hyde
I ran those queries against Calcite. Calcite gives the correct result. RelFieldTrimmer does the right thing. I was explaining, in simple terms, what would break if you made your proposed change to RelFieldTrimmer. > On Jun 2, 2021, at 3:12 PM, Sean Broeder wrote: > > Thank you Julian. > >

Re: Minified javascript in source releases

2021-06-02 Thread Francis Chuang
+1 for removing those 2 scripts. They are for compatibility with very old browsers that have been discontinued for many years, so it makes sense to reduce the bloat. Francis On 3/06/2021 5:25 am, Julian Hyde wrote: +0 to remove them entirely, if that is possible. -1 to keep in git but

Re: RelFieldTrimmer on union distinct

2021-06-02 Thread Sean Broeder
Thank you Julian. I am able to reproduce your findings. I am curious if the results highlight a bug in Calcite where we should expect 4 to be returned or is this a query that Calcite should not support? Sean > On Jun 2, 2021, at 11:37 AM, Julian Hyde wrote: > > Here’s a query that would

Re: Deduplicate correlate variables question.

2021-06-02 Thread James Starr
If it is de facto deprecated, then could it not just be deprecated with a comment stating why it has not been deleted? Calcite seems to have several API that are defacto deprecated and there is no way of knowing by reading the code. This would personally have made me much simpler for myself to

Re: Minified javascript in source releases

2021-06-02 Thread Julian Hyde
+0 to remove them entirely, if that is possible. -1 to keep in git but remove from source releases. We are already in compliance with Apache policy, best I can tell. On Wed, Jun 2, 2021 at 12:08 PM Haisheng Yuan wrote: > > +1 for removing them entirely. > > Thanks, > Haisheng Yuan > > On

Re: Minified javascript in source releases

2021-06-02 Thread Haisheng Yuan
+1 for removing them entirely. Thanks, Haisheng Yuan On 2021/06/02 13:36:05, Alessandro Solimando wrote: > Hi all, > +1 for removing them as well, all the mentioned sw versions in [5,6] > are extremely, extremely old. > > Best regards, > Alessandro > > On Wed, 2 Jun 2021 at 14:38, Michael

Re: Deduplicate correlate variables question.

2021-06-02 Thread Julian Hyde
Master is a moving target. Apparently you are using a version of the code from sometime in March. These links work better: [1] https://github.com/apache/calcite/blob/796675c9b33e0461bc45a72780162d474a4b098b/core/src/main/java/org/apache/calcite/sql2rel/SqlToRelConverter.java#L2881 [2]

Re: [VOTE] Release apache-calcite-1.27.0 (release candidate 0)

2021-06-02 Thread Igor Lozynskyi
Thanks Stamatis for taking care of this release. +1 - Calcite tests locally (macOS Catalina 10.15.7 + JVM 11.0.8 (AdoptOpenJDK 11.0.8+10) + Gradle 6.8.3): ok - Unit and integration tests of the downstream project using Calcite 1.27.0 RC0 : ok Igor On 31 May 2021, 18:29 +0300,

Re: RelFieldTrimmer on union distinct

2021-06-02 Thread Julian Hyde
Here’s a query that would give the wrong answer if you trim: select count(*) from ( select deptno from scott.emp union select deptno from scott.dept); returns 4. Note that ‘deptno’ is not used. But when I trim it away, select count(*) from ( select 'a' from scott.emp

[jira] [Created] (CALCITE-4631) sql->rel->sql bug with unparsing timestampdiff

2021-06-02 Thread Steven Talbot (Jira)
Steven Talbot created CALCITE-4631: -- Summary: sql->rel->sql bug with unparsing timestampdiff Key: CALCITE-4631 URL: https://issues.apache.org/jira/browse/CALCITE-4631 Project: Calcite Issue

[jira] [Created] (CALCITE-4630) Wrong logical plan for INNER JOIN with subquery

2021-06-02 Thread Dmitry Sysolyatin (Jira)
Dmitry Sysolyatin created CALCITE-4630: -- Summary: Wrong logical plan for INNER JOIN with subquery Key: CALCITE-4630 URL: https://issues.apache.org/jira/browse/CALCITE-4630 Project: Calcite

Re: Minified javascript in source releases

2021-06-02 Thread Alessandro Solimando
Hi all, +1 for removing them as well, all the mentioned sw versions in [5,6] are extremely, extremely old. Best regards, Alessandro On Wed, 2 Jun 2021 at 14:38, Michael Mior wrote: > > +1 for removing these entirely. I don't believe they are currently necessary. > -- > Michael Mior >

RelFieldTrimmer on union distinct

2021-06-02 Thread Sean Broeder
Currently the RefFieldTrimmer only trims on a UNION ALL operation. I've been experimenting to see if it is also possible to trim on UNION DISTINCT. Is there a simple query that demonstrates why this is not possible? Thanks, Sean

Re: Minified javascript in source releases

2021-06-02 Thread Michael Mior
+1 for removing these entirely. I don't believe they are currently necessary. -- Michael Mior mm...@apache.org Le mer. 2 juin 2021 à 06:13, Stamatis Zampetakis a écrit : > > Hi all, > > In the recent votes for Calcite [1] and Avatica [2] there were some -1 > votes regarding minified javascript

Minified javascript in source releases

2021-06-02 Thread Stamatis Zampetakis
Hi all, In the recent votes for Calcite [1] and Avatica [2] there were some -1 votes regarding minified javascript files present in the source releases. The concerns are raised for the following files: site/js/html5shiv.min.js site/js/respond.min.js >From discussions in other lists it seems

Re: [VOTE] Release apache-calcite-1.27.0 (release candidate 0)

2021-06-02 Thread Ruben Q L
Thanks Stamatis for taking care of this release. +1 (binding) - Signature: ok - Checksum: ok - Tests locally (Windows10 + JDK 1.8.0_212 + Gradle 6.8.3): ok - Calcite-based application test suite: ok - Release notes: ok (apart from the minor date issue mentioned by Alessandro) Ruben On Wed, Jun