2011/7/1 Shigeru Hanada <shigeru.han...@gmail.com>: > 2011/6/30 Alvaro Herrera <alvhe...@commandprompt.com>: >> Excerpts from 花田 茂's message of jue jun 30 06:00:23 -0400 2011: >> >>> I attached a patch which fixes file_fdw to check required option >>> (filename) in its validator function. I think that such requirement >>> should be checked again in PlanForeignScan(), as it had been so far. >>> Note that this patch requires fdw.patch has been applied. >> >> I think it'd be good to keep the error check in fileGetOptions() just to >> ensure that we don't crash in case a catalog is messed up with, though >> I'd degrade it to elog(ERROR) from ereport. > > Thanks for the comments. Please find attached a patch. Now file_fdw > validates filename in: > > * file_fdw_validator(), to catch lack of required option at DDL > * fileGetOptions(), to avoid crash caused by corrupted catalog > > I used ereport for the former check, because maybe such error usually > happens and is visible to users. This criteria was taken from the > document "Reporting Errors Within the Server". > http://developer.postgresql.org/pgdocs/postgres/error-message-reporting.html
Do we want to apply this to 9.1 before beta3? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers