Patch applied. Thanks.
---
Atsushi Ogawa wrote:
>
> Bruce Momjian wrote:
> > I have applied your patch, with slight adjustments in spacing and
> > documentation.
> >
> > Patch applied. Thanks.
>
> Thank you for applying
Bruce Momjian wrote:
> I have applied your patch, with slight adjustments in spacing and
> documentation.
>
> Patch applied. Thanks.
Thank you for applying patch.
An attached patch is a small additional improvement.
This patch use appendStringInfoText instead of appendStringInfoString.
There
The change below has broken tsearch2. See for example
http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=shrew&dt=2005-07-10%2015:02:01
cheers
andrew
Bruce Momjian wrote:
I have applied your patch, with slight adjustments in spacing and
documentation.
Patch applied. Thanks.
[snip]
I have applied your patch, with slight adjustments in spacing and
documentation.
Patch applied. Thanks.
---
Atsushi Ogawa wrote:
>
> I made the patch that implements regexp_replace again.
> The specification of this func
> > pg_catalog | replace | text | text, text, text
> >
>
> I think that regexp_replace is a good name. It is easy to understand.
>
> regards,
I prefere this name too
Regards
Pavel Stehule
---(end of broadcast)---
TIP 8: explain ana
Atsushi Ogawa wrote:
I think that regexp_replace is a good name. It is easy to understand.
I'll go with the flow.
cheers
andrew
---(end of broadcast)---
TIP 4: Don't 'kill -9' the postmaster
Bruce Momjian wrote:
> Andrew Dunstan wrote:
> > I'm very glad to see this. But is a nicer name possible? To perl
> > programmers at least, "substitute" should make sense.
>
> What is the matter with replace? We already have replace:
>
> test=> \df replace
>List of funct
Andrew Dunstan wrote:
> I'm very glad to see this. But is a nicer name possible? To perl
> programmers at least, "substitute" should make sense.
What is the matter with replace? We already have replace:
test=> \df replace
List of functions
Schema | Name | Result d
I'm very glad to see this. But is a nicer name possible? To perl
programmers at least, "substitute" should make sense.
cheers
andrew
Atsushi Ogawa wrote:
>I made the patch that implements regexp_replace again.
>The specification of this function is as follows.
>
>regexp_replace(source text, p
I made the patch that implements regexp_replace again.
The specification of this function is as follows.
regexp_replace(source text, pattern text, replacement text, [flags text])
returns text
Replace string that matches to regular expression in source text to
replacement text.
- pattern is r
Robert Treat wrote:
> On Tuesday 07 June 2005 10:57, David Fetter wrote:
> > On Tue, Jun 07, 2005 at 10:27:28PM +0900, Atsushi Ogawa wrote:
> > > My idea is opposite. I think that the regexp_replace() should make
> > > "replace all" a default. Because the replace() of pgsql replaces all
> > > strin
On Tuesday 07 June 2005 10:57, David Fetter wrote:
> On Tue, Jun 07, 2005 at 10:27:28PM +0900, Atsushi Ogawa wrote:
> > My idea is opposite. I think that the regexp_replace() should make
> > "replace all" a default. Because the replace() of pgsql replaces all
> > string, and regexp_replace() of ora
On Tue, Jun 07, 2005 at 10:27:28PM +0900, Atsushi Ogawa wrote:
>
> David Fetter wrote:
> > On Tue, Jun 07, 2005 at 09:35:56AM +0900, a_ogawa wrote:
> > > David Fetter wrote:
> > > > We don't yet have this functionality, as the patch allows for
> > > > using second and later regex matches "()" in t
David Fetter wrote:
> On Tue, Jun 07, 2005 at 09:35:56AM +0900, a_ogawa wrote:
> > David Fetter wrote:
> > > We don't yet have this functionality, as the patch allows for
> > > using second and later regex matches "()" in the replacement
> > > pattern.
> > >
> > > The function is misnamed. It sho
On Tue, Jun 07, 2005 at 09:35:56AM +0900, a_ogawa wrote:
>
> Bruce Momjian wrote:
> > David Fetter wrote:
> > > On Mon, Jun 06, 2005 at 12:02:18PM -0400, Bruce Momjian wrote:
> > > >
> > > > Patch removed because we already have this functionality.
> > >
> > > We don't yet have this functionality,
Bruce Momjian wrote:
> David Fetter wrote:
> > On Mon, Jun 06, 2005 at 12:02:18PM -0400, Bruce Momjian wrote:
> > >
> > > Patch removed because we already have this functionality.
> >
> > We don't yet have this functionality, as the patch allows for using
> > second and later regex matches "()" in
David Fetter wrote:
> On Mon, Jun 06, 2005 at 12:02:18PM -0400, Bruce Momjian wrote:
> >
> > Patch removed because we already have this functionality.
>
> We don't yet have this functionality, as the patch allows for using
> second and later regex matches "()" in the replacement pattern.
>
> The
On Mon, Jun 06, 2005 at 12:02:18PM -0400, Bruce Momjian wrote:
>
> Patch removed because we already have this functionality.
We don't yet have this functionality, as the patch allows for using
second and later regex matches "()" in the replacement pattern.
The function is misnamed. It should be
Nice. Patch removed.
---
Tom Lane wrote:
> Bruce Momjian writes:
> > Tom Lane wrote:
> >> Don't we have this functionality already? It's even SQL-spec ...
>
> > Uh, all I see it replace(), which isn't regex:
>
> The SQL-
Patch removed because we already have this functionality.
---
a_ogawa00 wrote:
>
> This patch provides a new function regexp_replace.
> regexp_replace extends a replace function and enables text search
> by the regular expr
Bruce Momjian writes:
> Tom Lane wrote:
>> Don't we have this functionality already? It's even SQL-spec ...
> Uh, all I see it replace(), which isn't regex:
The SQL-spec function is substring(string from pattern for escape-char);
see
http://www.postgresql.org/docs/8.0/static/functions-matching
Tom Lane wrote:
> Bruce Momjian writes:
> > Your patch has been added to the PostgreSQL unapplied patches list at:
>
> > a_ogawa00 wrote:
> >> This patch provides a new function regexp_replace.
> >> regexp_replace extends a replace function and enables text search
> >> by the regular expression.
Bruce Momjian writes:
> Your patch has been added to the PostgreSQL unapplied patches list at:
> a_ogawa00 wrote:
>> This patch provides a new function regexp_replace.
>> regexp_replace extends a replace function and enables text search
>> by the regular expression. And, a back reference can be u
I will add the documentation and make sure your oids are not duplicates.
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
It will be applied as soon as one of the PostgreSQL committers reviews
and approves it.
--
This has been saved for the 8.1 release:
http:/momjian.postgresql.org/cgi-bin/pgpatches2
---
a_ogawa00 wrote:
>
> This patch provides a new function regexp_replace.
> regexp_replace extends a replace function and e
This patch provides a new function regexp_replace.
regexp_replace extends a replace function and enables text search
by the regular expression. And, a back reference can be used within
a replace string.
(This patch for PostgreSQL 7.4.3)
Function: regexp_replace(str, pattern,
26 matches
Mail list logo
