mirror of
https://github.com/postgres/postgres.git
synced 2026-06-27 11:01:20 -04:00
Peter Eisentraut
7ac030d5b1
It seems like the SQL standard does require this after all, and it makes sense because these columns get changed. (This is not to be confused with *not* requiring INSERT permission to add the temporal leftovers.) Adding the column to RTEPermissionInfo->updatedCols also fixes a couple outstanding bugs from other (non-permission) features using that bitmapset to detect changes: GENERATED columns and UPDATE OF triggers. This patch includes test cases to exercise those scenarios, including on partitioned tables. Author: Paul A. Jungwirth <pj@illuminatedcomputing.com> Reviewed-by: jian he <jian.universality@gmail.com> Discussion: https://www.postgresql.org/message-id/flat/CAHg%2BQDcd%3Dt69gLf9yQexO07EJ2mx0Z70NFHo6h94X1EDA%3DhM0g%40mail.gmail.com