[jira] [Updated] (CALCITE-6393) Byte code of SqlFunctions is invalid

2024-04-30 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-6393: - Affects Version/s: 1.36.0 > Byte code of SqlFunctions is inva

Re: [VOTE] Release Apache Calcite 1.37.0 (release candidate 4)

2024-04-30 Thread Julian Hyde
It seems that Sergey has logged https://issues.apache.org/jira/browse/CALCITE-6393 for this. Please add further discussion to that case rather than to this vote thread. > On Apr 30, 2024, at 12:44 AM, Ruben Q L wrote: > > Thanks Guillaume for checking this! > When I looked at this issue on

Re: [DISCUSS] Move RTD files to their own repo

2024-04-30 Thread Julian Hyde
Something to consider is keeping the site and the code in the same repo, which is what Calcite does. Calcite builds a site (hosted on Apache rather than RTD) based on a Git branch (called “site”) of our main repo. This allows us to update the site without making a release. But we sometimes

Re: Is it posible to change synthetic input names ($0, $1, etc) in explain plans?

2024-04-30 Thread Julian Hyde
Gonzalo, I think the implementation is fairly straightforward. You’d write a new implementation of RelWriter. But, as Alessandro says, the specification is the hard part. What *exactly* would you want instead of $0? (Those fields have names, and most of the time they are the same as the user

[jira] [Commented] (CALCITE-4496) Measure columns ("SELECT ... AS MEASURE")

2024-04-30 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-4496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17842472#comment-17842472 ] Julian Hyde commented on CALCITE-4496: -- An update on the development of this feature. Basic

[jira] [Commented] (CALCITE-6300) Function MAP_VALUES/MAP_KEYS gives exception when mapVauleType and mapKeyType not equals map Biggest mapKeytype or mapValueType

2024-04-28 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17841666#comment-17841666 ] Julian Hyde commented on CALCITE-6300: -- Maybe it could get away with it, but it probably shouldn’t

[jira] [Updated] (CALCITE-6386) Elasticsearch adapter throws NullPointerException when used with with model.json and no username, password or pathPrefix

2024-04-27 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6386?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-6386: - Summary: Elasticsearch adapter throws NullPointerException when used with with model.json

[jira] [Commented] (CALCITE-6386) NPE when using ES adapter with model.json and no specified username, password or pathPrefix

2024-04-27 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17841501#comment-17841501 ] Julian Hyde commented on CALCITE-6386: -- I changed the summary to remove abbreviations. Please

[jira] [Commented] (CALCITE-6373) Distinct optimization produces broken sql query

2024-04-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17841285#comment-17841285 ] Julian Hyde commented on CALCITE-6373: -- I suspect that the fix only fixes the special case of one

Re: Some tests under SqlValidatorTest are disabled with strange annotation

2024-04-26 Thread Julian Hyde
Dtbug 280 is in the long-defunct bug database of a long-defunct company. The comments in calcite code are all that’s left of it. If you can reenable those tests you can safely declare the bug fixed. Julian PS Ironically, “dt” stands for “disruptive tech”. (“My name is Ozymandias, King of

[jira] [Commented] (CALCITE-6383) The class SameOperandTypeChecker is incorrectly documented

2024-04-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17841062#comment-17841062 ] Julian Hyde commented on CALCITE-6383: -- Pr looks good except than you can’t use “*” for emphasis

Re: [VOTE] Release Apache Baremaps 0.7.3-rc1 (incubating)

2024-04-22 Thread Julian Hyde
I am a mentor of Baremaps. Sorry that I didn't have time to vote on this RC on dev@. +1 (binding) Downloaded, checked keys, compiled and ran tests (following instructions in README), ran RAT. I am not familiar with "Creative Commons Public License". Is it compatible with ALv2? Julian On Mon,

[jira] [Commented] (CALCITE-6374) LatticeSuggester throw npe when agg call covered with cast

2024-04-18 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17838774#comment-17838774 ] Julian Hyde commented on CALCITE-6374: -- [~allendang], Can you please remove the PNG and replace

[jira] [Commented] (CALCITE-6375) Fix failing quidem tests for FORMAT in CAST

2024-04-18 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17838773#comment-17838773 ] Julian Hyde commented on CALCITE-6375: -- [~jerin_john], Jira summaries should not use the word 'fix

[jira] [Commented] (CALCITE-6373) Distinct optimization produces broken sql query

2024-04-17 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17838248#comment-17838248 ] Julian Hyde commented on CALCITE-6373: -- I agree that any SQL-to-SQL transformation that alters

[jira] [Commented] (CALCITE-6367) Add timezone support for FORMAT clause in CAST (enabled in BigQuery)

2024-04-15 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17837465#comment-17837465 ] Julian Hyde commented on CALCITE-6367: -- [~jerin_john] Can you add links to any related cases

[jira] [Commented] (CALCITE-6363) Introduce a rule to derive more filters from inner join condition

2024-04-15 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17837462#comment-17837462 ] Julian Hyde commented on CALCITE-6363: -- Thanks for finding JoinConditionPushRule (CALCITE-5073

[jira] [Commented] (CALCITE-6363) Introduce a rule to derive more filters from inner join condition

2024-04-15 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17837461#comment-17837461 ] Julian Hyde commented on CALCITE-6363: -- On the mailing list [~jamesstarr] replied: {quote

Re: [jira] [Created] (CALCITE-6363) Introduce a rule to derive more filters from inner join condition

2024-04-15 Thread Julian Hyde
James, Thanks for chiming in. I added your comments to the jira case. Julian > On Apr 15, 2024, at 12:58 PM, James Starr wrote: > > The keyword I think you want is transitive filter pushdown. The reduce > expression rule handles some of the trivial cases outlined as examples. > Also, you

Re: Supporting ASOF JOIN

2024-04-15 Thread Julian Hyde
I would regard this as two separate but related things: a new SQL syntax for joins, and a new relational operator. It is definitely worth keeping them separate; the operator will not map 1-1 to the syntax, may require its input to input to be sorted, and of course we would want queries to be

[jira] [Commented] (CALCITE-6363) Introduce a rule to derive more filters from inner join condition

2024-04-14 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836988#comment-17836988 ] Julian Hyde commented on CALCITE-6363: -- I really wish you had logged the case BEFORE writing a big

[jira] [Commented] (CALCITE-6363) Introduce a rule to derive more filters from inner join condition

2024-04-14 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836987#comment-17836987 ] Julian Hyde commented on CALCITE-6363: -- Don’t we have some existing rules (metadata) for this? Can

Re: [DISCUSS] rename master branch to main

2024-04-14 Thread Julian Hyde
It would also be helpful to send instructions to this list to help contributors migrate their sandboxes. GitHub makes it fairly easy. > On Apr 14, 2024, at 3:40 AM, hans.van.akel...@gmail.com wrote: > > Hi All, > > Ticket has been created for the ASF infra team to rename the branches [1]. >

[jira] [Commented] (CALCITE-6357) Calcite enforces select arguments count to be same as row schema fields which causes aliases to be ignored

2024-04-11 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836338#comment-17836338 ] Julian Hyde commented on CALCITE-6357: -- We added {{RelRoot}} way back in CALCITE-819 to express

[jira] [Commented] (CALCITE-6358) Support all PostgreSQL 14 date/time patterns

2024-04-11 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836284#comment-17836284 ] Julian Hyde commented on CALCITE-6358: -- Yes, in theory, it might. I don't have time to look

[jira] [Commented] (CALCITE-6358) Support all PostgreSQL 14 date/time patterns

2024-04-11 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836261#comment-17836261 ] Julian Hyde commented on CALCITE-6358: -- In Calcite, the set of format elements is tied

[jira] [Commented] (CALCITE-6361) Uncollect.deriveUncollectRowType crashes if the input data is not a collection

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836014#comment-17836014 ] Julian Hyde commented on CALCITE-6361: -- By the way, please don't say 'crashes'. Say what error

[jira] [Commented] (CALCITE-6361) Uncollect.deriveUncollectRowType crashes if the input data is not a collection

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836013#comment-17836013 ] Julian Hyde commented on CALCITE-6361: -- Is there a SQL query that reproduces this problem? It's

[jira] [Commented] (CALCITE-6358) Support all PostgreSQL 14 date/time patterns

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17835918#comment-17835918 ] Julian Hyde commented on CALCITE-6358: -- [~njordan], To be clear, which functions are you proposing

[jira] [Commented] (CALCITE-6357) Calcite enforces select arguments count to be same as row schema fields which causes aliases to be ignored

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17835917#comment-17835917 ] Julian Hyde commented on CALCITE-6357: -- [~brachi_packter], It might be useful to reduce

[jira] [Commented] (CALCITE-6357) Calcite enforces select arguments count to be same as row schema fields which causes aliases to be ignored

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17835893#comment-17835893 ] Julian Hyde commented on CALCITE-6357: -- *Caveat*: I haven't read every word you've written above

[jira] [Commented] (CALCITE-6357) Calcite enforces select arguments count to be same as row schema fields which causes aliases to be ignored

2024-04-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17835833#comment-17835833 ] Julian Hyde commented on CALCITE-6357: -- If the number of fields does not match, that's probably

[jira] [Commented] (CALCITE-6345) Intervals with more than 100 years are not supported

2024-04-03 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6345?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17833710#comment-17833710 ] Julian Hyde commented on CALCITE-6345: -- Can you reframe this as a feature request, rather than

Re: Draft: board report for 2024 Q1

2024-04-03 Thread Julian Hyde
+1 Thank you. Great idea to mention the new ASF members. Julian > On Apr 3, 2024, at 3:11 AM, Benchao Li wrote: > > Hello, > > Below you can find a draft of this quarter's board report. I plan to > submit the final version next Tuesday (Apr 9, 2024). > > Please let me know if you have any

[jira] [Commented] (CALCITE-6321) Add copy(List constants) method to Window class.

2024-04-02 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17833029#comment-17833029 ] Julian Hyde commented on CALCITE-6321: -- [~mbudiu], Which commit fixed this? > Add copy(L

[jira] [Commented] (CALCITE-6343) AS alias operator strips MEASUREness from measures

2024-03-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17831113#comment-17831113 ] Julian Hyde commented on CALCITE-6343: -- Your PR looks fine, but I would like you to add a simple

[jira] [Commented] (CALCITE-6343) AS alias operator strips MEASUREness from measures

2024-03-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17831110#comment-17831110 ] Julian Hyde commented on CALCITE-6343: -- I edited the description, removing the explicit link. Note

[jira] [Updated] (CALCITE-6343) AS alias operator strips MEASUREness from measures

2024-03-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6343?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-6343: - Description: CALCITE-5869 introduced a change which removes MEASURE when inferring the return

[jira] [Commented] (CALCITE-6315) Support PostgreSQL TO_CHAR, TO_DATE, TO_TIMESTAMP

2024-03-22 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829930#comment-17829930 ] Julian Hyde commented on CALCITE-6315: -- OK, great, make sure that it tests all format elements

[jira] [Commented] (CALCITE-6338) RelMdCollation#project can return an incomplete list of collations

2024-03-21 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829666#comment-17829666 ] Julian Hyde commented on CALCITE-6338: -- Makes sense. I would mention 'in the presence of aliasing

[jira] [Commented] (CALCITE-5155) Custom time frames

2024-03-20 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-5155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829343#comment-17829343 ] Julian Hyde commented on CALCITE-5155: -- I'm not aware of any SQL systems that have a SQL syntax

Re: Registering a release

2024-03-20 Thread Julian Hyde
think this is required for a podling release. Reporter tool > only seems to worry about TLP releases. > > The releases have to be announced on mailing lists including this one > and the main ASF announce list. > > On Thu, 21 Mar 2024 at 00:30, Julian Hyde wrote: > > > > Can

Re: Need access to reporter.apache.org

2024-03-20 Thread Julian Hyde
ndation/board/reporting#how > > On 2024/03/20 00:33:14 Julian Hyde wrote: > > I know I need to do something to register the release. As an ASF member I > > know that I have the necessary karma to do this thing, and I’m happy to > > help. But I don’t recall what exactly I need

Re: Registering a release

2024-03-20 Thread Julian Hyde
; to go here: >> https://dist.apache.org/repos/dist/release/ >> Kind Regards, >> Justin >>> On 21 Mar 2024, at 10:29 am, Julian Hyde wrote: >>> >>> Can someone remind me where I go (whimsy or reporter, I presume) to >>> register a release from a p

Registering a release

2024-03-20 Thread Julian Hyde
Can someone remind me where I go (whimsy or reporter, I presume) to register a release from a podling (in this case SDAP)? In Calcite, a top-level project that deploys Java artifacts to Maven Central, the process has been straightforward. I believe I even received an email from Reporter sending

Re: Need access to reporter.apache.org

2024-03-19 Thread Julian Hyde
I know I need to do something to register the release. As an ASF member I know that I have the necessary karma to do this thing, and I’m happy to help. But I don’t recall what exactly I need to do. As RM, I think you should have received an email from Reporter? Julian > On Mar 19, 2024, at

[jira] [Commented] (CALCITE-6315) Support PostgreSQL TO_CHAR, TO_DATE, TO_TIMESTAMP

2024-03-19 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17828522#comment-17828522 ] Julian Hyde commented on CALCITE-6315: -- If you want to help, add some tests. For CALCITE-2980 I

[jira] [Commented] (CALCITE-6335) Quidem tests should allow specifying optimization passes to apply to programs

2024-03-18 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17828131#comment-17828131 ] Julian Hyde commented on CALCITE-6335: -- I agree. Currently we look over XML plans produced

[jira] [Commented] (CALCITE-2980) Implement the FORMAT clause of the CAST operator

2024-03-18 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-2980?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17828130#comment-17828130 ] Julian Hyde commented on CALCITE-2980: -- It's a judgment call for you to make. I think it's based

Re: Using quidem tests to validate optimization rules

2024-03-18 Thread Julian Hyde
You are correct. There is no mechanism, and one would be really useful. > On Mar 18, 2024, at 1:19 PM, Mihai Budiu wrote: > > Hello, > > I was looking around to see whether it's possible to use a quidem test to > exercise specific optimization rules, but I couldn't find anything. Is there >

Re: March report for Apache Baremaps

2024-03-18 Thread Julian Hyde
I have signed off. Baremaps seems to be doing well. I am pleased to see your work, Bertil, on automating releases. Thank you for the reminder, and sorry that I missed the earlier one… with the ASF annual meeting, March is a very heavy month for ASF email. Julian > On Mar 18, 2024, at 3:54 

[jira] [Commented] (CALCITE-6270) Support FORMAT in CAST from Numeric and BYTES to String (Enabled in BigQuery)

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827761#comment-17827761 ] Julian Hyde commented on CALCITE-6270: -- [~jerin_john], Please use Calcite's type names in jira

[jira] [Commented] (CALCITE-6327) getValidatedNodeTypeIfKnown should never throw

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827760#comment-17827760 ] Julian Hyde commented on CALCITE-6327: -- A test case would be useful

[jira] [Commented] (CALCITE-6327) getValidatedNodeTypeIfKnown should never throw

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827759#comment-17827759 ] Julian Hyde commented on CALCITE-6327: -- That sounds right. However, mutations of the AST make me

[jira] [Commented] (CALCITE-6317) Incorrect constant replacement when group keys are NULL

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827758#comment-17827758 ] Julian Hyde commented on CALCITE-6317: -- Since the root cause is {{RelMdPredicates}}, can you add

Re: Supporting a function with different capabilities depending on database

2024-03-16 Thread Julian Hyde
is available, then tries to resolve arguments and > return values. > > Thanks for the help. > > > From: Julian Hyde > Date: Monday, March 11, 2024 at 5:08 PM > To: dev@calcite.apache.org > Subject: Re: Supporting a function with different capabilities depending on > datab

[jira] [Commented] (CALCITE-6209) Long queries are failing with "java.net.SocketTimeoutException: Read timed out" after 3 minutes

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827756#comment-17827756 ] Julian Hyde commented on CALCITE-6209: -- Thank you for adding the link. And, thanks for the fix

Re: Incorrect Quidem test in agg.iq

2024-03-16 Thread Julian Hyde
I started investigating but didn't get a chance to follow up. I don't know whether Calcite or Postgres is correct. I suspect the difference is due to the expression 'comm is null'. Because I don't know whether GROUPING_ID's behavior is defined if its arguments are not all columns. But I think the

[jira] [Commented] (CALCITE-6314) Add RANDOM function (enabled in Postgres library)

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6314?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827755#comment-17827755 ] Julian Hyde commented on CALCITE-6314: -- Looks like you figured it out. By the way, when we're

[jira] [Commented] (CALCITE-2980) Implement the FORMAT clause of the CAST operator

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-2980?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827753#comment-17827753 ] Julian Hyde commented on CALCITE-2980: -- A good test is really valuable. I wouldn't disable

[jira] [Commented] (CALCITE-6332) Optimization CoreRules.AGGREGATE_EXPAND_DISTINCT_AGGREGATES_TO_JOIN produces incorrect results for aggregates with groupSets

2024-03-16 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827752#comment-17827752 ] Julian Hyde commented on CALCITE-6332: -- There are two algorithms to rewrite distinct aggregates

Re: [VOTE] Graduate Apache SDAP (Incubating) as a Top Level Project

2024-03-16 Thread Julian Hyde
ED, that the persons listed immediately below be and hereby are > appointed to serve as the initial members of the Apache SDAP Project: > > - Edward M Armstrong > - Nga Thien Chung > - Thomas Cram > - Frank Greguska > - Thomas Huang > - Julian Hyde > - Jose

[jira] [Commented] (CALCITE-6333) Queries with distinct aggregations with filter throw NPE when planned using joins

2024-03-15 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17827570#comment-17827570 ] Julian Hyde commented on CALCITE-6333: -- Your PR seems to make the NPE go away by removing

[jira] [Commented] (CALCITE-6324) Type inferred for result of STDDEV, VAR_SAMP, etc. is incorrect

2024-03-12 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825877#comment-17825877 ] Julian Hyde commented on CALCITE-6324: -- You can't say the type is 'incorrect' when the standard

[jira] [Commented] (CALCITE-6309) Add REGEXP_LIKE function (enabled in Postgres library)

2024-03-12 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825856#comment-17825856 ] Julian Hyde commented on CALCITE-6309: -- Thank you for doing this excellent research. I think

[jira] [Commented] (CALCITE-6309) Add REGEXP_LIKE function (enabled in Postgres library)

2024-03-12 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825839#comment-17825839 ] Julian Hyde commented on CALCITE-6309: -- I think it would be OK (and simpler) to make the third

[jira] [Commented] (CALCITE-6209) Long queries are failing with "java.net.SocketTimeoutException: Read timed out" after 3 minutes

2024-03-12 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825753#comment-17825753 ] Julian Hyde commented on CALCITE-6209: -- [~stoty], Please add a link to the commit on main branch

Re: Supporting a function with different capabilities depending on database

2024-03-11 Thread Julian Hyde
"...different capabilities depending on database” What exactly do you mean by “database”? If I create a Calcite model that has one schema backed by a Postgres JDBC catalog and another schema backed by a MySQL JDBC catalog, what “database” would you expect a connection to this model to have?

Re: Pull Request Reviews for Avatica

2024-03-11 Thread Julian Hyde
Mihai, Can you list the PRs (jira cases) in this category? I thought I suggested a solution. disable tests in Calcite based on the Avatica version number. And make the changes just before an Avatica release, and quickly upgrade Calcite, so that any tests you need to disable aren’t disabled

[jira] [Commented] (CALCITE-6319) Add negative test to pow and power functions

2024-03-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6319?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825094#comment-17825094 ] Julian Hyde commented on CALCITE-6319: -- None of the tests you added are negative tests. >

[jira] [Commented] (CALCITE-6319) Add negative test to pow and power functions

2024-03-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6319?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825093#comment-17825093 ] Julian Hyde commented on CALCITE-6319: -- It's much easier to read your case summary if you use

[jira] [Commented] (CALCITE-6278) Add REGEXP, REGEXP_LIKE function (enabled in Spark library)

2024-03-10 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17825091#comment-17825091 ] Julian Hyde commented on CALCITE-6278: -- Regarding the character literal syntax. That's

Re: [VOTE] Release Apache SDAP (incubating) 1.2.0-rc4

2024-03-09 Thread Julian Hyde
Forwarding my vote from the dev@ list: +1 (binding) The points made by Sebb and PJ seem valid but they do not affect this vote. Julian On Fri, Mar 8, 2024 at 2:25 PM sebb wrote: > > On Fri, 8 Mar 2024 at 22:17, PJ Fanning wrote: > > > > Hi Stepheny, > > > > Can you remove the old RCs from >

[jira] [Commented] (CALCITE-6278) Add REGEXP, REGEXP_LIKE function (enabled in Spark library)

2024-03-09 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824994#comment-17824994 ] Julian Hyde commented on CALCITE-6278: -- I think by 'unescaping' you mean handling backslashes

[jira] [Commented] (CALCITE-6278) Add REGEXP, REGEXP_LIKE function (enabled in Spark library)

2024-03-07 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824456#comment-17824456 ] Julian Hyde commented on CALCITE-6278: -- Can you expand on “Since Spark 2.0, string literals

[jira] [Commented] (CALCITE-6305) Increase coverage of unit tests for arrow adapter

2024-03-07 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6305?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824453#comment-17824453 ] Julian Hyde commented on CALCITE-6305: -- Should be “Arrow” not “arrow”. Sorry to be pedantic

[jira] [Commented] (CALCITE-6306) FILTER clauses for aggregate functions are not supported in MySQL, MariaDB and Starrocks

2024-03-07 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824451#comment-17824451 ] Julian Hyde commented on CALCITE-6306: -- Can you change the subject so it’s clear which calcite

Re: [VOTE] Release Apache SDAP (incubating) 1.2.0-rc4

2024-03-06 Thread Julian Hyde
Sounds to me as if the ‘hiccup’ is just a bug and can be addressed by documenting it in the release notes (i.e. without needing a new RC). The bigger issue addressed by the release is the need to remove all category X dependencies. That said, I’m not the RM so it’s not my call. +1 (binding)

[jira] [Commented] (CALCITE-6284) Invalid conversion triggers ClassCastException

2024-03-06 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824169#comment-17824169 ] Julian Hyde commented on CALCITE-6284: -- Thank you! > Invalid conversion trigg

[jira] [Commented] (CALCITE-6293) Support OR condition in arrow adapter

2024-03-06 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17824168#comment-17824168 ] Julian Hyde commented on CALCITE-6293: -- [~asolimando], +1 on merging what we have. The new tickets

[jira] [Updated] (CALCITE-6302) Release Calcite 1.37.0

2024-03-06 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6302?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-6302: - Fix Version/s: 1.37.0 > Release Calcite 1.3

Re: Override built-in function signature for query parsing

2024-03-05 Thread Julian Hyde
The parser does its work without looking at the operator table. (Like any sane parser, it deals only with syntax, not semantics.) So, it’s impossible for your TRIM function to affect what the parser considers to be valid. One option is for you to modify the Babel parser, which is a more lenient

Re: Reviewing blog repository

2024-03-05 Thread Julian Hyde
The web site has a ’news’ section which could be used for blog posts. I think it would be great to add this kind of content to the site (copy-pasted from your blog, and with a link to the original blog). People would find it more easily, and it would trust it because it is linked from the

[jira] [Commented] (CALCITE-6248) Illegal dates are accepted by casts

2024-03-05 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823772#comment-17823772 ] Julian Hyde commented on CALCITE-6248: -- Postel's law applies: "be conservative in what you

[jira] [Commented] (CALCITE-6284) Invalid conversion triggers ClassCastException

2024-03-05 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823770#comment-17823770 ] Julian Hyde commented on CALCITE-6284: -- Can you fill out the description? Describe what's going

[jira] [Commented] (CALCITE-6239) Add a PostGIS dialect that supports ST_ functions

2024-03-05 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823753#comment-17823753 ] Julian Hyde commented on CALCITE-6239: -- [~bchapuis], In my [6239-postgis-dialect|https

[jira] [Commented] (CALCITE-6292) Support more arrow type

2024-03-05 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823745#comment-17823745 ] Julian Hyde commented on CALCITE-6292: -- Arrow types are very similar to SQL and JDBC types. You

[jira] [Commented] (CALCITE-6293) Support OR condition in arrow adapter

2024-03-05 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823744#comment-17823744 ] Julian Hyde commented on CALCITE-6293: -- Can you provide more details? This case (and others you

Re: [ANNOUNCE] Sergey Nuyanzin joins Calcite PMC

2024-03-05 Thread Julian Hyde
Welcome, Sergey! Thank you for your efforts over many years in and around the Calcite community (including in SQLLine and Flink) making Calcite better and the community more friendly and efficient. Julian > On Mar 5, 2024, at 5:59 AM, Michael Mior wrote: > > Congratulations and welcome

[jira] [Commented] (CALCITE-6297) Support sub-queries in arrow adapter

2024-03-04 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6297?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823468#comment-17823468 ] Julian Hyde commented on CALCITE-6297: -- I don’t understand what this means. Add description

Re: [DISCUSS] Formal process for adding committers / PMC members

2024-03-04 Thread Julian Hyde
personally in favor of the two-tier solution as I think it would be > > more suitable for > > > growing our community. As Julian said it's a lower barrier to entry. > > > > > > On 2024/01/16 22:53:38 Julian Hyde wrote: > > > > In favor of one tier (every

[jira] [Commented] (CALCITE-6278) Add REGEXP、REGEXP_LIKE function (enabled in Spark library)

2024-03-04 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823342#comment-17823342 ] Julian Hyde commented on CALCITE-6278: -- Use a real comma in the subject, please. Does RLIKE have

[jira] [Resolved] (CALCITE-6288) Error while compiling LIKE function after update to 1.36

2024-03-04 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-6288. -- Resolution: Not A Problem Make sure that the column has SQL type {{{}MAP{}}}. Add some test

[jira] [Commented] (CALCITE-6289) View in union cannot be reused

2024-03-04 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823332#comment-17823332 ] Julian Hyde commented on CALCITE-6289: -- What would the plan look like if reuse was occurring

[jira] [Commented] (CALCITE-6288) Error while compiling LIKE function after update to 1.36

2024-03-04 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17823070#comment-17823070 ] Julian Hyde commented on CALCITE-6288: -- The SQL doesn’t seem to have a valid type. An item

Re: [VOTE] Move Arrow DataFusion Subproject to new Top Level Apache Project

2024-03-03 Thread Julian Hyde
+1 (binding) > On Mar 2, 2024, at 2:28 PM, Dewey Dunnington > wrote: > > +1 (binding) > >> On Sat, Mar 2, 2024 at 8:08 AM vin jake wrote: >> >> +1 (binding) >> >>> On Fri, Mar 1, 2024 at 7:33 PM Andrew Lamb wrote: >>> >>> Hello, >>> >>> As we have discussed[1][2] I would like to vote

[jira] [Commented] (CALCITE-6281) Add test infrastructure to ensure that Calcite is consistent with the SQL dialects it is trying to emulate

2024-02-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17820939#comment-17820939 ] Julian Hyde commented on CALCITE-6281: -- Work in progress, based on Stamatis' [PR 3704|https

[jira] [Assigned] (CALCITE-6281) Add test infrastructure to ensure that Calcite is consistent with the SQL dialects it is trying to emulate

2024-02-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6281?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde reassigned CALCITE-6281: Assignee: Julian Hyde > Add test infrastructure to ensure that Calcite is consist

Re: Simulataneous changes to Avatica and Calcite

2024-02-26 Thread Julian Hyde
In https://github.com/apache/calcite/commit/614b4350f558a4fabaafc06fef9689ec4c267994 you'll see the Stamatis upgraded Avatica and removed the "!if (fixed.calcite2776) {" wrapper in a Quidem test. That is a reasonably clean way to comment out tests pending a bug fix or an upgrade. Calcite release

[jira] [Resolved] (CALCITE-6219) Support SQL Validation for Tables with columns tagged as 'filter required'

2024-02-26 Thread Julian Hyde (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6219?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-6219. -- Fix Version/s: 1.37.0 Resolution: Fixed Fixed in [2558c13c|https://github.com/apache

  1   2   3   4   5   6   7   8   9   10   >