Thread

1. new Group BY code

   Vadim Mikheev <vadim@sable.krasnoyarsk.su> — 1998-02-18T12:29:24Z

   I hope that GROUP BY fixed.
   Unfortunately, I can't get any errors under FreeBSD 
   when using GROUP BY with aggregates. I only got core
   for something like this: select y, z from tgrp group by y, z.
   Two different funcs are used for these two cases and so 
   please test new code on your platforms.
   Please test with different -S XXX to check that all works
   for both in-memory and on-disk sorts (you'll see pg_psort.xxxx
   files in last case).
   
   New code are for 6.3. This is not patch (sorry, I lost orig version
   of execnodes.h...) and has to be un-tarred in src dir.
   
   If someone hasn't 6.3 - ok, I'll post patch for 6.2.1 tomorrow.
   
   CVS updated.
   
   Vadim

2. Re: [HACKERS] new Group BY code

   Bruce Momjian <maillist@candle.pha.pa.us> — 1998-02-18T15:10:25Z

   > 
   > This is a multi-part message in MIME format.
   > --------------0478E3EE3CFC638DB65D6A96
   > Content-Type: text/plain; charset=us-ascii
   > Content-Transfer-Encoding: 7bit
   > 
   > I hope that GROUP BY fixed.
   > Unfortunately, I can't get any errors under FreeBSD 
   > when using GROUP BY with aggregates. I only got core
   > for something like this: select y, z from tgrp group by y, z.
   > Two different funcs are used for these two cases and so 
   > please test new code on your platforms.
   > Please test with different -S XXX to check that all works
   > for both in-memory and on-disk sorts (you'll see pg_psort.xxxx
   > files in last case).
   > 
   > New code are for 6.3. This is not patch (sorry, I lost orig version
   > of execnodes.h...) and has to be un-tarred in src dir.
   > 
   > If someone hasn't 6.3 - ok, I'll post patch for 6.2.1 tomorrow.
   
   6.3 is only ten days away.  I suggest you forget a 6.2.1 patch if it is
   any trouble.
   
   -- 
   Bruce Momjian
   maillist@candle.pha.pa.us
   
   

3. Re: [HACKERS] new Group BY code

   Vadim Mikheev <vadim@sable.krasnoyarsk.su> — 1998-02-19T01:13:04Z

   Bruce Momjian wrote:
   > 
   > >
   > > If someone hasn't 6.3 - ok, I'll post patch for 6.2.1 tomorrow.
   > 
   > 6.3 is only ten days away.  I suggest you forget a 6.2.1 patch if it is
   > any trouble.
   
   I'm not sure that I will upgrade my production system to 6.3 in near feature ->
   I need in 6.2.1 patch for myself. And - it's easy.
   
   Vadim
   
   

4. Re: [HACKERS] new Group BY code

   Ronald Baljeu <rjb@xs4all.nl> — 1998-02-19T12:04:04Z

   > I hope that GROUP BY fixed.
   > Unfortunately, I can't get any errors under FreeBSD 
   > when using GROUP BY with aggregates. I only got core
   > for something like this: select y, z from tgrp group by y, z.
   > Two different funcs are used for these two cases and so 
   > please test new code on your platforms.
   > Please test with different -S XXX to check that all works
   > for both in-memory and on-disk sorts (you'll see pg_psort.xxxx
   > files in last case).
   > 
   > New code are for 6.3. This is not patch (sorry, I lost orig version
   > of execnodes.h...) and has to be un-tarred in src dir.
   > 
   > If someone hasn't 6.3 - ok, I'll post patch for 6.2.1 tomorrow.
   
   Vadim,
   
   I downloaded the snapshot of Feb 18 and tried it. I noticed it's much
   harder indeed to reproduce the bug (as compared to 6.2.1p6), but
   occasionaly the bug appeared.
   
   After installing your code the bug disappeared completely. I also tried
   a couple of group-by queries on my production database, and tried both
   in-memory sorting and on-disk sorting, but everything seems ok now.
   
   So I guess congratulations are in order.
   
   Thanks!
   
   Cheers,
   Ronald