On Fri, Feb 26, 2021, at 19:55, Tom Lane wrote: > "Joel Jacobson" <j...@compiler.org> writes: > > On Fri, Feb 26, 2021, at 01:16, Tom Lane wrote: > >> 0007-smarter-regex-allocation-2.patch > > > I've successfully tested this patch. > > Cool, thanks for testing!
I thought it would be interesting to see if any differences in *where* matches occur not only *what* matches. I've compared the output from regexp_positions() between REL_13_STABLE and HEAD. I'm happy to report no differences were found, except some new expected invalid regular expression: invalid character range errors due to the fixes. This time I also ran into the (["'`])(?:\\\1|.)*?\1 pattern due to using the flags, which caused a timeout on REL_13_STABLE, but the same pattern is fast on HEAD. All good. /Joel