Re: Doc patch: add RECURSIVE to bookindex

2018-08-09 Thread Alvaro Herrera
On 2018-Aug-01, Fabien COELHO wrote: > Hello Daniel, > > > Patch applies cleanly, doc build ok, works for me. > > I have added it to the next CF and marked it as ready. Pushed, thanks. I applied it to 11 too. I would have added it even further back, but it didn't apply cleanly. How about an

Re: Doc patch: add RECURSIVE to bookindex

2018-08-01 Thread Fabien COELHO
Hello Daniel, Patch applies cleanly, doc build ok, works for me. I have added it to the next CF and marked it as ready. -- Fabien.

Re: Doc patch: add RECURSIVE to bookindex

2018-07-31 Thread Fabien COELHO
Why referencing only create_view, but not delete, insert, update, select or select_into where RECURSIVE is also used? ISTM that at least the select page should be referenced, I'm less sure of the others because there it appears only in the synopsys. Looking at other occurrences of , it seems

Re: Doc patch: add RECURSIVE to bookindex

2018-07-31 Thread Daniel Verite
Fabien COELHO wrote: > Why referencing only create_view, but not delete, insert, update, select > or select_into where RECURSIVE is also used? > > ISTM that at least the select page should be referenced, I'm less sure of > the others because there it appears only in the synopsys. Looki

Re: Doc patch: add RECURSIVE to bookindex

2018-07-31 Thread Fabien COELHO
Hello Daniel, I've noticed that RECURSIVE as a term is not in the index, and thought it should be. PFA a patch to add it with references to WITH queries and CREATE VIEW. Sounds reasonable. Why referencing only create_view, but not delete, insert, update, select or select_into where RECURS