On Fri, 2013-01-18 at 10:00 -0500, Stephen Frost wrote:
> I've done another review of this patch and it looks pretty good to me.
> My only complaint is that there isn't a single comment inside
> makeRecursiveViewSelect().
Added some of that and committed.
> One other thought is- I'm guessing this isn't going to work:
>
> CREATE RECURSIVE VIEW name (columns) AS WITH ... SELECT ...;
>
> Does the spec explicitly allow or disallow that? Should we provide any
> comments about it?
That works fine, AFAICT. It just becomes another level of WITH.