Hitoshi Harada wrote:
> I'm glad to hear that. Actually thanks to git it is quite easy for me
> to merge my own repository with the HEAD. It tells me which lines are
> new coming and which lines I modified are newer than else in CVS. This
> is my first project where I use git (and I am not guru of cvs either
> -:) but I love it now.
> So feel free to tell me to recreate a new patch against head. Not so
> heavy work as making on-the-fly script.
>

CUME_DIST and COUNT(*) seem to be working as expected now with the original
tests.  I'll continue to test more and also test the other windowing
functions that I've not got to yet.

I did receive a small warning from the compiler when compiling nodeWindow.c.
It was complaining about a possible un-initialised f_shrinking. Just to keep
it quiet I changed the break's to return's after the elog() call.



*** nodeWindow.c        2008-11-09 10:54:50.000000000 +0000
--- nodeWindow.c        2008-11-09 10:39:24.000000000 +0000
***************
*** 818,824 ****
                        case FRAME_CURRENT_RANGE:
                                /* UNSUPPORTED */
                                elog(ERROR, "unknown preceding type %d",
node->preceding_type);
!                               break;
                        case FRAME_VALUE_ROWS:
                                if (node->preceding_rows > 0)
                                        f_shrinking = 0;
--- 818,824 ----
                        case FRAME_CURRENT_RANGE:
                                /* UNSUPPORTED */
                                elog(ERROR, "unknown preceding type %d",
node->preceding_type);
!                               return; /* keep compiler quiet */
                        case FRAME_VALUE_ROWS:
                                if (node->preceding_rows > 0)
                                        f_shrinking = 0;
***************
*** 922,928 ****
                        case FRAME_CURRENT_RANGE:
                                /* UNSUPPORTED */
                                elog(ERROR, "unknown preceding type %d",
node->preceding_type);
!                               break;
                        case FRAME_VALUE_ROWS:
                                if (node->preceding_rows <=
winobj->p_currentpos + 1)
                                        f_shrinking = 1;
--- 922,928 ----
                        case FRAME_CURRENT_RANGE:
                                /* UNSUPPORTED */
                                elog(ERROR, "unknown preceding type %d",
node->preceding_type);
!                               return; /* keep compiler quiet */
                        case FRAME_VALUE_ROWS:
                                if (node->preceding_rows <=
winobj->p_currentpos + 1)
                                        f_shrinking = 1;




-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to