On 2017/01/03 15:57, Ashutosh Bapat wrote:
The patch looks good to me, but I feel there are too many testscases.
Now that we have changed the approach to invalidate caches in all
cases, should we just include cases for SELECT or UPDATE or INSERT or
DELETE instead of each statement?

I don't object to that, but (1) the tests I added wouldn't be that time-consuming, and (2) they would be more expected to help find bugs, in general, so I'd vote for keeping them. How about leaving that for the committer's judge?

Thanks for the review!

Best regards,
Etsuro Fujita

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to