On 2015-07-17 19:57:22 +0100, Andrew Gierth wrote:
Attached is the current version of my fix (with Jeevan's regression
tests plus one of mine).
Pushed, thanks for the report and fix!
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
On 2015-07-17 11:37:26 +0530, Jeevan Chalke wrote:
However I wonder why we are supporting GROUPING SETS inside GROUPING SETS.
On Oracle, it is throwing an error.
We are not trying to be Oracle compatible, but just curious to know.
The SQL specification seems to be pretty unambigous about
Hello,
At Mon, 20 Jul 2015 15:45:21 +0530, Jeevan Chalke
jeevan.cha...@enterprisedb.com wrote in
CAM2+6=X9QWgbjJrR-dcLXh-RvvpGy=9enhuoghzrxhcj2kv...@mail.gmail.com
On Sat, Jul 18, 2015 at 12:27 AM, Andrew Gierth and...@tao11.riddles.org.uk
wrote:
Kyotaro == Kyotaro HORIGUCHI
On Sat, Jul 18, 2015 at 12:27 AM, Andrew Gierth and...@tao11.riddles.org.uk
wrote:
Kyotaro == Kyotaro HORIGUCHI horiguchi.kyot...@lab.ntt.co.jp
writes:
Kyotaro Hello, this looks to be a kind of thinko. The attached patch
Kyotaro fixes it.
No, that's still wrong. Just knowing that there
On Wed, Jul 15, 2015 at 10:21 PM, Andrew Gierth and...@tao11.riddles.org.uk
wrote:
Jeevan == Jeevan Chalke jeevan.cha...@enterprisedb.com writes:
Jeevan Hi,
Jeevan It looks like we do support nested GROUPING SETS, I mean Sets
Jeevan withing Sets, not other types. However this nesting
Hello, this looks to be a kind of thinko. The attached patch
fixes it.
===
According to the comment of transformGroupingSet, it assumes that
the given GROUPING SETS node is already flatted out and
flatten_grouping_sets() does that. The details of the
transformation is described in the comment for
Kyotaro == Kyotaro HORIGUCHI horiguchi.kyot...@lab.ntt.co.jp writes:
Kyotaro Hello, this looks to be a kind of thinko. The attached patch
Kyotaro fixes it.
No, that's still wrong. Just knowing that there is a List is not enough
to tell whether to concat it or append it.
Jeevan's original
Jeevan == Jeevan Chalke jeevan.cha...@enterprisedb.com writes:
Jeevan Hi,
Jeevan It looks like we do support nested GROUPING SETS, I mean Sets
Jeevan withing Sets, not other types. However this nesting is broken.
Good catch, but I'm not yet sure your fix is correct; I'll need to look
into
Hi,
It looks like we do support nested GROUPING SETS, I mean Sets withing
Sets, not other types. However this nesting is broken.
Here is the simple example where I would expect three rows in the
result. But unfortunately it is giving unrecognized node type
error. Which is something weird and
http://www.postgresql.org/list/pgsql-hackers/since/200905171950Hello,
Am looking for the patch related to 'Implementation of GROUPING SETS'.
Where can get this from?
Related thread:
http://www.postgresql.org/message-id/162867790905121420p7c910054x24d8e327abd58...@mail.gmail.com
Regards...
Hello
2013/7/4 Dev Kumkar devdas.kum...@gmail.com:
Hello,
Am looking for the patch related to 'Implementation of GROUPING SETS'.
Where can get this from?
Related thread:
http://www.postgresql.org/message-id/162867790905121420p7c910054x24d8e327abd58...@mail.gmail.com
I don't work on this
On Thu, Jul 4, 2013 at 6:31 PM, Pavel Stehule pavel.steh...@gmail.comwrote:
Hello
I don't work on this topic now, and my code is not usable for production.
Ok, no problem. Will await for any other pointers regarding any related
patch here.
Currently using UNION to archive similar results
On Thu, Jul 4, 2013 at 6:56 PM, Dev Kumkar devdas.kum...@gmail.com wrote:
On Thu, Jul 4, 2013 at 6:31 PM, Pavel Stehule pavel.steh...@gmail.com
wrote:
Hello
I don't work on this topic now, and my code is not usable for production.
Ok, no problem. Will await for any other pointers
On Thu, Jul 4, 2013 at 7:53 PM, Atri Sharma atri.j...@gmail.com wrote:
On Thu, Jul 4, 2013 at 6:56 PM, Dev Kumkar devdas.kum...@gmail.com
wrote:
Ok, no problem. Will await for any other pointers regarding any related
patch here.
Currently using UNION to archive similar results but
40 hours or more like 5000 hours, in your estimate?
Thanks.
--
Regards
David
-Original Message-
From: Pavel Stehule [mailto:pavel.steh...@gmail.com]
Sent: Monday, September 19, 2011 10:45 PM
To: edwbro...@gmail.com
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] Grouping Sets
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] Grouping Sets
Hello
2011/9/20 David Rinaldi edwbro...@gmail.com:
Paul,
I was able to apply the patch to 9.0.4 and so far looks good. My Oracle
results match. Nice.
But, when trying to calculate some percentages and control some
On Sun, Sep 18, 2011 at 02:08:01PM -0500, David Rinaldi wrote:
I tried to apply the Grouping Sets Patch to 8.4, but received several Hunks
failed messages, does anyone know if the failing hunks can be applied
manually? Or what version they were applied to specifically?
Your best bet is
83. 83
Thanks
--
Regards
David
-Original Message-
From: Pavel Stehule [mailto:pavel.steh...@gmail.com]
Sent: Sunday, September 18, 2011 2:34 PM
To: David Rinaldi
Subject: Re: [HACKERS] Grouping Sets
Hello
A last patch should be applied on 8.4 or 9.0 - should to try it. I
18000 15000 0.8333 83. 83
Thanks
--
Regards
David
-Original Message-
From: Pavel Stehule [mailto:pavel.steh...@gmail.com]
Sent: Sunday, September 18, 2011 2:34 PM
To: David Rinaldi
Subject: Re: [HACKERS] Grouping Sets
Hello
A last patch should be applied
Hi,
I tried to apply the Grouping Sets Patch to 8.4, but received several Hunks
failed messages, does anyone know if the failing hunks can be applied
manually? Or what version they were applied to specifically?
--
Regards
David
Hello
I have not any newest patch related to GROUPING SETS. The last version
of this patch is probably correct, but it is not well tested.
Actually, this patch has not quality to production usage :(. It is
just concept. You can test it.
Regards
Pavel Stehule
2011/6/18 Mariano Mara
Hi hackers (and specially Pavel Stehule),
I could really use the grouping set feature for some complex queries I'm
migrating from other db vendor. If my WEB searching is precise, this
wiki page [1] and this thread[2] are the last updates on the subject.
I'm willing to test how these functions
Hello
I found a break in GROUPING SETS implementation. Now I am playing with
own executor and planner node and I can't to go forward :(. Probably
this feature will need a significant update of our agg implementation.
Probably needs a some similar structure like CTE but it can be a
little bit
Hello
I was confused when I though so I found a solution of 1 shift/reduce conflict :(
All identificators used for buildin functions have to be a
col_name_keywords or reserved keyword. There is conflict with our
(probably obsolete) feature SELECT colname(tabname). So for this
moment the real
On Mon, Aug 09, 2010 at 10:59:26PM +0200, Pavel Stehule wrote:
Hello
I fixed an issues with empty sets. It just work, but there are some ugly
hacks.
It's really needs own planner node - now grouping functions are not
supported by ORDER BY clause.
I haven't made it through the last
2010/8/10 Joshua Tolley eggyk...@gmail.com:
On Mon, Aug 09, 2010 at 10:59:26PM +0200, Pavel Stehule wrote:
Hello
I fixed an issues with empty sets. It just work, but there are some ugly
hacks.
It's really needs own planner node - now grouping functions are not
supported by ORDER BY
On Thu, Aug 05, 2010 at 04:46:51PM +0200, Pavel Stehule wrote:
I am sending a updated version.
I've been looking at the changes to gram.y, and noted the comment under
func_expr
where you added CUBE and ROLLUP definitions. It says that CUBE can't be a
reserved keyword because it's already used
2010/8/7 Joshua Tolley eggyk...@gmail.com:
On Thu, Aug 05, 2010 at 04:46:51PM +0200, Pavel Stehule wrote:
I am sending a updated version.
I've been looking at the changes to gram.y, and noted the comment under
func_expr
where you added CUBE and ROLLUP definitions. It says that CUBE can't be
On Thu, Aug 05, 2010 at 04:46:51PM +0200, Pavel Stehule wrote:
So Joshua, can you look on code?
Sure... thanks :)
--
Joshua Tolley / eggyknap
End Point Corporation
http://www.endpoint.com
signature.asc
Description: Digital signature
I found other issue :(
postgres=# select name, place from cars group by grouping sets(name, place,());
name | place
---+
bmw |
skoda |
opel |
| germany
| czech rep.
skoda | czech rep.
skoda | germany
bmw | czech rep.
bmw | germany
opel | czech
On Wed, Aug 04, 2010 at 04:44:05AM +0200, Pavel Stehule wrote:
Yeah, I seem to have done a poor job of producing the patch based on the
repository I was working from. That said, it seems Pavel's working actively
on
a patch anyway, so perhaps my updating the old one isn't all that
2010/8/4 Joshua Tolley eggyk...@gmail.com:
On Wed, Aug 04, 2010 at 04:44:05AM +0200, Pavel Stehule wrote:
Yeah, I seem to have done a poor job of producing the patch based on the
repository I was working from. That said, it seems Pavel's working
actively on
a patch anyway, so perhaps my
On Thu, Aug 05, 2010 at 06:21:18AM +0200, Pavel Stehule wrote:
I hope, so next week you can do own work on this job - I am not a
native speaker, and my code will need a checking and fixing comments
I haven't entirely figured out how the code in the old patch works, but I
promise I *can* edit
Hello
2010/8/3 Joshua Tolley eggyk...@gmail.com:
In case anyone's interested, I've taken the CTE-based grouping sets patch from
[1] and made it apply to 9.1, attached. I haven't yet done things like checked
it for whitespace consistency, style conformity, or anything else, but (tuits
2010/8/3 Pavel Stehule pavel.steh...@gmail.com:
Hello
2010/8/3 Joshua Tolley eggyk...@gmail.com:
In case anyone's interested, I've taken the CTE-based grouping sets patch
from
[1] and made it apply to 9.1, attached. I haven't yet done things like
checked
it for whitespace consistency,
2010/8/3 Hitoshi Harada umi.tan...@gmail.com:
2010/8/3 Pavel Stehule pavel.steh...@gmail.com:
Hello
2010/8/3 Joshua Tolley eggyk...@gmail.com:
In case anyone's interested, I've taken the CTE-based grouping sets patch
from
[1] and made it apply to 9.1, attached. I haven't yet done things
On Mon, Aug 02, 2010 at 11:50:00PM -0600, Josh Tolley wrote:
In case anyone's interested, I've taken the CTE-based grouping sets
patch from [1] and made it apply to 9.1, attached. I haven't yet
done things like checked it for whitespace consistency, style
conformity, or anything else, but
On Tue, Aug 03, 2010 at 12:58:03PM -0700, David Fetter wrote:
On Mon, Aug 02, 2010 at 11:50:00PM -0600, Josh Tolley wrote:
In case anyone's interested, I've taken the CTE-based grouping sets
patch from [1] and made it apply to 9.1, attached. I haven't yet
done things like checked it for
2010/8/3 Joshua Tolley eggyk...@gmail.com:
On Tue, Aug 03, 2010 at 12:58:03PM -0700, David Fetter wrote:
On Mon, Aug 02, 2010 at 11:50:00PM -0600, Josh Tolley wrote:
In case anyone's interested, I've taken the CTE-based grouping sets
patch from [1] and made it apply to 9.1, attached. I
In case anyone's interested, I've taken the CTE-based grouping sets patch from
[1] and made it apply to 9.1, attached. I haven't yet done things like checked
it for whitespace consistency, style conformity, or anything else, but (tuits
permitting) hope to figure out how it works and get it closer
Hello
I got a possibility to continue on GROUPING SETS see
http://wiki.postgresql.org/wiki/Grouping_Sets
My last patch was based on an sharing functionality with non recursive
CTE. I would to recapitulate all options here and I would to prepare
this patch for next commitfest.
a) using
41 matches
Mail list logo
