Re: [DISCUSS] Parser with involved tables

2022-04-13 Thread Alessandro Solimando
Hi Yanjing, I found a discussion in the ML suggesting to use RelOptUtil#findAllTables to get all tables involved in a query: https://lists.apache.org/thread/wcfv2bhdytwsoyc3qlsnxcd7nmnnlb06 Can you check if that's what you want? Best regards, Alessandro On Wed, 13 Apr 2022 at 09:33, Yanjing Wan

Re: Draft: board report for 2022 Q1

2022-04-01 Thread Alessandro Solimando
ly 7:3. > > Community changes, past quarter: > - No new PMC members. Last addition was Ruben Q L on 2020-08-10. > - No new committers. Last addition was Alessandro Solimando on 2021-12-17. > > ## Project Activity: > Calcite 1.30.0 was released on 2022-03-20, it contains contri

Re: Calcite jars with shaded guava

2022-03-29 Thread Alessandro Solimando
Hello Abhishek, maybe I am re-stating what Stamatis suggested without realising, but since Calcite works well with a broad range of guava versions, can't you simply exclude guava when you include it in druid? At that point if the guava version of druid is in the list of those supported by Calcite

Re: [DISCUSS] Best practice for synchronizing master and site branches

2022-03-29 Thread Alessandro Solimando
Hello everyone, I totally agree on automating the website publication and having a single branch, the less we do manually, the lower the chances to mess something up. I am also in favour of versioned docs in the website, it's confusing to land on updated pages from an older context like a message

Re: A basic, DB-agnostic question about the concept of a "Physical Plan"

2022-03-23 Thread Alessandro Solimando
Hi Gavin, in a nutshell, a logical plan consists of logical operators (say, a join), which can be implemented in several ways at the physical level (say, merge join, hash join, etc.), and are therefore associated with some corresponding physical operators. How the logical vs physical planning is p

Re: [VOTE] Release Apache Calcite 1.30.0 (release candidate 3)

2022-03-16 Thread Alessandro Solimando
Hello everyone, thanks a lot Liya Fan for all the hard work on the release and for battle-testing our process documentation which is very important. My vote is +1 (non-binding), tested as follows: - verified gpg signature: OK $ curl "https://downloads.apache.org/calcite/KEYS"; | gpg --import ...

Travis CI is broken

2022-03-08 Thread Alessandro Solimando
Hello everyone, I have noticed that openjdk15-17 are broken on Travis since yesterday (e.g., https://github.com/apache/calcite/runs/5463494141), after digging a bit I found it's a Travis issue, you can track it here: https://travis-ci.community/t/update-install-jdk-sh-with-the-new-revision/12761

[jira] [Created] (CALCITE-5011) CassandraAdapterDataTypesTest fails with initialization error

2022-02-15 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-5011: - Summary: CassandraAdapterDataTypesTest fails with initialization error Key: CALCITE-5011 URL: https://issues.apache.org/jira/browse/CALCITE-5011

Re: Allow Cascades driver invoking "derive" on the nodes produced by "passThrough"

2022-02-11 Thread Alessandro Solimando
Hello everyone, @Vladimir, +1 on the change introducing "enforceDerive()". @Roman, could you walk us through the limitations you found that forced you to copy-paste the whole class? Maybe there is some middle ground for your problem(s) too, similar in spirit to what Vladimir proposed for the othe

Re: Invalid IntervalSQLType

2022-02-10 Thread Alessandro Solimando
I agree with Stamatis, and as always adding a unit test and using git bisect can help spot the exact commit that introduced the change. HTH, Alessandro On Thu, 10 Feb 2022 at 12:37, Stamatis Zampetakis wrote: > Hi Chathura, > > It is difficult to reason about correctness without having the actu

Re: [DISCUSS] New community section for articles/blogs/papers

2022-01-24 Thread Alessandro Solimando
+1 on the proposal, I share Ruben's POV also, but I think we can revise the content from time to time and deprecate/remove links when they get stale. Best regards, Alessandro On Mon, 24 Jan 2022 at 16:29, Zhe Hu wrote: > It’s definitely a terrific proposal, especially for those new comers and >

Re: New rule for Converting UNION ALL with same inputs but different filters to single input with OR FILTER

2022-01-23 Thread Alessandro Solimando
o this > optimization though, since it requires the tables of the union to be > identical rather than different tables representing unions of different > ontologies, that is, this will only apply optimizations when an ontology is > used more than once in a union. Unless I am mistaken. >

Re: New rule for Converting UNION ALL with same inputs but different filters to single input with OR FILTER

2022-01-23 Thread Alessandro Solimando
Hello everyone, SQL queries automatically generated by semantic queries (see [1] for instance) are pretty generous on UNIONs, so I'd say that in this area, you will have many practical cases matching the patterns highlighted here. The main idea is that when you have a concept hierarchy in your ont

[jira] [Created] (CALCITE-4991) Improve RuleLogger to also print input rels in FULL_PLAN mode

2022-01-21 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4991: - Summary: Improve RuleLogger to also print input rels in FULL_PLAN mode Key: CALCITE-4991 URL: https://issues.apache.org/jira/browse/CALCITE-4991

Re: [ANNOUNCE] New Calcite PMC chair: Ruben Q L

2022-01-20 Thread Alessandro Solimando
Congratulations to Ruben and thanks a lot to Haisheng! On Thu, 20 Jan 2022 at 08:13, 953396112 <13282155...@qq.com.invalid> wrote: > Congratulations to Ruben!   > Thanks for serving as Chair, Haisheng! > > > Best regards, > Zhaohui Xu > > > > > -- 原始邮件 -- > 发件人: >

Re: Apache Calcite - Understanding RelNode vs RexNode

2022-01-12 Thread Alessandro Solimando
Hi all, I agree with what Jacques said, in addition to that, what helps me is to observe that RelNode objects can be seen as what "builds" the plan/DAG itself, via the "input" field(s). For instance, a project will have another RelNode as it's input, the same for a join who has two inputs (left an

Re: StackOverflowError in HepPlanner

2022-01-05 Thread Alessandro Solimando
Hi Jeremy, if you manage to repro as a unit test in Calcite (as Stamatis suggested) you can "git bisect" on that specific test to find the right commit. Best regards, Alessandro On Wed, 5 Jan 2022 at 17:12, Jeremy Dyer wrote: > I just tried removing the "S7" CASE WHEN portion and now it works.

Re: [DISCUSS] Trolls and community

2021-12-23 Thread Alessandro Solimando
Dear Calciters, it is indeed a hard conversation to reply to, but I really like this community, and I feel that the largest possible participation to this discussion is needed, I strongly encourage others to chime in as well, even if they are mostly readers of the ML. I think that Jacques summaris

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

2021-12-22 Thread Alessandro Solimando
Hello everyone, thank you Rui for all the work as a release manager! +1 (non-binding): - checked signature and checksum: OK - ./gradlew build (copied gradlew and gradle folder from "master"): OK - visual inspection of uploaded artifacts: OK - release notes: OK (but I agree with the observations m

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-19 Thread Alessandro Solimando
ockers at this moment. > > Hi Alessandro, > > I will wait for another 24 hours before blocking the main branch. It should > be enough time so please feel free to merge PR once there is no objection. > > -Rui > > On Sat, Dec 18, 2021 at 11:45 AM Alessandro Solimando < >

Re: [Calcite Release] Preparing for 1.29.0 Release

2021-12-18 Thread Alessandro Solimando
Hi everyone, I'd like to merge https://github.com/apache/calcite/pull/2628 ( https://issues.apache.org/jira/browse/CALCITE-4917) which is ready if there are no objections. Since it's just adding an extra test it's not urgent at all, so if it causes any troubles it can be postponed after the releas

Re: [ANNOUNCE] New committer: Alessandro Solimando

2021-12-18 Thread Alessandro Solimando
pache Hive (which extensively uses Apache Calcite). Best regards, Alessandro On Sat, 18 Dec 2021 at 15:08, Stamatis Zampetakis wrote: > Apache Calcite's Project Management Committee (PMC) has invited Alessandro > Solimando to become a committer, and we are pleased to announce t

Re: RelNode semantic evolution vs adapter implementations

2021-12-13 Thread Alessandro Solimando
Hi, what you mention really sounds like breaking changes, so they should be treated as such and be explicitly mentioned in the release notes. Whenever possible, these behaviours should also be covered by unit tests, in order to limit the chance of changes sneaking in silently. Concerning the vers

Re: [VOTE] Release apache-calcite-avatica-1.20.0 (release candidate 0) (second vote)

2021-12-13 Thread Alessandro Solimando
Hi everyone, my vote is +1 (non-binding) after checking the RC as follows: - Checked hash and signature: OK - Checked release notes: OK - Checked the release tag with right commit: OK - Checking the tag and the tar.gz content: OK diff -qr apache-calcite-avatica-1.20.0-src ~/git/calcite-avatica:

Re: how to represent and optimize a query over partitioned storage

2021-12-13 Thread Alessandro Solimando
; interpreter). > > > > Sorry I don't have a great pointer to a high-quality selectivity > estimator > > in OSS that uses more advanced stats. Maybe someone else can point to > one. > > > > On Sat, Dec 11, 2021 at 11:43 PM Alessandro Solimando < > &

Re: how to represent and optimize a query over partitioned storage

2021-12-11 Thread Alessandro Solimando
Do you have any code pointer for achieving that, Jacques? My main concern is how to estimate the new cost. Do you leverage the estimation of predicate selectivity over the partitioning expression maybe? Il Dom 12 Dic 2021, 05:48 Jacques Nadeau ha scritto: > What we have done in the past is push

Re: Druid test hangs?

2021-12-11 Thread Alessandro Solimando
I started noticing that intermittently around a month ago, I had a quick look back then but I could not pinpoint the root cause. I don't think it is expected, and I guess it comes from test infra setup rather than the Calcite code itself. Il Dom 12 Dic 2021, 05:43 Jacques Nadeau ha scritto: > I

Re: Enable "squash and merge" button in GitHub UI

2021-12-07 Thread Alessandro Solimando
In Spark they have bots that understand commands like "test"/"retest", and it helps IMO, in absence of a UI button. So I think it would be great to have a "/squash" command à la IRC :) Best regards, Alessandro On Mon, 6 Dec 2021 at 22:12, Vladimir Sitnikov wrote: > I wonder if we can add GitHu

[jira] [Created] (CALCITE-4917) Add test for 'IS NOT NULL(a) AND a=b' simplifies to "a=b" for UnknownAsFalse semantics

2021-12-02 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4917: - Summary: Add test for 'IS NOT NULL(a) AND a=b' simplifies to "a=b" for UnknownAsFalse semantics Key: CALCITE-4917 URL: https://issues.ap

Re: Feedback on a generic return type version of RelShuttle?

2021-12-01 Thread Alessandro Solimando
Hi Jacques, I have faced the same issue recently and I think it's a good idea to have a generic version of it and have RelShuttle be one of its concrete implementations. Best regards, Alessandro Il Gio 2 Dic 2021, 06:02 Jacques Nadeau ha scritto: > Right now, RelNode's accept method and RelShu

[jira] [Created] (CALCITE-4894) MV rewriting fails for expressions with more than a field reference in the view/query projection

2021-11-18 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4894: - Summary: MV rewriting fails for expressions with more than a field reference in the view/query projection Key: CALCITE-4894 URL: https://issues.apache.org/jira

Re: [DISCUSS] Apache Calcite Online Meetup January 2022

2021-11-17 Thread Alessandro Solimando
I'd be interested too! Il Mer 17 Nov 2021, 19:26 Jacques Nadeau ha scritto: > Great idea. +1 from me. > > On Wed, Nov 17, 2021 at 8:22 AM Stamatis Zampetakis > wrote: > > > Hi all, > > > > Last meetup [1] was roughly 10 months ago. How about organising another > one > > around mid January 2022?

Re: How to get the real column name and data type from a SQL view ?

2021-11-09 Thread Alessandro Solimando
Hi Justin, the code is indeed recursive, I have never played with too complex queries so I can't say for sure but I bet it can handle complex cases like the one you described. `getExpressionLineage` too could come handy for such tasks. Armstrong, if possible it would definitely help sharing the s

Re: Is there a mechanism for constant folding in Calcite?

2021-10-26 Thread Alessandro Solimando
Hi Ian, regarding commutativity/associativity I think this ML discussion could be relevant and it has some pointers. For what concerns decorrelation, I think that most of

help review PR (ugrade to Cassandra 4 - CALCITE-4768)

2021-10-25 Thread Alessandro Solimando
Hello everyone, if someone has spare cycles I'd appreciate some help merging this PR: https://github.com/apache/calcite/pull/2592 This is the associated ticket: https://issues.apache.org/jira/browse/CALCITE-4768 The PR adds Cassandra 4 support, and upgrades as well the DataStax driver we use to t

Re: Looking forward to joining the Calcite community

2021-10-17 Thread Alessandro Solimando
Hello Zhe Hu, welcome to the community! Best regards, Alessandro Il Dom 17 Ott 2021, 09:38 Francis Chuang ha scritto: > Hey Zhe Hu, > > Welcome to the project! We look forward to your contributions. I've > added you as a contributor in jira, so you can start assigning issues to > yourself. > >

Re: [ANNOUNCE] New committer: Zhaohui Xu

2021-10-07 Thread Alessandro Solimando
Congratulations Zhaohui, I have followed some of the contributions on the view-based rewritings, thanks for all the improvements! Best regards, Alessandro On Thu, 7 Oct 2021 at 07:27, Haisheng Yuan wrote: > Congratulations, Zhaohui, well deserved! > > Haisheng > > On 2021/10/06 21:14:00, Franci

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

2021-10-07 Thread Alessandro Solimando
+1 (non-binding), thanks Julian for preparing the RC! -) Checked hash and signature: OK -) Checked release notes: OK (but NoboGo should be NobiGo I think) -) Checked the release tag with right commit: OK -) Checking the tag and the tar.gz content: OK diff -qr apache-calcite-avatica-1.19.0-src ~/

Re: Need help: exception when using Elasticsearch adapter

2021-10-01 Thread Alessandro Solimando
use > Elasticsearch/ElasticsearchTable in model json/yaml to parse the SQL > statement? I assumed Calcite can convert any supported SQL to Elasticsearch > DSL no matter what the Schema/Table specification of the SQL are. > > Thanks, > Justin > > From > Alessandro Solimand

Re: [DISCUSS] Add extra check for Avatica to run tests with different timezone

2021-10-01 Thread Alessandro Solimando
Thanks Vladimir for the proposal. +1 frome me, there are more and more "axes" for testing (OS, locale, JVM, timezone, etc.), a randomized approach sounds like a good solution to tame the combinatorial explosion. Best regards, Alessandro On Fri, 1 Oct 2021 at 08:32, Vladimir Sitnikov wrote: > >

Re: [DISCUSS] Add extra check for Avatica to run tests with different timezone

2021-09-30 Thread Alessandro Solimando
Of course it would be fine to just change the timezone for one of more existing tests which are targeting different JVM versions, no need for a brand new check. BR, Alessandro On Fri, 1 Oct 2021 at 08:07, Alessandro Solimando < alessandro.solima...@gmail.com> wrote: > Hello everyone,

[DISCUSS] Add extra check for Avatica to run tests with different timezone

2021-09-30 Thread Alessandro Solimando
Hello everyone, we have recently had an issue for a change in Avatica which was not working for all timezones that almost slipped through (see CALCITE-4600 ) if it was not for a local run which happened to be in a different timezone. As for Calci

[jira] [Created] (CALCITE-4816) Make Gradle pass the 'user.timezone' property to the test JVM (Avatica)

2021-09-30 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4816: - Summary: Make Gradle pass the 'user.timezone' property to the test JVM (Avatica) Key: CALCITE-4816 URL: https://issues.apache.org/jira/browse/CA

Re: Need help: exception when using Elasticsearch adapter

2021-09-29 Thread Alessandro Solimando
Each adapter "speaks" the language of the supported data source, but only to the extent of querying it, and for the tables stored in it. If your input tables are not from ES I have a hard time to even understand what you are trying to achieve in this way. Consider also that a plan consists genera

[REVIEW-REQUEST] [CALCITE-4790] Make Gradle pass the 'user.timezone' to test JVM

2021-09-28 Thread Alessandro Solimando
Hello everyone, can someone with some spare cycles check this 1 line PR: https://github.com/apache/calcite/pull/2534 We all agree that tests must pass in all timezones, but since we have CI jobs running in different timezones, it would be nice to be able to easily test under the same conditions ev

Re: [DISCUSS] Next releases

2021-09-27 Thread Alessandro Solimando
Hi everyone, I have prepared the PR, you can find it here: https://github.com/apache/calcite-avatica/pull/154. Best regards, Alessandro On Mon, 27 Sept 2021 at 20:27, Alessandro Solimando < alessandro.solima...@gmail.com> wrote: > Hi Julian, > I can help with PR-143 > <https:/

Re: [DISCUSS] Next releases

2021-09-27 Thread Alessandro Solimando
Hi Julian, I can help with PR-143 , since I have contributed to the modified classes in the past. I will still need a committer to merge but I can rebase on master and fix issues, if any. Best regards, Alessandro On Mon, 27 Sept 2021 at 07:52,

Re: [DISCUSS] Remove contributors name from commit summary

2021-09-23 Thread Alessandro Solimando
Nowadays Github allows you to fetch that info very easily, I think it's a leftover from the past where the task proved to be harder. On Thu, 23 Sept 2021 at 22:09, Jacques Nadeau wrote: > I lead towards allowing people to use github if they want to know who > contributed what. Simple is better.

[jira] [Created] (CALCITE-4796) Travis links in README.md should point to 'travis-ci.com' instead of ' travis-ci.org'

2021-09-23 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4796: - Summary: Travis links in README.md should point to 'travis-ci.com' instead of ' travis-ci.org' Key: CALCITE-4796 URL: https://issues.ap

[jira] [Created] (CALCITE-4793) CassandraAdapterDataTypesTest.testCollectionsInnerValues fails for guava <= 25.0-jre

