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 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
15 matches
Mail list logo