Re: [VOTE] Release apache-calcite-1.19.0 (release candidate 1)

2019-03-18 Thread Stamatis Zampetakis
=false -Pit verify -fn) KO A brief summary of the errors is given below: [ERROR] Tests run: 290, Failures: 1, Errors: 0, Skipped: 21, Time elapsed: 23.703 s <<< FAILURE! - in org.apache.calcite.test.JdbcTest (MySQL) [ERROR] Tests run: 290, Failures: 0, Errors: 1, Skipped: 21, Time elapsed:

Re: [VOTE] Release apache-calcite-1.19.0 (release candidate 1)

2019-03-15 Thread Michael Mior
+1 (binding) checked hashes and signature, compiled and ran tests and a RAT check. -- Michael Mior mm...@apache.org Le ven. 15 mars 2019 à 10:38, Kevin Risden a écrit : > > Hi all, > > I have created a build for Apache Calcite 1.19.0, release candidate 1. > > Thanks t

Re: [VOTE] Release apache-calcite-1.19.0 (release candidate 1)

2019-03-15 Thread Julian Hyde
+1 (binding) Checked hashes, KEYS, LICENSE, NOTICE, README; checked that contents of tar match git; compiled and ran tests using JDK 11 on Ubuntu linux. Verified that maven-wrapper.jar has gone. Julian On Fri, Mar 15, 2019 at 7:38 AM Kevin Risden wrote: > > Hi all, > > I have cre

[VOTE] Release apache-calcite-1.19.0 (release candidate 1)

2019-03-15 Thread Kevin Risden
Hi all, I have created a build for Apache Calcite 1.19.0, release candidate 1. Thanks to everyone who has contributed to this release. Since RC 0, we have fixed the following issues: * [CALCITE-2925] Exclude maven-wrapper.jar from source distribution You can read the release notes here: https

[GitHub] vlsi commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler

2019-02-12 Thread GitBox
vlsi commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler URL: https://github.com/apache/calcite-avatica/pull/85#discussion_r256155266 ## File path: server/src/main/java/org/apache/calcite/avatica/server

[GitHub] vlsi opened a new pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler

2019-02-12 Thread GitBox
vlsi opened a new pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler URL: https://github.com/apache/calcite-avatica/pull/85 I'm pretty confident that this one fixes https://issues.apache.org/jira/browse/CALCITE-2704, and this change should be

[jira] [Created] (CALCITE-2749) org.apache.calcite.sql.parser.SqlParseException: Lexical error at line 1, column 13. Encountered: "`" (96), after : ""

2018-12-19 Thread ambition (JIRA)
ambition created CALCITE-2749: - Summary: org.apache.calcite.sql.parser.SqlParseException: Lexical error at line 1, column 13. Encountered: "`" (96), after : "" Key: CALCITE-2749 URL: https://is

