On 23 September 2010 11:31, Boxuan Zhai <bxzhai2010@gmail.com> wrote:
> Dear All,
> I have just generate a new patch of MERGE command.
> One main change in this edition is the removal of RASIE ERROR action from
> MEREG, because its semantics is not well defined yet.
> I also rewrote the regress test file merge.sql, to make it consistent with
> the examples I used in my wiki page.
> Some little (error and warning) bugs are fixed.
> In this patch, all the main features of MERGE (sublinks, explain, rule and
> trigger, inheritance) are not changed. And so is the DO NOTHING action.
> I do wish the MERGE command can be added into psql 9.1. And I wonder what
> improvement should be made on current edition.
> Could you please have a review on this patch, if you have time and interest?
> Your feedback will be highly appreciated.
> Thanks
>
> Yours Boxuan
A few corrections:
in src/backend/executor/nodeModifyTable.c: s/excute/execute/ s/orignial/original/
in src/backend/optimizer/plan/planner.c s/expreesions/expressions/ s/So,we/So, we/ s/comand/command/
s/fileds/fields/
in src/backend/optimizer/prep/preptlist.c: s/aggresive/aggressive/
in src/backend/optimizer/util/var.c s/targe/target/ -- this appears twice s/sourse/source/
in src/backend/parser/analyze.c s/takend/taken/ s/relaion/relation/ s/targe/target/ -- this appears twice
s/consitency/consistency/ s/commond/command/ s/seperate/separate/
in src/backend/rewrite/rewriteHandler.c s/acton/action/
in src/include/nodes/execnodes.h s/meger/merge/
in src/include/nodes/parsenodes.h s/proecess/process/ s/allwo/allow/ s/elments/elements/
in src/test/regress/expected/merge.out s/qulifications/qualifications/ -- this appears twice s/suceeds/succeeds/ --
thisappears twice
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935