Re: range_agg

From: Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Jeff Davis <pgsql(at)j-davis(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, David Fetter <david(at)fetter(dot)org>
Subject: Re: range_agg
Date: 2020-03-14 18:13:54
Message-ID: CA+renyXBduODHi34ASUZrQA=Yza-q6WXdrMa102soxAfw-Be0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Mar 13, 2020 at 10:06 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> writes:
> > On Wed, Mar 11, 2020 at 4:39 PM Paul A Jungwirth
> > <pj(at)illuminatedcomputing(dot)com> wrote:
> >> Oh, my last email left out the most important part. :-) Is this
> >> failure online somewhere so I can take a look at it and fix it?
>
> Look for your patch(es) at
>
> http://commitfest.cputube.org
>
> Right now it's not even applying, presumably because Alvaro already
> pushed some pieces, so you need to rebase. But when it was applying,
> one or both of the test builds was failing.

Here are all Alvaro's changes rolled into one patch, along with the
get_multirange_io_data parameter renamed to mltrngtypid, and this
small fix to the dependency regression test:
diff --git a/src/test/regress/expected/dependency.out
b/src/test/regress/expected/dependency.out
index 778699a961..8232795148 100644
--- a/src/test/regress/expected/dependency.out
+++ b/src/test/regress/expected/dependency.out
@@ -140,8 +140,8 @@ owner of sequence deptest_a_seq
owner of table deptest
owner of function deptest_func()
owner of type deptest_enum
-owner of type deptest_range
owner of type deptest_multirange
+owner of type deptest_range
owner of table deptest2
owner of sequence ss1
owner of type deptest_t

I think that should fix the cfbot failure.

Yours,
Paul

Attachment Content-Type Size
v13-multiranges.patch application/octet-stream 326.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message James Coleman 2020-03-14 18:41:09 Re: [PATCH] Incremental sort (was: PoC: Partial sort)
Previous Message Julien Rouhaud 2020-03-14 17:53:51 Re: Feature improvement: can we add queryId for pg_catalog.pg_stat_activity view?