The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: tested, passed
Spec compliant: tested, failed
Documentation: not tested
Reviewed v1. The optimization looks worthwhile, but the current patch needs a correctness fix for target-list SRFs, as
discussedon-list, plus a regression test. This seems like a contained revision. There is also a broader design question
aboutwhether this should eventually be handled through planner-level uniqueness infrastructure, as raised earlier on
thethread. Marking Waiting on Author for now.
The new status of this patch is: Waiting on Author
