Andres Freund <and...@anarazel.de> writes: > In the context of my expression evaluation patch, I was trying to > increase test coverage of execQual.c. I'm a bit confused about > $subject. ExecEvalArrayRef() has the following codepath:
I think it may indeed be unreachable at present, because we don't support something like this: regression=# create table tt (f1 complex[]); CREATE TABLE regression=# update tt set f1[2:3].r = array[7,11]; ERROR: cannot assign to field "r" of column "f1" because its type complex[] is not a composite type LINE 1: update tt set f1[2:3].r = array[7,11]; ^ regression=# update tt set f1[2:3].r = array[7,11]; ERROR: cannot assign to field "r" of column "f1" because its type complex[] is not a composite type LINE 1: update tt set f1[2:3].r = 42; ^ I would not like to remove it though. This particular bit of the executor has no business making assumptions about how array and field references can be nested. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers