Re: regexp_matches and regexp_split are inconsistent

From: Stephan Szabo <sszabo(at)megazone(dot)bigpanda(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: regexp_matches and regexp_split are inconsistent
Date: 2007-08-11 16:59:03
Message-ID: 20070810193026.J20339@megazone.bigpanda.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On Fri, 10 Aug 2007, Tom Lane wrote:

> I noticed the following behavior in CVS HEAD, using a pattern that is
> capable of matching no characters:
>
> regression=# SELECT foo FROM regexp_matches('ab cde', $re$\s*$re$, 'g') AS foo;
> foo
> -------
> {""}
> {""}
> {" "}
> {""}
> {""}
> {""}
> {""}
> (7 rows)
>
> regression=# SELECT foo FROM regexp_split_to_table('ab cde', $re$\s*$re$) AS foo;
> foo
> -----
> a
> b
> c
> d
> e
> (5 rows)
>
> If you count carefully, you will see that regexp_matches() reports a
> match of the pattern at the start of the string and at the end of the
> string, and also just before 'c' (after the match to the single space).
> However, regexp_split() disregards these "degenerate" matches of the
> same pattern.
>
> Is this what we want? Arguably regexp_split is doing the most
> reasonable thing for its intended usage, but the strict definition of
> regexp matching seems to require what regexp_matches does. I think
> we need to either change one function to match the other, or else
> document the inconsistency.
>
> Thoughts?

I'm not sure how many languages do this, but at least perl seems to work
similarly, which makes me guess that it's probably similar in a bunch of
languages. If it is, then we should probably just document the
inconsistency.

Perl seems to document the split behavior with "Empty leading (or
trailing) fields are produced when there are positive width matches at the
beginning (or end) of the string; a zero-width match at the beginning (or
end) of the string does not produce an empty field."

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2007-08-11 18:36:12 Interesting misbehavior of repalloc()
Previous Message Joshua D. Drake 2007-08-11 15:27:33 Re: Wrote a connect-by feature