On 12.01.2016 02:31, Alvaro Herrera wrote:
I gave a quick look through the patch and noticed a few minor things
while trying to understand it.
I think the test corpus isn't particularly interesting for how big it
is. I'd rather have (a) a small corpus (say 100 words) with which to do
detailed regression testing, and (b) some larger document for more
extensive testing. I'm not sure (b) is actually necessary.
Overall I think the new functions could stand a lot more commentary.
Thank you for a review. I will send fixed patch in a few days.
--
Artur Zakirov
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
