Re: pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit() - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit()
Date
Msg-id 1964.1458234712@sss.pgh.pa.us
Whole thread Raw
In response to pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit()  (Teodor Sigaev <teodor@sigaev.ru>)
Responses Re: pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit()  (Teodor Sigaev <teodor@sigaev.ru>)
Re: pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit()  (Teodor Sigaev <teodor@sigaev.ru>)
List pgsql-committers
Teodor Sigaev <teodor@sigaev.ru> writes:
> GUC variable pg_trgm.similarity_threshold insead of set_limit()

If that's to be an actual GUC variable, it's not okay for set_limit()
to just do this:

    similarity_threshold = nlimit;

It should go through SetConfigOption(); compare for example
tsa_set_curcfg_byname().  Which would also mean you could drop the
range-checking code there, because guc.c would enforce it.

            regards, tom lane


pgsql-committers by date:

Previous
From: Teodor Sigaev
Date:
Subject: pgsql: Improve support of Hunspell
Next
From: Teodor Sigaev
Date:
Subject: Re: pgsql: GUC variable pg_trgm.similarity_threshold insead of set_limit()