btw ... I just got around to looking at a code coverage report for this patched version, and that reminded me of something I'd already suspected: EEOP_INNER_SYSVAR and EEOP_OUTER_SYSVAR seem to be dead code. That's unsurprising, because we never try to access a tuple's system columns above the scan level. If a query asks for system columns, those get passed up to upper query levels as ordinary user-side columns.
We could keep the execution support for those opcodes, or we could rip it out and throw an error in execExpr.c if one would need to be generated. I'm a little inclined to the latter, because it seems like the plan is to grow more support for every opcode in the future. We don't need to be adding support for unreachable opcodes. regards, tom lane -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers