Re: "Patch" flag vs. "pull-request-available" label

2019-01-07 Thread Hequn Cheng
+1 for “pull-request-available”. On Tue, Jan 8, 2019 at 2:13 PM Jark Wu wrote: > +1 for "pull-request-available" > > > On Tue, 8 Jan 2019 at 05:33, Francis Chuang > wrote: > > > +1 for "pull-request-available" > > > > On 8/01/2019 8:13 am, Julian Hyde wrote: > > > There seem to be two mechanism

Re: "Patch" flag vs. "pull-request-available" label

2019-01-07 Thread Jark Wu
+1 for "pull-request-available" On Tue, 8 Jan 2019 at 05:33, Francis Chuang wrote: > +1 for "pull-request-available" > > On 8/01/2019 8:13 am, Julian Hyde wrote: > > There seem to be two mechanisms to indicate that a pull-request is > available for a JIRA case. > > > > 1. The “Patch” flag (13 i

[GitHub] F21 closed pull request #36: [CALCITE-2775] Upgrade dependencies and regenerate protobufs

2019-01-07 Thread GitBox
F21 closed pull request #36: [CALCITE-2775] Upgrade dependencies and regenerate protobufs URL: https://github.com/apache/calcite-avatica-go/pull/36 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] F21 opened a new pull request #36: [CALCITE-2775] Upgrade dependencies and regenerate protobufs

2019-01-07 Thread GitBox
F21 opened a new pull request #36: [CALCITE-2775] Upgrade dependencies and regenerate protobufs URL: https://github.com/apache/calcite-avatica-go/pull/36 This is an automated message from the Apache Git Service. To respond t

[jira] [Created] (CALCITE-2775) Update dependencies and regenerate protobufs

2019-01-07 Thread Francis Chuang (JIRA)
Francis Chuang created CALCITE-2775: --- Summary: Update dependencies and regenerate protobufs Key: CALCITE-2775 URL: https://issues.apache.org/jira/browse/CALCITE-2775 Project: Calcite Issue

Re: Breaking change in Avatica-Go when handling null and empty strings for Apache Phoenix

2019-01-07 Thread Francis Chuang
The sql.NullString [1] type exists for this. There are a few other ones: NullFloat64, NullInt64 and NullBool. However, while it does exist, we're at the mercy of Phoenix. Since Phoenix only returns a Rep_Type of Null and null = true regardless of whether the string is empty or an actual null,

[jira] [Created] (CALCITE-2774) Improve release documentation for Avatica-Go

2019-01-07 Thread Francis Chuang (JIRA)
Francis Chuang created CALCITE-2774: --- Summary: Improve release documentation for Avatica-Go Key: CALCITE-2774 URL: https://issues.apache.org/jira/browse/CALCITE-2774 Project: Calcite Issue

Re: Breaking change in Avatica-Go when handling null and empty strings for Apache Phoenix

2019-01-07 Thread Julian Hyde
OK, it seems you’ve given this plenty of thought. I don’t know Go, but it seems that string values can never be null, in the same way that int values cannot be null in Java. But SQL INTEGER columns can have NULL values, so JDBC needs a way to represent the NULL values on the client. In JDBC, if

Re: Info need on the lib encryption

2019-01-07 Thread Francis Chuang
I have had instances where replies to messages on Apache mailing lists went into my spam folder. @Hermakumar, if you did not see Michael's reply in your inbox, please check your spam folder. There is an archive of his message here: https://mail-archives.apache.org/mod_mbox/calcite-dev/201812.m

Re: Breaking change in Avatica-Go when handling null and empty strings for Apache Phoenix

2019-01-07 Thread Francis Chuang
This behavior is the default for Phoenix and will only apply to Phoenix. Unfortunately, I don't think there it would be possible to turn this behavior on or off in the Go client as there is no way to distinguish a null and an empty string that is being returned by Avatica. This is what the Go

Re: Info need on the lib encryption

2019-01-07 Thread Julian Hyde
Sorry, my bad moderating it through this time. > On Jan 7, 2019, at 1:34 PM, Josh Elser wrote: > > Please read Michael's reply to you from the first time you sent this message > instead of sending it again. > > Let's not moderate this through a third time. > > On 1/7/19 4:06 PM, Hemakumar Gok

Re: Info need on the lib encryption

2019-01-07 Thread Josh Elser
Please read Michael's reply to you from the first time you sent this message instead of sending it again. Let's not moderate this through a third time. On 1/7/19 4:06 PM, Hemakumar Gokulakannan wrote: Hello Team, We are trying to use Apache Calcite core 1.17.0*(including avatica-core-1.12.0,

Re: "Patch" flag vs. "pull-request-available" label

2019-01-07 Thread Francis Chuang
+1 for "pull-request-available" On 8/01/2019 8:13 am, Julian Hyde wrote: There seem to be two mechanisms to indicate that a pull-request is available for a JIRA case. 1. The “Patch” flag (13 issues); see https://issues.apache.org/jira/issues/?jql=project%20%3D%20CALCITE%20AND%20flags%20%3D%20

Re: Info need on the lib encryption

2019-01-07 Thread Hemakumar Gokulakannan
Hello Team, We are trying to use Apache Calcite core 1.17.0*(including avatica-core-1.12.0, avatica-metrics-1.12.0 and calcite-linq4j-1.17.0)* in our application, however we need some info regarding this library. We do want to know whether this library contain encryption or hooks to encryption?

Re: "Patch" flag vs. "pull-request-available" label

2019-01-07 Thread Vladimir Sitnikov
> “pull-request-available” +1 Vladimir

"Patch" flag vs. "pull-request-available" label

2019-01-07 Thread Julian Hyde
There seem to be two mechanisms to indicate that a pull-request is available for a JIRA case. 1. The “Patch” flag (13 issues); see https://issues.apache.org/jira/issues/?jql=project%20%3D%20CALCITE%20AND%20flags%20%3D%20%22Patch%22

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Vladimir Sitnikov
Julian>I don’t know how this is possible. It may be a bug. I did spent noticeable amount of time debugging traits, and I did try my best at summarizing the issue in the very first mail of the thread. The very first email is enough. It is truly sad you don' The description follows after "My analys

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Julian Hyde
I don’t know how this is possible. It may be a bug. I’m sorry that I am not fully reading these threads, nor running the test cases in the debugger. My time to spend on Calcite is very limited. I am trying to contribute my wisdom / experience / intuition. Discussion threads where I am forced to

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Vladimir Sitnikov
Julian> No, it is not the “unknown collation” subset. It is more like the “don’t care” subset. Would you please care to explain how the following is possible? (it is from the start of the thread) LogicalProject(subset=[rel#11:Subset#1.NONE.[0]], input=rel#9:Subset#0.NONE.[], Y=[$1]) I read that

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Julian Hyde
No, it is not the “unknown collation” subset. It is more like the “don’t care” subset. Traits are a form of quality-of-service. Like when you order a package, you can pay extra for 2-day shipping, and even more for 1-day shipping. You pay more for a tighter guarantee. The subset with collation

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Vladimir Sitnikov
Julian> I don’t think the [] means “no collation”. It means “order by zero columns”. The list of columns is present, but it is empty. I agree I might be wrong from technical point of view. Frankly speaking, I don't think I've ever understood what Calcite traits are even though I happen to land a c

Re: Breaking change in Avatica-Go when handling null and empty strings for Apache Phoenix

2019-01-07 Thread Julian Hyde
How this behavior enabled? Apache Phoenix’s behavior contravenes the SQL standard (which states that empty strings and null strings are different) and so clients must not get this behavior unless they ask for it. It would be OK if they get it by default for Phoenix, but even then, there should

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Julian Hyde
I don’t think the [] means “no collation”. It means “order by zero columns”. The list of columns is present, but it is empty. Of course, this “empty collation” provides no guarantees as to the ordering of rows. It is a collation; it’s just the weakest possible collation. > On Jan 7, 2019, at 1

[jira] [Created] (CALCITE-2773) Prune of empty values causes assertion error "rel cost is lower than best cost"

2019-01-07 Thread Vladimir Sitnikov (JIRA)
Vladimir Sitnikov created CALCITE-2773: -- Summary: Prune of empty values causes assertion error "rel cost is lower than best cost" Key: CALCITE-2773 URL: https://issues.apache.org/jira/browse/CALCITE-2773

Re: Help wit Calcite

2019-01-07 Thread Michael Mior
This is the sort of situation I'm hoping the notebooks I recently published will be useful for (link below). I haven't written up any documentation yet but you can click through to any of the notebooks to see pre-generated output or click one of the launch buttons to modify the code and play with i

Help wit Calcite

2019-01-07 Thread Tom Shashaty
Hi, I am trying to find resources to help me understand Calcite better. I am mainly looking to use Calcite to convert a query based on a logical model and transform the query to a physical model representation that I can use to execute against a database. This would be accomplished using

Re: [Query] Can we use calcite for rewriting the query based on Materialized View

2019-01-07 Thread Kumar Vishal
Hi Stamatis, I have converted my testcase to UT, please find it in the attachment. -Regards Kumar Vishal On Sat, Dec 22, 2018 at 1:32 PM Stamatis Zampetakis wrote: > Hi Kumar, > > I think it would be helpful if you could shape your code into a unit test > and share it with us. It will be easie

Re: Review request for CALCITE-1912: Support "FOR SYSTEM_TIME AS OF" in regular queries

2019-01-07 Thread 伍翀(云邪)
Hi Francis, Thank you for bring it here. The “FOR SYSTEM_TIME AS OF” operation is introduced in SQL:2011 temporal feature. It is a standard way to do a temporal table join using this syntax which we want to support in Apache Flink. We will be very grateful if anyone can review it. Thanks, Jar

[jira] [Created] (CALCITE-2772) Support variable parameter for udf

2019-01-07 Thread pengzhiwei (JIRA)
pengzhiwei created CALCITE-2772: --- Summary: Support variable parameter for udf Key: CALCITE-2772 URL: https://issues.apache.org/jira/browse/CALCITE-2772 Project: Calcite Issue Type: Improvement

Re: CALCITE-2593 / CALCITE-2764. Composite collations: review / comment request

2019-01-07 Thread Stamatis Zampetakis
Hi Vladimir, >From the analysis so far, the following part caught my attention: Vladimir>So far so good, however the RelSubset for LogicalValues is awkward: rel#9:Subset#0.NONE.[] [] means "no collation". In other words, it puts "multi-collated" values into a subset that has "no" collation. ...