Re: [PATCHES] [HACKERS] Interval aggregate regression failure

[Michael Glaesemann]

On Sep 5, 2006, at 10:14 , Bruce Momjian wrote:

Advertising

Bruce Momjian wrote:

OK, updated patch. It will fix the >=24:00:00 case because it cascades

up if the remainder number of seconds is greater or equal to one day.
One open item is that it still might show >24 hours if the seconds

that should be handled or not.  If you fix that, you really are
cascading up because the resulting seconds might be less than the

up would be 1 day, 01:00:00.  I am unsure we want to do that.  Right
now, this will show 25:00:00.

computation combined with the remaning seconds >24 hours. Not sure if computed value, e.g. result is 23:00:00, remainder is 02:00:00, cascade

Updated patch that uses TSROUND for partial month and days.  Michael
says the tests look good on his system.  I think we are done with this
bug.  :-)

Please find attached regression tests for this patch.

Michael Glaesemann
grzm seespotcode net

17interval_muldiv_0905T2053+0900.diff
Description: Binary data

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend