Re: ctidscan as an example of custom-scan (Re: [v9.5] Custom Plan API)

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: ctidscan as an example of custom-scan (Re: [v9.5] Custom Plan API)
Дата
Msg-id 20150714190709.GB2301@postgresql.org
обсуждение исходный текст
Ответ на Re: ctidscan as an example of custom-scan (Re: [v9.5] Custom Plan API)  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: ctidscan as an example of custom-scan (Re: [v9.5] Custom Plan API)  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Robert Haas wrote:

> We don't have anything that currently tests the Custom Scan interface
> in the tree.  The question is how important that is, and whether it's
> worth having what's basically a toy implementation just to demonstrate
> that the feature can work.  If so, I think ctidscan is as good a toy
> example as any; in the interest of full disclosure, I was the one who
> suggested it in the first place.  But I am not entirely sure it's a
> good idea to saddle ourselves with that maintenance effort.  It would
> be a lot more interesting if we had an example that figured to be
> generally useful.

As a general principle, I think it's a good idea to have a module that's
mostly just a skeleton that guides people into writing something real to
use whatever API is being tested.  It needs to be simple enough that not
much need to be deleted when writing the real thing, and complex enough
to cover the parts that need covering.  If whatever replaces ctidscan is
too complex, it will not serve that purpose.

My guess is that something whose only purpose is to test the custom scan
interface for coverage purposes can be simpler than this module.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: Could be improved point of UPSERT
Следующее
От: Dean Rasheed
Дата:
Сообщение: Re: RLS fails to work with UPDATE ... WHERE CURRENT OF