nanookclaw commented on PR #22284: URL: https://github.com/apache/datafusion/pull/22284#issuecomment-4543833234
It looks stale on current `main` rather than a code change needed in this PR. Before opening this I checked the reported query against current `main` and it already returned PostgreSQL's expected result (`true` for `regexp_like(E'a\nb', '^b', 'm')`). So this PR is intentionally regression coverage only: it locks the multiline `m` behavior in both SQL logic tests and the Rust scalar/array paths. I did not trace the exact commit that fixed the originally reported mismatch; it may have been corrected by another regex-path change between #22268 being filed and this PR. If coverage-only is not useful here, I'm fine closing it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
