On Sat, Oct 19, 2024, at 09:52, Joel Jacobson wrote: > However, since my last email, I've found some other problems in this area, > and think we should do a more ambitious improvement, by rearranging the > incorrect options tests into three categories: > > 1. incorrect COPY {FROM|TO} options > 2. incorrect COPY FROM options > 3. incorrect COPY TO options > > Also, I've found two new problems: > > 1. Incorrect options tests are missing for the QUOTE and ESCPAE options. > This was discovered by Jian He in a different thread. > > 2. One of the ON_ERROR incorrect options tests also depend on the order > of checks in ProcessCopyOptions. > > To explain my current focus on the COPY tests, it's because we're currently > working on the new raw format for COPY, and it would be nice to cleanup these > tests as a preparatory step first.
I realize these suggested changes are out of scope for $subject, that was about a bug fix. Therefore closing this patch and marking it as Committed. /Joel