2021-09-22 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4793: - Summary: CassandraAdapterDataTypesTest.testCollectionsInnerValues fails for guava <= 25.0-jre Key: CALCITE-4793 URL: https://issues.apache.org/jira/bro

[jira] [Created] (CALCITE-4790) Make Gradle pass the 'user.timezone' property to the test JVM

2021-09-22 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4790: - Summary: Make Gradle pass the 'user.timezone' property to the test JVM Key: CALCITE-4790 URL: https://issues.apache.org/jira/browse/CA

Re: Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-08-16 Thread Alessandro Solimando
. Looking forward to hearing the comments from the community! Best regards, Alessandro On Thu, 18 Feb 2021 at 09:11, Alessandro Solimando < alessandro.solima...@gmail.com> wrote: > Hello all, > Thanks Francis for your message. > > I have filed CALCITE-4503 > <https://issue

Re: [ANNOUNCE] New committer: Vladimir Ozerov

2021-06-23 Thread Alessandro Solimando
Congratulations Vladimir, well deserved, I had the chance to read some of the blog posts and I have appreciated them very much. Best regards, Alessandro On Thu, 24 Jun 2021 at 07:58, Viliam Durina wrote: > > Congratulations! > > Viliam > > On Thu, 24 Jun 2021 at 06:58, Forward Xu wrote: > > > C

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 > mm...@apach

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

2021-06-01 Thread Alessandro Solimando
Hi all, +1 (non-binding) after verifying the release as follows: 1) Checksum: OK 2) Signature: OK 3) Running tests: “./gradlew test”: OK (see JVM, OS and gradle details at the bottom) "gradlew" and the content of "gradle" folder has been copied from the git local checkout 4) Compared archive conte

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

2021-05-31 Thread Alessandro Solimando
Hi Stamatis, thanks a lot for the release candidate, I will check it asap in more details. In the meantime I have just noticed in the release notes that the date is off (the year): 1.27.0 / 2020-06-XX Best regards, Alessandro On Mon, 31 May 2021 at 17:29, Stamatis Zampetakis wrote: > > Hi all,

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

2021-05-14 Thread Alessandro Solimando
(See > https://issues.apache.org/jira/browse/CALCITE-4575 < > https://issues.apache.org/jira/browse/CALCITE-4575>.) > > Julian > > > > On May 12, 2021, at 3:00 AM, Alessandro Solimando < > alessandro.solima...@gmail.com> wrote: > > > > Hi, >

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

2021-05-12 Thread Alessandro Solimando
Hi, +1 (non-binding) after verifying the release as follows. 1) Checksum: OK 2) Signature: OK 3) Running tests: - “./gradlew build -Prelease -PskipSign”: OK (see JVM, OS and gradle details at the bottom) - “docker-compose run test”: OK - Check CI on github for the commi

Re: Questions on JDBC adapter

2021-04-29 Thread Alessandro Solimando
Hi Francesco, regarding the "where to record the issues" bit, we use JIRA: https://issues.apache.org/jira/projects/CALCITE/summary You might have already seen this, but that's the contributor's guideline: https://calcite.apache.org/develop/#contributing Best regards, Alessandro On Thu, 29 Apr 20

Re: Reset/restart CI

2021-04-16 Thread Alessandro Solimando
Hi all, locally on MacOS I have to set the SPARK_LOCAL_IP environment variable to overcome that very same problem (something along the line of export SPARK_LOCAL_IP="127.0.0.1"). I wonder if it wouldn't be less intrusive to set an environment variable rather than statically setting a SparkConf opt

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

2021-04-12 Thread Alessandro Solimando
Hi everyone, +1 (non-binding) after verifying the release as follows. 1) Checksum: OK 2) Signature: OK (same warning as Julien Hyde but that’s explained and makes sense) 3) Running tests: - “./gradlew build -Prelease -PskipSign”: OK (see JVM and OS details at the bottom) - “do

[jira] [Created] (CALCITE-4556) CalciteMetaImpl#createEmptyResultSet override of (Avatica) MetaImpl#createEmptyResultSet should not pass a class instance to CursorFactory#deduce

2021-03-28 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4556: - Summary: CalciteMetaImpl#createEmptyResultSet override of (Avatica) MetaImpl#createEmptyResultSet should not pass a class instance to CursorFactory#deduce Key: CALCITE-4556

Re: Time zone management and DST in Avatica

2021-03-28 Thread Alessandro Solimando
Hi Vladimir, your analysis seems correct to me, as well as your proposed solution. Best regards, Alessandro On Sun, 28 Mar 2021 at 09:51, Vladimir Ozerov wrote: > Hi, > > Avatica assumes that the underlying engine operates on relative time > without time zone as opposed to global time (UTC). Wh

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-24 Thread Alessandro Solimando
gt; Francis > > On 23/02/2021 7:06 pm, Alessandro Solimando wrote: > > Hi, > > regarding CALCITE-2489: > https://github.com/apache/calcite-avatica/pull/138 > > > > I have improved the PR description with a more detailed walkthrough of > the > > changes, hopi

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-23 Thread Alessandro Solimando
, Alessandro On Sat, 20 Feb 2021 at 21:13, Alessandro Solimando < alessandro.solima...@gmail.com> wrote: > Hi Francis, all, > I saw Stamatis' comment under > https://github.com/apache/calcite-avatica/pull/105 (CALCITE-3163) asking > to add unit-tests to avoid regressions, I have

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-20 Thread Alessandro Solimando
gards, Alessandro On Fri, 19 Feb 2021 at 08:30, Francis Chuang wrote: > Hi Alessandro, > > Thanks! That would be really awesome! > > Francis > > On 19/02/2021 5:47 pm, Alessandro Solimando wrote: > > Hi, > > I can review and open a new PR for > > https:

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-18 Thread Alessandro Solimando
Hi, I can review and open a new PR for https://github.com/apache/calcite-avatica/pull/105 (CALCITE-3163) if a committer can then help me merging it. Best regards, Alessandro Il Ven 19 Feb 2021, 00:45 Julian Hyde ha scritto: > I'll take PR 121, [CALCITE-3881]. > > On Thu, Feb 18, 2021 at 2:19 PM

Re: Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-02-18 Thread Alessandro Solimando
lso has a corresponding PR > > for Calcite, which would require this PR to go into Avatica if we want > > the corresponding PR for Calcite to go into the next Calcite release. > > > > Francis > > > > On 16/02/2021 10:15 am, Alessandro Solimando wrote: > > >

[jira] [Created] (CALCITE-4503) Order of fields in records should follow that of the SQL types

2021-02-18 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4503: - Summary: Order of fields in records should follow that of the SQL types Key: CALCITE-4503 URL: https://issues.apache.org/jira/browse/CALCITE-4503

Re: Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-02-15 Thread Alessandro Solimando
Hi again, sorry I have hit "send" too soon, here is the PR: https://github.com/apache/calcite-avatica/pull/138 Best regards, Alessandro On Mon, 15 Feb 2021 at 23:46, Alessandro Solimando < alessandro.solima...@gmail.com> wrote: > Hi Julian, > sorry for the late reply. &

Re: Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-02-15 Thread Alessandro Solimando
Hi Julian, sorry for the late reply. It's true that with third-party classes annotations are not viable, and that's an important use case. Regarding the test case and the user experience, I like the idea of making the strategy configurable, I'd capture the strategies you have listed via an enum a

Re: Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-02-09 Thread Alessandro Solimando
Stamatis, Vladimir, thanks for your comments. The empty constructor Stamatis is suggesting would probably be enough, but what Vladimir is suggesting would also improve on the annotation front (in particular, it's easier to annotate just the constructor rather than each field in the whole hierarchy

Discussion around CALCITE-2489 (risks linked to Class.getFields())

2021-02-07 Thread Alessandro Solimando
Hello everyone, I had a look at CALCITE-2489 (Order of fields in JavaTypeFactoryImpl#createStructType is unstable) since it was linked to few PRs I worked on in the last months. In a nutshell, the output of "Class.getFields()" is JVM implementat

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-04 Thread Alessandro Solimando
The tests on your rebased > branch look good. > > Can you open a new PR with your rebased version? The commit message > should also be updated to the correct format: > > [CALCITE-3401] Assume empty keystore passwords by default (Istvan Toth, > Alessandro Solimando) > >

Re: [DISCUSS] Towards Avatica 1.18.0

2021-02-04 Thread Alessandro Solimando
Hi Francis, I had a look at https://github.com/apache/calcite-avatica/pull/115, LGTM, I haven't done a review on the PR itself as I have nothing I'd change, if you think my approval can help I will mark it as such. I noticed that the PR has conflicts with master, I rebased it and run all tests loc

Re: [ANNOUNCE] New Calcite PMC chair: Haisheng Yuan

2020-12-18 Thread Alessandro Solimando
Thanks Stamatis for your hard work and dedication, and congratulations to Haisheng for this appointment! Best regards, Alessandro On Fri, 18 Dec 2020 at 07:31, Xin Wang wrote: > Congrats Haisheng! Thanks for your work, Stamatis! > > > Fan Liya 于2020年12月18日周五 下午12:08写道: > > > Congratulations, H

Re: how to set string representation in calcite SQL;

2020-12-16 Thread Alessandro Solimando
Hello, you can achieve what you want via the SqlParser configuration (.setQuoting() method), for instance: private static final SqlParser.Config SQL_PARSER_CONFIG = SqlParser.configBuilder(SqlParser.Config.DEFAULT) > .setCaseSensitive(false) > .setConformance(SqlConformanceEnum.DEFAULT) >

[jira] [Created] (CALCITE-4436) Use the fields order from the struct type for 'ITEM(STRUCT, INDEX)' access

2020-12-10 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4436: - Summary: Use the fields order from the struct type for 'ITEM(STRUCT, INDEX)' access Key: CALCITE-4436 URL: https://issues.apache.org/jira/browse/CA

Re: [DISCUSS] Screenshots of stack traces, code, and debugger sessions

2020-10-29 Thread Alessandro Solimando
Hi all, +1 for text over images whenever possible (for the same reason, it's convenient to have it indexed by search engines). +0 for image support, I see the benefits but I am afraid we will see less and less text/stack traces. Best regards, Alessandro On Wed, 28 Oct 2020 at 01:13, Michael Mior

[jira] [Created] (CALCITE-4354) ITEM operator does not support synthetic struct type

2020-10-24 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4354: - Summary: ITEM operator does not support synthetic struct type Key: CALCITE-4354 URL: https://issues.apache.org/jira/browse/CALCITE-4354 Project

Re: why DECIMAL is in SqlTypeName#EXACT_TYPES but DOUBLE is in SqlTypeName#APPROX_TYPES

2020-10-19 Thread Alessandro Solimando
o Tao wrote: > Thanks, Alessandro Solimando > Got it, very clear explanation. > My main question here becomes that calcite doesn't support double mod. > > Regards! > > Aron Tao > > > Alessandro Solimando 于2020年10月15日周四 > 下午10:32写道: > > > Hi Aron Tao,

Re: why DECIMAL is in SqlTypeName#EXACT_TYPES but DOUBLE is in SqlTypeName#APPROX_TYPES

2020-10-15 Thread Alessandro Solimando
Hi Aron Tao, Calcite adheres to SQL standard in that respect (citing): The data types NUMERIC, DECIMAL, INTEGER and SMALLINT are collectively > referred to > as exact numeric types. The data types FLOAT, REAL, and DOUBLE PRECISION are collectively referred > to as approx- > imate numeric types.

Re: [DISCUSS] Apache Calcite Online Meetup January 2021

2020-10-15 Thread Alessandro Solimando
Hi Stamatis, that's a very nice idea, I am definitely interested in attending a meetup and hearing presentations around Calcite. Best regards, Alessandro On Thu, 15 Oct 2020 at 15:09, Stamatis Zampetakis wrote: > Hi guys, > > How do you feel about participating in an online meetup for Calcite i

Re: It's my pleasure to introduce myself to you

2020-10-14 Thread Alessandro Solimando
Hello Guangyuan, welcome to the community. There is a dedicated set of labels ("newbie", "beginner", "starter", "easyfix") in JIRA for familiarising with the project. Best regards, Alessandro On Wed, 14 Oct 2020 at 08:29, guangyuan wang wrote: > Hi Danny Chan > Nice to meet you. I'm very glad

Re: [CassandraAdapter] selecting tuple elements always returns "null"

2020-10-01 Thread Alessandro Solimando
t; I have zero experience with CassandraAdapter, but just a naive comment from > what you describe: could the issue be that it ends up calling > SqlFunctions#item instead of SqlFunctions#structAccess [1] ? > > Best, > Ruben > > [1] > > https://github.com/apache/ca

[jira] [Created] (CALCITE-4301) Unit test 'testCollectionsInnerValues()' for Cassandra adapter is wrong

2020-10-01 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4301: - Summary: Unit test 'testCollectionsInnerValues()' for Cassandra adapter is wrong Key: CALCITE-4301 URL: https://issues.apache.org/jira/browse/CA

Re: [CassandraAdapter] selecting tuple elements always returns "null"

2020-09-28 Thread Alessandro Solimando
t[]" (instead of a "List"), so most probably the real cause is elsewhere, and what I am observing (the call to "item" function with an array) is just a symptom. In the meantime I have opened CALCITE-4293 <https://issues.apache.org/jira/browse/CALCITE-4293> to report t

[jira] [Created] (CALCITE-4293) cassandra adapter returns null when selecting non-null tuple elements

2020-09-28 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-4293: - Summary: cassandra adapter returns null when selecting non-null tuple elements Key: CALCITE-4293 URL: https://issues.apache.org/jira/browse/CALCITE-4293

[CassandraAdapter] selecting tuple elements always returns "null"

2020-09-27 Thread Alessandro Solimando
Hello all, at the time I wrote the unit tests for the extended support for the missing Cassandra data types, I have disabled the one at CassandraAdapterDataTypesTest.java:192

Re: IntelliJ and Gradle

2020-02-09 Thread Alessandro Solimando
Hello, I recently had the chance to try the new building system and I wanted to provide a quick feedback. At first I forgot to scratch and re-import the project when I switched from maven to gradle: the errors where not super clear but I had memories of this thread so it did not take much to figur

Re: Calcite-Master - Build # 1588 - Failure

2020-02-01 Thread Alessandro Solimando
Thanks a lot Vladimir, your fix on master indeed solved my issue. Unfortunately this time AppVeyor failed on ElasticSearchAdapterTest due to a "java.net.SocketTimeoutException" (link to the build https://ci.appveyor.com/project/ApacheSoftwareFoundation/calcite/builds/30510892/job/0o5meoj2omuxadi8

Re: Calcite-Master - Build # 1588 - Failure

2020-02-01 Thread Alessandro Solimando
Hi Vladimir, I see, so most probably not related to the problem I am having. What do you mean by "Calcite never uses Jenkins for PR validation."? My PR is this one: https://github.com/apache/calcite/pull/1774 Best regards, Alessandro On Sat, 1 Feb 2020 at 22:39, Vladimir Sitnikov wrote: > Hi,

Re: Calcite-Master - Build # 1588 - Failure

2020-02-01 Thread Alessandro Solimando
Hello, I had a CI failure on my PR (on code unrelated to my PR which concerns only CassandraAdapter), as per documentation I have re-triggered with an empty commit and I get errors again. At first I thought about a difference between my local (Linux) setup and the Linux job in Travis, but the seco

Re: Re: [ANNOUNCE] New Calcite PMC chair: Stamatis Zampetakis

2019-12-18 Thread Alessandro Solimando
Congratulations, Stamatis! Il Gio 19 Dic 2019, 07:16 Enrico Olivelli ha scritto: > Congratulations Stamatis! > > Enrico > > Il gio 19 dic 2019, 04:40 Rui Wang ha scritto: > > > Congratulations and Thanks Stamatis! > > > > > > > > -Rui > > > > On Wed, Dec 18, 2019 at 6:52 PM XING JIN > wrote: >

Re: Quicksql

2019-12-12 Thread Alessandro Solimando
Adapters must be needed by data sources not supporting SQL, I think this is what Juan Pan was asking for. On Thu, 12 Dec 2019 at 04:05, Haisheng Yuan wrote: > Nope, it doesn't use any adapters. It just submits partial SQL query to > different engines. > > If query contains table from single sour

[avatica] STRUCT type information missing in type name

2019-11-05 Thread Alessandro Solimando
Hello, I noticed that type information inside the "name" field into "ColumnMetaData" are dropped for the inner components of STRUCT, while they are preserved for MULTISET, ARRAY, LIST, MAP. For example the following "RelRecordType": > RecordType( > INTEGER f_int, > INTEGER NOT NULL ARRAY f_li

Re: CassandraAdapter (Add Type) and WHERE statement.

2019-10-31 Thread Alessandro Solimando
Hello, I have logged a Jira ticket for this: https://issues.apache.org/jira/browse/CALCITE-3465 I have listed all the data types for Cassandra 3.x, and I have tried to compile a table with the current status, but there are few entries for which I am not sure. If you have time to contribute to the

[jira] [Created] (CALCITE-3465) Cover Cassandra 3.x data types

2019-10-31 Thread Alessandro Solimando (Jira)
Alessandro Solimando created CALCITE-3465: - Summary: Cover Cassandra 3.x data types Key: CALCITE-3465 URL: https://issues.apache.org/jira/browse/CALCITE-3465 Project: Calcite Issue

Re: [DISCUSS] State of the project 2019

2019-10-24 Thread Alessandro Solimando
Hello, I have the impression too that Calcite is lesser known than other Apache projects, but this is not that surprising to me given the nature of the project itself, which requires in-depth knowledge of a rather complex topic, which makes it is less prone to a casual use and plug&play adoption. B

Re: Apache Calcite meetup group

2019-10-22 Thread Alessandro Solimando
Hello, despite not being able to contribute actively I keep following the ML and the project, I'd love to attend meetups here in Paris if any! Best regards, Alessandro On Tue, 22 Oct 2019 at 05:08, Julian Hyde wrote: > Best thing would be for you and Jesus to collaborate. Feel free to do it > o

<    1   2   3   >