[CANCEL] [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Julian Hyde
at 8:44 AM Zoltan Haindrich wrote: >> >> Hello, >> >> I've run the Hive tests and I've found something which might cause problems: >> in CALCITE-1413 we were extra cautious to not simplify cases like: >> CASE a=0 THEN 0 ELSE 1/a END >> and it worked c

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Julian Hyde
he Hive tests and I've found something which might cause problems: > in CALCITE-1413 we were extra cautious to not simplify cases like: > CASE a=0 THEN 0 ELSE 1/a END > and it worked correctly; however: RexProgramBuilder is also executes simplify > - but the protection at that point

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Zoltan Haindrich
Hello, I've run the Hive tests and I've found something which might cause problems: in CALCITE-1413 we were extra cautious to not simplify cases like: CASE a=0 THEN 0 ELSE 1/a END and it worked correctly; however: RexProgramBuilder is also executes simplify - but the protection at

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Kevin Risden
+1 (binding) checked sig and hashes, compiled and ran tests from git commit and tar.gz. Francis - [ERROR] Failures: [ERROR] OsAdapterTest.testFiles:146 Expected: "type=d\ntype=f" but: was "" I'm pretty sure this happens when Alpine linux doesn't have

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-07 Thread Enrico Olivelli
+1 (non binding) - checked signature and checksum - built from source, all tests pass on OracleJDK8 on Linux - all tests from downstream project (HerdDB) are passing Still great work ! Thank you Julian for driving the release Enrico Il giorno ven 7 dic 2018 alle ore 02:44 Francis Chuang ha

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-06 Thread Francis Chuang
Thanks for making this available for voting, Julian! +1 (binding) Environment: Debian Sid, OpenJDK 11.0.1, maven 3.6.0 in a docker container - Verified GPG signature - OK - Verified SHA256 - OK - Ran "mvn -DskipTests clean install" - OK - Ran "mvn test" - OK I also teste

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-06 Thread Michael Mior
+1 (binding) checked hashes and signature and compiled and ran tests. Thanks for the fixes! -- Michael Mior mm...@apache.org Le jeu. 6 déc. 2018 à 03:19, Julian Hyde a écrit : > OK, let's try again. > > I have created a build for Apache Calcite 1.18.0, release candidate 1.

Re: [VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-06 Thread Stamatis Zampetakis
+1 (non binding) System: Ubuntu 18.04 LTS and jdk1.8.0.66 -run mvn clean install on staged sources and git repo -checked signatures and checksums -went quickly over release note Everything looks good! I have only a small question: should the release note mention new dependencies (e.g

[VOTE] Release apache-calcite-1.18.0 (release candidate 1)

2018-12-06 Thread Julian Hyde
OK, let's try again. I have created a build for Apache Calcite 1.18.0, release candidate 1. Thanks to everyone who has contributed to this release. Since RC 0, we have fixed the following issues: * [CALCITE-2726] ReduceExpressionRule oversimplifies filter conditions containing nulls * [CA

[jira] [Created] (CALCITE-2593) Failed to plan when SQL is like "sum(X + 1) filter (where Y)"

2018-09-25 Thread Hongze Zhang (JIRA)
Hongze Zhang created CALCITE-2593: - Summary: Failed to plan when SQL is like "sum(X + 1) filter (where Y)" Key: CALCITE-2593 URL: https://issues.apache.org/jira/browse/CALCITE-2593 Projec

[jira] [Created] (CALCITE-2581) RexSimplify: and(nullBool, not(or(nullBool, vBool(1)))) fails with assertion error

2018-09-20 Thread Vladimir Sitnikov (JIRA)
Vladimir Sitnikov created CALCITE-2581: -- Summary: RexSimplify: and(nullBool, not(or(nullBool, vBool(1 fails with assertion error Key: CALCITE-2581 URL: https://issues.apache.org/jira/browse/CALCITE-2581

[jira] [Created] (CALCITE-2572) Calcite substring fails with a start position less than 1.

2018-09-19 Thread Andrew Pilloud (JIRA)
Andrew Pilloud created CALCITE-2572: --- Summary: Calcite substring fails with a start position less than 1. Key: CALCITE-2572 URL: https://issues.apache.org/jira/browse/CALCITE-2572 Project: Calcite

[CANCEL][VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Francis Chuang
The vote for Apache Calcite Avatica Go 3.1.0, release candidate 1 has been cancelled. A new release will be tagged as 3.1.0-rc2 and will be available for voting soon. Francis

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Francis Chuang
TICE, copyright should be 2012-2018 not 2012-2017. Due to these issues, my vote is -1. Checked LICENSE, README.md, checksums/signatures, file headers. I did not build or run tests, or check that dependencies have suitable licenses. I don't know enough about Go. Can someone else please do that.

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Julian Hyde
gt; .idea/dataSources.local.xml >> .idea/usage.statistics.xml >> .idea/misc.xml >> .idea/dataSources.xml >> .idea/modules.xml >> >> In NOTICE, copyright should be 2012-2018 not 2012-2017. >> >> Due to these issues, my vote is -1. >> >>

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Francis Chuang
hese issues, my vote is -1. Checked LICENSE, README.md, checksums/signatures, file headers. I did not build or run tests, or check that dependencies have suitable licenses. I don't know enough about Go. Can someone else please do that. Julian On Tue, Sep 4, 2018 at 3:38 PM Francis Chu

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Sergey Nuyanzin
xml > > .idea/dataSources/ed276130-2ebd-463d-b1a0-78e39ca4d29f.xml > > .idea/calcite-avatica-go.iml > > .idea/watcherTasks.xml > > .idea/dataSources.local.xml > > .idea/usage.statistics.xml > > .idea/misc.xml > > .idea/dataSources.xml > > .idea/modules.xml > &g

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Francis Chuang
.xml .idea/usage.statistics.xml .idea/misc.xml .idea/dataSources.xml .idea/modules.xml In NOTICE, copyright should be 2012-2018 not 2012-2017. Due to these issues, my vote is -1. Checked LICENSE, README.md, checksums/signatures, file headers. I did not build or run tests, or check that dependencies hav

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-05 Thread Julian Hyde
.idea/calcite-avatica-go.iml .idea/watcherTasks.xml .idea/dataSources.local.xml .idea/usage.statistics.xml .idea/misc.xml .idea/dataSources.xml .idea/modules.xml In NOTICE, copyright should be 2012-2018 not 2012-2017. Due to these issues, my vote is -1. Checked LICENSE, README.md, checksums

[jira] [Created] (CALCITE-2530) Reasonable behavior of TRIM function when the length of trim character is not 1(one)

2018-09-04 Thread Hongze Zhang (JIRA)
Hongze Zhang created CALCITE-2530: - Summary: Reasonable behavior of TRIM function when the length of trim character is not 1(one) Key: CALCITE-2530 URL: https://issues.apache.org/jira/browse/CALCITE-2530

Re: [VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-09-04 Thread Francis Chuang
/2018 10:35 AM, Francis Chuang wrote: Hi all, I have created a release for Apache Calcite Avatica Go 3.1.0, release candidate 1. The release notes are available here: https://github.com/apache/calcite-avatica-go/blob/master/site/_docs/go_history.md The commit to be voted on: http://git-wip

[VOTE] Release calcite-avatica-go-3.1.0 (release candidate 1)

2018-08-30 Thread Francis Chuang
Hi all, I have created a release for Apache Calcite Avatica Go 3.1.0, release candidate 1. The release notes are available here: https://github.com/apache/calcite-avatica-go/blob/master/site/_docs/go_history.md The commit to be voted on: http://git-wip-us.apache.org/repos/asf/calcite-avatica

[jira] [Created] (CALCITE-2505) RexSimplify: isFalse(isNotDistinctFrom(vBool(0), vBool(1))) causes AssertionError: wrong operand count 1 for IS DISTINCT FROM

2018-08-29 Thread Vladimir Sitnikov (JIRA)
Vladimir Sitnikov created CALCITE-2505: -- Summary: RexSimplify: isFalse(isNotDistinctFrom(vBool(0), vBool(1))) causes AssertionError: wrong operand count 1 for IS DISTINCT FROM Key: CALCITE-2505 URL: https

[RESULT] [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-26 Thread Francis Chuang
Thanks to everyone who has tested the release candidate and given their comments and votes. The tally is as follows. 4 binding +1s: Francis Chuang Josh Elser Julian Hyde Michael Mior 1 non-binding +1s: Kevin Risden No 0s or -1s. Therefore I am delighted to announce that the proposal to

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-24 Thread Francis Chuang
Thanks for the heads up, Julian! I will update the make-release-artifacts script and the documentation. On 25/04/2018 9:01 AM, Julian Hyde wrote: One more thing. The new release policy[1] says "SHOULD NOT provide a MD5-file”. My vote on this RC is still +1. But let’s remove the .md5 file

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-24 Thread Julian Hyde
One more thing. The new release policy[1] says "SHOULD NOT provide a MD5-file”. My vote on this RC is still +1. But let’s remove the .md5 file next release. .sha256 and .asc are all we need. [1] http://www.apache.org/dev/release-distribution#sigs-and-sums <http://www.apache.org/dev

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-24 Thread Michael Mior
+1 (binding) Compiled and ran tests and checked hashes and signature. Thanks Francis! -- Michael Mior mm...@uwaterloo.ca 2018-04-22 20:05 GMT-04:00 Francis Chuang : > Hi all, > > I have created a release for Apache Calcite Avatica Go 3.0.0, release > candidate 1. > > The

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-23 Thread Julian Hyde
+1 (binding) Downloaded, checked checksums (could not check signatures due to gpg problems on my machine), checked headers, LICENSE, NOTICE, README.md. Julian On Mon, Apr 23, 2018 at 7:04 PM, Josh Elser wrote: > +1 (binding) > > * xsums+sigs OK > * KEYS updated and signing key

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-23 Thread Josh Elser
+1 (binding) * xsums+sigs OK * KEYS updated and signing key is published * No binary files in src release * License headers on relevant files * LICENSE is OK * NOTICE is OK Good work, Francis! On 4/22/18 8:05 PM, Francis Chuang wrote: Hi all, I have created a release for Apache Calcite

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-22 Thread Francis Chuang
supported to reduce confusion. +1 for removing 1.7+ since it seems like a promise we can't easily keep. Kevin Risden On Sun, Apr 22, 2018 at 8:42 PM, Francis Chuang wrote: Thanks for the review, Kevin! Please see my response in line. On 23/04/2018 11:18 AM, Kevin Risden wrote: * Ran tes

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-22 Thread Kevin Risden
Thanks for the quick cleanup response. > I will add Go 1.9 to the travis file, but maybe we should just remove the note that Go 1.7+ is supported to reduce confusion. +1 for removing 1.7+ since it seems like a promise we can't easily keep. Kevin Risden On Sun, Apr 22, 2018 at 8:42 PM,

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-22 Thread Francis Chuang
ore - doesn't ignore golang/intellij files - I've copied all IDE files from Calcite's gitignore into avatica-go's gitignore. * goreportcard - https://goreportcard.com/report/github.com/apache/calcite-avatica-go - lists some minor things - These are some minor things we can

Re: [VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-22 Thread Kevin Risden
m/report/github.com/apache/calcite-avatica-go - lists some minor things None of the above are show stoppers for me. +1 (non-binding) Kevin Risden On Sun, Apr 22, 2018 at 7:05 PM, Francis Chuang wrote: > Hi all, > > I have created a release for Apache Calcite Avatica Go 3.0.0, rele

[VOTE] Release calcite-avatica-go-3.0.0 (release candidate 1)

2018-04-22 Thread Francis Chuang
Hi all, I have created a release for Apache Calcite Avatica Go 3.0.0, release candidate 1. The release notes are available here: https://github.com/apache/calcite-avatica-go/blob/master/site/go_history.md The commit to be voted on: http://git-wip-us.apache.org/repos/asf/calcite-avatica-go

[jira] [Created] (CALCITE-2218) AvaticaResultSet#getRow() is not 1-indexed based

2018-03-18 Thread Laurent Goujon (JIRA)
Laurent Goujon created CALCITE-2218: --- Summary: AvaticaResultSet#getRow() is not 1-indexed based Key: CALCITE-2218 URL: https://issues.apache.org/jira/browse/CALCITE-2218 Project: Calcite

Re: Why not optimize the parameters of COUNT (1) in OVER window ?

2018-01-09 Thread jincheng sun
gt; > > On Jan 7, 2018, at 4:00 AM, jincheng sun wrote: > > Hi Julian, > > I'm a bit confused by the different optimization of COUNT (1) in OVER > window and tumble window. > > When we parse a SQL: > > select COUNT(1) from T tumble(rowtime, i

[jira] [Created] (CALCITE-2126) Optimization of COUNT (1) in OVER window

2018-01-09 Thread sunjincheng (JIRA)
sunjincheng created CALCITE-2126: Summary: Optimization of COUNT (1) in OVER window Key: CALCITE-2126 URL: https://issues.apache.org/jira/browse/CALCITE-2126 Project: Calcite Issue Type

Re: Why not optimize the parameters of COUNT (1) in OVER window ?

2018-01-08 Thread Julian Hyde
NULL; we make that optimization in non-windowed AVG, and it is a useful one. Julian > On Jan 7, 2018, at 4:00 AM, jincheng sun wrote: > > Hi Julian, > > I'm a bit confused by the different optimization of COUNT (1) in OVER window > and tumble window. > > Whe

Why not optimize the parameters of COUNT (1) in OVER window ?

2018-01-07 Thread jincheng sun
Hi Julian, I'm a bit confused by the different optimization of COUNT (1) in OVER window and tumble window. When we parse a SQL: select COUNT(1) from T tumble(rowtime, interval 2 seconds) In RexBuild.addAggCall will do the optimize as follows: if(aggCall.getAggregation() insta

[RESULT] [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-09 Thread Julian Hyde
Thanks to everyone who has tested both release candidates and given their comments and votes. The tally is as follows: 3 binding +1s (Aman, Julian, Michael), 1 non-binding +1 (Enrico), no 0s or -1s. Therefore I am delighted to announce that the proposal to release Apache Calcite 1.15.0 has

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-08 Thread Aman Sinha
LGTM. +1 Downloaded the source tar file from [1] on my Linux VM. Untarred and built using openJdk 1.8 and ran unit tests. All tests ran successfully. Also, I did not see the extraneous 'scott.tmp' directory that I saw in RC0. -Aman [1] https://dist.apache.org/repos/dist/dev/calc

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-08 Thread Julian Hyde
Vote ends in 12 hours. Please download the release and vote! > On Dec 7, 2017, at 9:45 AM, Michael Mior wrote: > > +1 (binding) > > Verified signature and hashes of the tarball and compiled and ran tests. > Thanks for the fixes Julian! > > -- > Michael Mior > mm

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-07 Thread Michael Mior
+1 (binding) Verified signature and hashes of the tarball and compiled and ran tests. Thanks for the fixes Julian! -- Michael Mior mm...@apache.org 2017-12-06 2:34 GMT-05:00 Julian Hyde : > Hi all, > > I have created a build for Apache Calcite 1.15.0, release candidate 1. > > Ch

Re: [VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-06 Thread Enrico Olivelli
+1 (non binding) Checked source tar ball, notes. - mvn clean install, build OK - All tests pass except from one (OsAdapterTest), see the below, not a blocker, it depends on my SO (Linux Fedora 25) - All tests passes on HerdDB Enrico Tests run: 12, Failures: 0, Errors: 1, Skipped: 2, Time

[VOTE] Release apache-calcite-1.15.0 (release candidate 1)

2017-12-05 Thread Julian Hyde
Hi all, I have created a build for Apache Calcite 1.15.0, release candidate 1. Changes since the previous release candidate: fixed the show-stopper CALCITE-2074, fixed the two minor issues raised in the last vote (CALCITE-2070 and a doc fix to watch out for scott.tmp), and also fixed CALCITE

[jira] [Created] (CALCITE-1939) DatabaseMetaData.getColumns() returns -1 for columns COLUMN_SIZE , BUFFER_LENGTH and DECIMAL_DIGITS

2017-08-11 Thread Sergio Sainz (JIRA)
Sergio Sainz created CALCITE-1939: - Summary: DatabaseMetaData.getColumns() returns -1 for columns COLUMN_SIZE , BUFFER_LENGTH and DECIMAL_DIGITS Key: CALCITE-1939 URL: https://issues.apache.org/jira/browse

[GitHub] calcite-avatica-go pull request #1: WIP: Initial import of the avatica-go cl...

2017-08-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica-go/pull/1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] calcite-avatica-go issue #1: WIP: Initial import of the avatica-go client

2017-08-09 Thread F21
Github user F21 commented on the issue: https://github.com/apache/calcite-avatica-go/pull/1 @julianhyde I've added the headers. Please take a look. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project doe

[GitHub] calcite-avatica-go issue #1: WIP: Initial import of the avatica-go client

2017-08-09 Thread julianhyde
Github user julianhyde commented on the issue: https://github.com/apache/calcite-avatica-go/pull/1 If you add Apache headers to all files, I think this change is good to go. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] calcite-avatica-go issue #1: WIP: Initial import of the avatica-go client

2017-08-09 Thread julianhyde
Github user julianhyde commented on the issue: https://github.com/apache/calcite-avatica-go/pull/1 By "release history" I mean something like https://github.com/apache/calcite-avatica/blob/master/site/_docs/history.md, which is updated on each release and beco

[GitHub] calcite-avatica-go pull request #1: WIP: Initial import of the avatica-go cl...

2017-08-08 Thread F21
GitHub user F21 opened a pull request: https://github.com/apache/calcite-avatica-go/pull/1 WIP: Initial import of the avatica-go client cc @joshelser @julianhyde Todo: - [x] Switch imports from github.com/Boostport/avatica to github.com/apache/calcite-avatica-go

[jira] [Created] (CALCITE-1907) Table function with 1 column gives ClassCastException

2017-07-27 Thread Julian Hyde (JIRA)
Julian Hyde created CALCITE-1907: Summary: Table function with 1 column gives ClassCastException Key: CALCITE-1907 URL: https://issues.apache.org/jira/browse/CALCITE-1907 Project: Calcite

[jira] [Created] (CALCITE-1865) select count(1) from (select sum(1) .... without group by failed

2017-06-29 Thread zhou degao (JIRA)
zhou degao created CALCITE-1865: --- Summary: select count(1) from (select sum(1) without group by failed Key: CALCITE-1865 URL: https://issues.apache.org/jira/browse/CALCITE-1865 Project

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-12 Thread Shaoxuan Wang
input table. Hence, the resulting table is a dynamic table as well." Regards, Shaoxuan On Sat, May 13, 2017 at 1:05 AM, Tyler Akidau wrote: > Being able to support an EMIT config independent of the query itself sounds > great for compatible use cases (which should be many :-). >

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-12 Thread Tyler Akidau
the name is to simply to imply it can evolve and change over time? -Tyler On Fri, May 12, 2017 at 1:59 AM Shaoxuan Wang wrote: > Thanks to Tyler and Fabian for sharing your thoughts. > > Regarding to the early/late update control of FLINK. IMO, each dynamic > table can have an EMIT

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-12 Thread Shaoxuan Wang
ort for group window (TUMBLE, HOP, SESSION), > > OVER > > > RANGE/ROW window (without FOLLOWING), and non-windowed GROUP BY > > > aggregations. The group windows are triggered by watermark and the over > > > window and non-windowed aggregations emit for each input rec

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-12 Thread Fabian Hueske
ndowed GROUP BY > > aggregations. The group windows are triggered by watermark and the over > > window and non-windowed aggregations emit for each input record > > (AtCount(1)). The window aggregations do not support early or late firing > > (late records are dropped), so no upd

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-10 Thread Tyler Akidau
the upcoming 1.3.0 > release, we will have support for group window (TUMBLE, HOP, SESSION), OVER > RANGE/ROW window (without FOLLOWING), and non-windowed GROUP BY > aggregations. The group windows are triggered by watermark and the over > window and non-windowed aggregations emit for

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-09 Thread Fabian Hueske
BY aggregations. The group windows are triggered by watermark and the over window and non-windowed aggregations emit for each input record (AtCount(1)). The window aggregations do not support early or late firing (late records are dropped), so no updates here. However, the non-windowed aggregates

Re: Towards a spec for robust streaming SQL, Part 1

2017-05-08 Thread Tyler Akidau
> >> > On Fri, Apr 21, 2017 at 2:26 PM Kenneth Knowles > > >> > wrote: >> > >> > > There's something to be said about having different triggering >> depending >> > on >> > > which side of a join data comes from, perhaps? >> > >

[jira] [Created] (CALCITE-1775) Druid adapter: "GROUP BY ()" on empty relation should return 1 row

2017-05-05 Thread Julian Hyde (JIRA)
Julian Hyde created CALCITE-1775: Summary: Druid adapter: "GROUP BY ()" on empty relation should return 1 row Key: CALCITE-1775 URL: https://issues.apache.org/jira/browse/CALCITE-1775

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-26 Thread Tyler Akidau
Apr 21, 2017 at 2:26 PM Kenneth Knowles > > wrote: > > > > > There's something to be said about having different triggering > depending > > on > > > which side of a join data comes from, perhaps? > > > > > > (delightful doc, as usual) > >

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-25 Thread Fabian Hueske
; -Tyler > > On Fri, Apr 21, 2017 at 2:26 PM Kenneth Knowles > wrote: > > > There's something to be said about having different triggering depending > on > > which side of a join data comes from, perhaps? > > > > (delightful doc, as usual) > > &g

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-21 Thread Tyler Akidau
s usual) > > Kenn > > On Fri, Apr 21, 2017 at 1:33 PM, Tyler Akidau > wrote: > > > Thanks for reading, Luke. The simple answer is that CoGBK is basically > > flatten + GBK. Flatten is a non-grouping operation that merges the input > > streams into a single output st

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-21 Thread Kenneth Knowles
There's something to be said about having different triggering depending on which side of a join data comes from, perhaps? (delightful doc, as usual) Kenn On Fri, Apr 21, 2017 at 1:33 PM, Tyler Akidau wrote: > Thanks for reading, Luke. The simple answer is that CoGBK is basically &

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-21 Thread Tyler Akidau
drive forward agreement on a general streaming SQL > > spec between our three communities (well, technically I volunteered to do > > that w/ Beam and Calcite, but I figured you Flink folks might want to > join > > in since you're going that direction already anyway and will h

Re: Towards a spec for robust streaming SQL, Part 1

2017-04-21 Thread Lukasz Cwik
technically I volunteered to do > that w/ Beam and Calcite, but I figured you Flink folks might want to join > in since you're going that direction already anyway and will have useful > insights :-). > > My plan was to do this by sharing two docs: > >1. The Beam

Towards a spec for robust streaming SQL, Part 1

2017-04-20 Thread Tyler Akidau
an was to do this by sharing two docs: 1. The Beam Model : Streams & Tables - This one is for context, and really only mentions SQL in passing. But it describes the relationship between the Beam Model and the "streams & tables" way of thinking, which turns out to be useful

[GitHub] calcite-avatica pull request #1: CALCITE-1717 Add Avatica and remove the res...

2017-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/calcite-avatica/pull/1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] calcite-avatica pull request #1: CALCITE-1717 Add Avatica and remove the res...

2017-03-31 Thread julianhyde
Github user julianhyde commented on a diff in the pull request: https://github.com/apache/calcite-avatica/pull/1#discussion_r109268544 --- Diff: .travis.yml --- @@ -28,11 +28,11 @@ branches: - /^branch-.*$/ - /^[0-9]+-.*$/ install: - - cd avatica &&

Re: [GitHub] calcite-avatica issue #1: CALCITE-1717 Add Avatica and remove the rest of Ca...

2017-03-31 Thread Julian Hyde
one through, by the way. Julian > On Mar 31, 2017, at 5:15 PM, julianhyde wrote: > > Github user julianhyde commented on the issue: > >https://github.com/apache/calcite-avatica/pull/1 > >Builds fine. Tests pass. Intellij loads up fine. > >* In pom

[GitHub] calcite-avatica issue #1: CALCITE-1717 Add Avatica and remove the rest of Ca...

2017-03-31 Thread julianhyde
Github user julianhyde commented on the issue: https://github.com/apache/calcite-avatica/pull/1 Builds fine. Tests pass. Intellij loads up fine. * In pom.xml, change connection and developerConnection to scm:git:https://git-wip-us.apache.org/repos/asf/calcite.git - otherwise

[GitHub] calcite-avatica pull request #1: CALCITE-1717 Add Avatica and remove the res...

2017-03-31 Thread joshelser
GitHub user joshelser opened a pull request: https://github.com/apache/calcite-avatica/pull/1 CALCITE-1717 Add Avatica and remove the rest of Calcite You can merge this pull request into a Git repository by running: $ git pull https://github.com/joshelser/calcite-avatica

[RESULT] [VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-23 Thread Julian Hyde
Thanks to everyone who has tested the release candidate and given their comments and votes. The tally is as follows. 4 binding +1s: Aman Jesus Josh Julian 1 non-binding +1s: Michael No 0s or -1s. Therefore I am delighted to announce that the proposal to release Apache Calcite 1.12.0 has

Re: [VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-23 Thread Aman Sinha
+1 Downloaded the source tarball to my Linux VM and built, ran unit tests. Encountered 1 failure in PigRelBuilderStyleTest.testImplWithGroupByCountDistinct which is similar to Calcite-1561 but re-running within the suite succeeded, so I am inclined to go with +1. All other tests passed

Re: [VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-23 Thread Jesus Camacho Rodriguez
+1 (binding) Downloaded the tarball, compiled, ran testsuite. Integrated with Hive + HIVE-15708 and ran some q tests. -- Jesús On 3/21/17, 10:37 PM, "Josh Elser" wrote: >+1 (binding) > >* Re-checked sigs/xsums >* Verified changes made over rc0 > >Julian Hyde wr

Re: [VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-21 Thread Josh Elser
+1 (binding) * Re-checked sigs/xsums * Verified changes made over rc0 Julian Hyde wrote: Hi all, I have created a build for Apache Calcite 1.12.0, release candidate 1. The only changes since release candidate 0 are: * fixes to issues CALCITE-1715 and CALCITE-1716, * corresponding

Re: [VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-21 Thread Michael Mior
+1 (non-binding). Checked signature and hash of the tarball, compiled and ran tests. Cassandra integration tests pass this time. -- Michael Mior michael.m...@gmail.com 2017-03-20 22:33 GMT-04:00 Julian Hyde : > Hi all, > > I have created a build for Apache Calcite 1.12.0, release ca

[VOTE] Release apache-calcite-1.12.0 (release candidate 1)

2017-03-20 Thread Julian Hyde
Hi all, I have created a build for Apache Calcite 1.12.0, release candidate 1. The only changes since release candidate 0 are: * fixes to issues CALCITE-1715 and CALCITE-1716, * corresponding modifications to the release history, * some unrelated changes on in the avatica sub-project. Thanks

Re: calcite regress 824d7e7 origin/master jdk8 status: 1 "[CALCITE-1628] Add an alternative match pattern for SemiJoinRule"

2017-02-10 Thread Maryann Xue
. Therefore you need to implement the ‘test’ method. I recommend > that you make your Predicate class extend PredicateImpl. > > You can test using ‘mvn clean site -Dguava.version=21.0’. > > Julian > > > Begin forwarded message: > > *Subject: **calcite regress 824d7e7 ori

Fwd: calcite regress 824d7e7 origin/master jdk8 status: 1 "[CALCITE-1628] Add an alternative match pattern for SemiJoinRule"

2017-02-10 Thread Julian Hyde
using ‘mvn clean site -Dguava.version=21.0’. Julian > Begin forwarded message: > > Subject: calcite regress 824d7e7 origin/master jdk8 status: 1 "[CALCITE-1628] > Add an alternative match pattern for SemiJoinRule" > Date: February 10, 2017 at 3:11:25 AM PST > >

Re: calcite regress a07c742 origin/master jdk9 status: 1 "[CALCITE-1619] CAST is ignored by rules pushing operators into DruidQuery"

2017-02-08 Thread Jesus Camacho Rodriguez
bject: Re: calcite regress a07c742 origin/master jdk9 status: 1 "[CALCITE-1619] CAST is ignored by rules pushing operators into DruidQuery" Jesus, I can check in a fix for this with some other PRs. Since it’s late for you, I assume you’re not working on it, but let me know if you are

Re: calcite regress a07c742 origin/master jdk9 status: 1 "[CALCITE-1619] CAST is ignored by rules pushing operators into DruidQuery"

2017-02-08 Thread Julian Hyde
error. It shows up on JDK > 8 and 9, not sure about 7. Please run “mvn site” under JDK 8 or 9 before > committing. > > Julian > > >> Begin forwarded message: >> >> Subject: calcite regress a07c742 origin/master jdk9 status: 1 >> "[CALCITE-16

Fwd: calcite regress a07c742 origin/master jdk9 status: 1 "[CALCITE-1619] CAST is ignored by rules pushing operators into DruidQuery"

2017-02-08 Thread Julian Hyde
Jesus, Your recent commit broke the build due to a javadoc error. It shows up on JDK 8 and 9, not sure about 7. Please run “mvn site” under JDK 8 or 9 before committing. Julian > Begin forwarded message: > > Subject: calcite regress a07c742 origin/master jdk9 status: 1 "

[jira] [Created] (CALCITE-1552) Add RAND function, returning DOUBLE values in the range 0..1

2016-12-21 Thread Julian Hyde (JIRA)
Julian Hyde created CALCITE-1552: Summary: Add RAND function, returning DOUBLE values in the range 0..1 Key: CALCITE-1552 URL: https://issues.apache.org/jira/browse/CALCITE-1552 Project: Calcite

[RESULT][VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-30 Thread Julian Hyde
Thanks to everyone who has tested the release candidate and given their comments and votes. The vote passes with 4 binding +1 votes (Julian, Alan, James, Josh) and 1 non-binding +1 (Aaron). I am therefore delighted to announce that the proposal to release Apache Calcite 1.9.0 has passed. I’ll

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-30 Thread Julian Hyde
Josh’s patch did the trick (with a couple of other changes to keep maven happy with respect to unused dependencies), so we’re good. My +1 remains a +1. And thanks, Josh, for finding a solution. I’ll send an official result. Julian > On Oct 30, 2016, at 5:07 PM, Julian Hyde wrote: >

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-30 Thread Julian Hyde
due to Avatica fixing a > long-standing dependency-bundling issue (in its shaded jar). Calcite was > expecting to use Jackson (non-relocated) with the Avatica shaded jar which is > now correctly relocating Jackson because it bundles it. > > I tried to write up a better description o

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-30 Thread Josh Elser
ed to write up a better description on JIRA [1] I believe the proper fix is to switch Calcite to using org.apache.calcite.avatica:avatica-core instead of org.apache.calcite.avatica:avatica. If Calcite wants to create shaded artifacts for downstream consumption, we should be creating custom s

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-30 Thread James Taylor
is a showstopper. I am > going to let the vote hang in limbo until I hear some other opinions > about this issue. > > On Fri, Oct 28, 2016 at 3:54 PM, James Taylor > wrote: > > +1. Validated that all tests pass in Phoenix with Avatica 1.9.0 RC. > > > > On Fri, Oct 2

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-29 Thread Julian Hyde
, 2016 at 3:54 PM, James Taylor wrote: > +1. Validated that all tests pass in Phoenix with Avatica 1.9.0 RC. > > On Fri, Oct 28, 2016 at 1:45 PM, Alan Gates wrote: > >> +1. Did a build, checked the LICENSE and NOTICE files, checked the >> signatures. >> >> Ala

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-28 Thread James Taylor
+1. Validated that all tests pass in Phoenix with Avatica 1.9.0 RC. On Fri, Oct 28, 2016 at 1:45 PM, Alan Gates wrote: > +1. Did a build, checked the LICENSE and NOTICE files, checked the > signatures. > > Alan. > > > On Oct 28, 2016, at 11:05, Julian Hyde wrote: > &g

Re: [VOTE] Release apache-calcite-avatica-1.9.0 (release candidate 1)

2016-10-28 Thread Alan Gates
+1. Did a build, checked the LICENSE and NOTICE files, checked the signatures. Alan. > On Oct 28, 2016, at 11:05, Julian Hyde wrote: > > We need one more PMC vote. Vote ends in 30 hours. But you probably don’t want > to vote on a weekend. :) > > Julian > > >&g

<    1   2   3   4   >