Hello Robert,
Done.
Great!
I removed some of the new error-reporting stuff because (1) I wasn't sure it was right
Hmmm. Although I'm not sure it was right, I'm sure that not enough error reporting is too rough:
sh> ./pgbench -f bad.sql syntax error at column 25 set: parse error
and (2) it touched more places than just the ones directly relevant to the patch.
It just tried to report errors consistently from all places.
I think those things can be resubmitted as a separate patch,
Yep, I'll do that.
but I'd like to have a more robust discussion about what we want the error reporting to look like rather than just sliding it into this patch.
As an input, I suggest that the error reporting feature should provide a clue about where the issue is, including a line number and possibly the offending line. Not sure what else is needed.
-- Fabien. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers