On 24.9.2013 17:57, Pavel Stehule wrote:
>
>
>
> 2013/9/24 Robert Haas <robertmhaas@gmail.com
> <mailto:robertmhaas@gmail.com>>
>
> On Mon, Sep 23, 2013 at 4:15 PM, Tomas Vondra <tv@fuzzy.cz
> <mailto:tv@fuzzy.cz>> wrote:
>> Seems "ready for commiter" to me. I'll wait a few days for others
>> to comment, and then I'll update the commitfest page.
>
> Some thoughts:
>
> The documentation doesn't reflect the restriction to type internal.
> On a related note, why restrict this to type internal?
>
>
>
> Now, for almost all types Postgres well estimate size of state
> value. Only arrays with unknown size can be a different. When we
> enable this value for all types, then users can specify some bad
> values for scalar buildin types. Next argument is simply and bad - I
> don't see a good use case for customization this value for other than
> types than internal type.
>
> I have no strong position here - prefer joining with internal type
> due little bit higher robustness.
I share this oppinion. I was not able to come up with a single use case
benefiting from allowing this for types other than internal. Seems like
a footgun to me, with no potential benefit.
So +1 to keeping the patch 'type internal only' from me.
With the formatting issues now fixed, I believe the patch is ready for
committer (and someone already switched it to that state).
Tomas