Incremental Update: v2 Patch Posted
Paul Jungwirth posted a v2 of his own patch for this issue. The key technical development:
Patch Evolution: v1 → v2
-
v1 (referenced but not in this message set): Performed the GENERATED column check during the analysis phase (query analysis/parsing). Paul initially thought separating this check from broader validation-move work (moving validations into planner/executor) would be difficult.
-
v2: Performs the check as a stand-alone patch that is decoupled from the broader effort to relocate validations into the planner/executor. Paul found this separation easier than anticipated.
Context on the Broader Validation Migration
Paul references a discussion (Tom Lane's message at sss.pgh.pa.us) about moving FOR PORTION OF validations from the analysis phase into the planner/executor. The v2 patch is explicitly designed to be independent of that larger refactoring effort, making it committable on its own without waiting for the architectural restructuring.
Authorship Note
This clarifies that Paul Jungwirth is now carrying his own version of this fix (v2), rather than relying solely on Nathan Bossart's earlier patch. The two developers appear to have been working on the same problem from slightly different angles.