Re: Fix to not check included columns in ANALYZE on indexes - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Fix to not check included columns in ANALYZE on indexes
Date
Msg-id D0EAA77B-1F8F-4D1A-B8E0-576D568D92A2@anarazel.de
Whole thread Raw
In response to Re: Fix to not check included columns in ANALYZE on indexes  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Fix to not check included columns in ANALYZE on indexes
List pgsql-hackers

On June 28, 2018 4:18:36 PM PDT, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>Yugo Nagata <nagata@sraoss.co.jp> writes:
>> I found that both key columns and included columns are checked when
>analyze
>> is run on indexes. This is almost harmless because non-expression
>columns
>> are not processed. However, this check is obviously unnecessary and
>we
>> can fix this to not check included columns. If we decide to support
>expressions
>> in included columns in future, this must be fixed eventually.
>
>AFAICS, we'd just have to revert this patch later, so I don't see
>much value in it.
>
>Also, is it really true that we don't support included expression
>columns now?  In what way would that not be a bug?

I don't think IOS supports expression columns, right?  Away from code for a bit, so can't check.  If indeed true,
there'dbe little point in allowing it, right? 

Andres

Andres

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


pgsql-hackers by date:

Previous
From: "David G. Johnston"
Date:
Subject: Re: CREATE TABLE .. LIKE .. EXCLUDING documentation
Next
From: Tom Lane
Date:
Subject: Re: Fix to not check included columns in ANALYZE on indexes