Magnus Hagander wrote:
Alvaro Herrera wrote:
Stefan Kaltenbrunner escribió:
yeah - the test install is available on
http://reviewdemo.postgresql.org if people want to test judge for
themself - contact magnus or me if you need permissions to do/test
stuff there.
Thanks. I tried submitting a review request against anoncvs but it
failed with
No valid separator after the filename was found in the diff header
"patch" can apply the patch correctly -- I'm not sure what does this
patch have that RB does not like.
The patch is attached in case you want to play with it.
If I'm not mistaken, I read somewhere in the docs that the diff has to
be unified, not context. Could that be it?
Yes, and second problem is CVS diff. When you use CVS diff you must
strip root (/home/alvherre/Code/cvs/) from following line:
RCS file: /home/alvherre/Code/cvs/pgsql/src/backend/commands/analyze.c,v
and ",v" as well. After that RB accepted your patch.
Zdenek
---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match
