Michael Paquier <mich...@paquier.xyz> writes: > This patch seems to have little incidence on the stability, but FWIW I > am not cool with the concept of asking for objections and commit a > patch only 4 hours after-the-fact, particularly after feature freeze.
FWIW, I figured it was okay since ECPG has essentially no impact on the rest of the system. The motivation for having feature freeze is to get us to concentrate on stability, but any new bugs in ECPG aren't going to affect the stability of anything else. Also, I don't think it's that hard to look at this as a bug fix rather than a new feature. The general expectation is that ECPG can parse any command the backend can --- that's why we went to all the trouble of automatically building its grammar from the backend's. So I was surprised to hear that it didn't work on some EXECUTE variants, and filling in that gap doesn't seem like a "new feature" to me. Note the lack of any documentation additions in the patch. regards, tom lane