[jira] [Created] (CALCITE-6337) Distinguish naked measure support between inside and outside aggregation

2024-03-20 Thread Barry Kelly (Jira)
Barry Kelly created CALCITE-6337:


 Summary: Distinguish naked measure support between inside and 
outside aggregation
 Key: CALCITE-6337
 URL: https://issues.apache.org/jira/browse/CALCITE-6337
 Project: Calcite
  Issue Type: Improvement
  Components: core
Reporter: Barry Kelly


Measure type and AGGREGATE function (CALCITE-5105) introduced a configuration 
flag for naked measures.

Naked measures are measure-typed columns that are referenced outside an 
{{AGGREGATE()}} function call.

At Looker, we're trying to support a specific semantic using measure-typed 
columns:

- selecting from a naked measure-typed column outside an aggregating query 
evaluates to {{NULL}}
  This permits basic introspection of the schema like `{{SELECT * FROM foo 
LIMIT 1}}`
  For this, we need naked measures outside a grouping context.
- selecting from a naked measure-typed column inside an aggregating query is an 
error
  We want all expressions in a grouping query to either be part of the grouping 
key or to have
  an aggregation function applied. For this, we don't want naked measures 
inside a grouping context.

This change proposes:

- {{nakedMeasuresOutsideAggregatingQuery}} - boolean flag permitting measure 
references outside aggregating query
- {{nakedMeasuresInsideAggregatingQuery}} - boolean flag permitting measure 
references inside aggregating query
- deprecating {{nakedMeasures}} flag, which is now implemented by setting both 
of the above to the same value




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [DISCUSS] Towards Calcite 1.37.0

2024-03-20 Thread Jing Zhang
Thanks Benchao for driving the discussion.

I would like to be release manager for 1.39.0 version.
Besides, I could help to review some open pull requests.

--

Best,
Jing Zhang

James Duong  于2024年3月19日周二 01:14写道:

> I’ll help take a look at some of the open Avatica and Calcite PRs.
>
> From: Mihai Budiu 
> Date: Monday, March 18, 2024 at 9:48 AM
> To: dev@calcite.apache.org 
> Subject: Re: [DISCUSS] Towards Calcite 1.37.0
> This would be great, I am fixing two very basic bugs.
> Was busy with some other stuff, so I didn't have time to figure out how to
> submit the PRs so that they pass, if anyone has more detailed instructions
> I would be very happy. Otherwise I will dig into it.
>
> Mihai
> 
> From: Francis Chuang 
> Sent: Sunday, March 17, 2024 2:44 PM
> To: dev@calcite.apache.org 
> Subject: Re: [DISCUSS] Towards Calcite 1.37.0
>
> Hey everyone,
>
> I just wanted to jump in regarding CALCITE-6248 [1] and CALCITE-6282 (no
> corresponding PRs in the Calcite repo yet).
>
> The corresponding PRs in the Avatica repo are:
> - https://github.com/apache/calcite-avatica/pull/238
> - https://github.com/apache/calcite-avatica/pull/241
>
> As the issue will require changes to both Calcite and Avatica in a
> coordinated fashion, what do you guys think about releasing Avatica
> 1.25.0 first? I can be RM for the Avatica release.
>
> Mihai, can you please confirm if this approach works for you?
>
> Discussion for getting changes simultaneously into both Avatica and
> Calcite is here [2].
>
> Francis
>
> [1] https://github.com/apache/calcite/pull/3708
> [2] https://lists.apache.org/thread/5w3ry3zbm7671ffp4pxh7lx6cgc5r3tq
>
> On 13/03/2024 10:07 pm, Benchao Li wrote:
> > Thank you, Hongyu, for volunteering to be a release manager.
> >
> > What's worth to mention is that to be a release manager does not
> > require PMC membership nor being a committer, and it is a good chance
> > to be involved in community activities apart from code/discussion,
> > which would be a good additional credit for being considered as
> > committer or PMC member candidate.
> >
> > Hongyu Guo  于2024年3月13日周三 11:29写道:
> >>
> >> I would like to take on the role of RM for one release
> >>
> >> On Thu, Mar 7, 2024 at 2:58 AM Alessandro Solimando <
> >> alessandro.solima...@gmail.com> wrote:
> >>
> >>> Hey Sergey,
> >>> thanks for being the release manager for 1.37.0.
> >>>
> >>> I have added fixVersion=1.37.0 to CALCITE-2040
> >>> , as its
> associated PR
> >>>  is in good shape IMO,
> and we
> >>> are trying to get it done shortly with Hongyu.
> >>>
> >>> Best regards,
> >>> Alessandro
> >>>
> >>> On Wed, 6 Mar 2024 at 16:49, Sergey Nuyanzin 
> wrote:
> >>>
>  Updates about 1.37.0 releasing
> 
>  According to our Jira dashboard [1] and Github [2], there
>  are 17 pending issues that could / should be part of the release.
>  I'd propose to make a collective effort to try to clean up our 1.37.0
>  backlog and merge the PRs which are in a good state. I'd propose to
>  aim for about 2 weeks to release 1.37.0, this will give us about
>  some time to clean up pending PRs for the next version. What do you
> >>> think?
> 
>  And contributors, if you have any work that is in good shape and want
>  to be included in 1.37.0, please mark the fixVersion to 1.37.0, this
>  will inform the release manager, and we'll try our best to get it in.
> 
>  A quick update on the current status for issues targeted 1.37.0
>  We need more reviewers for #2
> 
>  #1. Issues already have reviewers, and seems promising to be merged
> >>> shortly
>  https://issues.apache.org/jira/browse/CALCITE-6138
>  https://issues.apache.org/jira/browse/CALCITE-6015
>  https://issues.apache.org/jira/browse/CALCITE-6283
>  https://issues.apache.org/jira/browse/CALCITE-5607
> 
>    #2. Issues has been reviewed, and needs another reviewer
>  https://issues.apache.org/jira/browse/CALCITE-3522
>  https://issues.apache.org/jira/browse/CALCITE-6071
>  https://issues.apache.org/jira/browse/CALCITE-6210
>  https://issues.apache.org/jira/browse/CALCITE-6259
>  https://issues.apache.org/jira/browse/CALCITE-2067
>  https://issues.apache.org/jira/browse/CALCITE-6226
>  https://issues.apache.org/jira/browse/CALCITE-6193
> 
> 
>  It could happen that I miss something, please let me know
>  if there is something with fixVersion 1.37.0 ready for review and not
>  mentioned here
> 
>  Also I created a JIRA issue for releasing 1.37.0
>  https://issues.apache.org/jira/browse/CALCITE-6302
> 
>  [1]
> 
> >>>
> https://issues.apache.org/jira/secure/Dashboard.jspa?selectPageId=12333950
>  [2] https://github.com/apache/calcite/pulls
> 
>  On Fri, Feb 23, 2024 at 9:24 PM Julian Hyde 
>  wrote:
> 
> > +1