Ashutosh Sharma <ashu.coe...@gmail.com> writes: > Attached is the patch that allows us to create view on a table without > columns. I've also added some test-cases for it in create_view.sql. > Please have a look and let me know your opinion.
Haven't read the patch, but a question seems in order here: should we regard this as a back-patchable bug fix? The original example shows that it's possible to create a zero-column view in existing releases, which I believe would then lead to dump/reload failures. So that seems to qualify as a bug not just a missing feature. On the other hand, given the lack of field complaints, maybe it's not worth the trouble to back-patch. I don't have a strong opinion either way. BTW, has anyone checked on what the matview code paths will do? Or SELECT INTO? regards, tom lane