pg.ddx.io pgsql-bugs@postgresql.org mailing list archive
help / color / mirror / Atom feedBUG #14441: trim function bug (hit a keyword somewhere internally)
5+ messages / 4 participants
[nested] [flat]
* BUG #14441: trim function bug (hit a keyword somewhere internally)
@ 2016-11-30 01:17 mark.pether@gmail.com
2016-11-30 06:09 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) David Gould <daveg@sonic.net>
2016-11-30 06:24 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 2 replies; 5+ messages in thread
From: mark.pether@gmail.com @ 2016-11-30 01:17 UTC (permalink / raw)
To: pgsql-bugs
The following bug has been logged on the website:
Bug reference: 14441
Logged by: mark pether
Email address: mark.pether@gmail.com
PostgreSQL version: 9.5.4
Operating system: macos
Description:
select TRIM(LEADING 'EXCLUDE' FROM 'C001');
Will incorrectly strip off the 'C' leaving '001'.
select TRIM(LEADING 'exclude' FROM 'C001');
Will not strip trim anything as expected.
Cheers
--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #14441: trim function bug (hit a keyword somewhere internally)
2016-11-30 01:17 BUG #14441: trim function bug (hit a keyword somewhere internally) mark.pether@gmail.com
@ 2016-11-30 06:09 ` David Gould <daveg@sonic.net>
2016-11-30 06:16 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) mark pether <mark.pether@gmail.com>
1 sibling, 1 reply; 5+ messages in thread
From: David Gould @ 2016-11-30 06:09 UTC (permalink / raw)
To: mark.pether@gmail.com; +Cc: pgsql-bugs
On Wed, 30 Nov 2016 01:17:10 +0000
mark.pether@gmail.com wrote:
> The following bug has been logged on the website:
>
> Bug reference: 14441
> Logged by: mark pether
> Email address: mark.pether@gmail.com
> PostgreSQL version: 9.5.4
> Operating system: macos
> Description:
>
> select TRIM(LEADING 'EXCLUDE' FROM 'C001');
>
> Will incorrectly strip off the 'C' leaving '001'.
Perhaps I'm misreading the doc, but that looks like the correct result.
That is, any character in 'EXCLUDE' will be stripped from the leading part of
'C001', ie, the 'C'. Why do you think this is a bug?
-dg
--
David Gould daveg@sonic.net
If simplicity worked, the world would be overrun with insects.
--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #14441: trim function bug (hit a keyword somewhere internally)
2016-11-30 01:17 BUG #14441: trim function bug (hit a keyword somewhere internally) mark.pether@gmail.com
2016-11-30 06:09 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) David Gould <daveg@sonic.net>
@ 2016-11-30 06:16 ` mark pether <mark.pether@gmail.com>
2016-11-30 06:51 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 5+ messages in thread
From: mark pether @ 2016-11-30 06:16 UTC (permalink / raw)
To: David Gould <daveg@sonic.net>; +Cc: pgsql-bugs
Sorry, I misread the document it's not a bug, doco was unclear.
It replaces the largest match which may be all characters or simply
individual characters. I realised this after I created the bug.
I have switched to REGEXP_REPLACE to do the correct operation.
Regards
On Wed, Nov 30, 2016 at 5:09 PM, David Gould <daveg@sonic.net> wrote:
> On Wed, 30 Nov 2016 01:17:10 +0000
> mark.pether@gmail.com wrote:
>
> > The following bug has been logged on the website:
> >
> > Bug reference: 14441
> > Logged by: mark pether
> > Email address: mark.pether@gmail.com
> > PostgreSQL version: 9.5.4
> > Operating system: macos
> > Description:
> >
> > select TRIM(LEADING 'EXCLUDE' FROM 'C001');
> >
> > Will incorrectly strip off the 'C' leaving '001'.
>
>
> Perhaps I'm misreading the doc, but that looks like the correct result.
> That is, any character in 'EXCLUDE' will be stripped from the leading part
> of
> 'C001', ie, the 'C'. Why do you think this is a bug?
>
> -dg
>
>
>
> --
> David Gould daveg@sonic.net
> If simplicity worked, the world would be overrun with insects.
>
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #14441: trim function bug (hit a keyword somewhere internally)
2016-11-30 01:17 BUG #14441: trim function bug (hit a keyword somewhere internally) mark.pether@gmail.com
2016-11-30 06:09 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) David Gould <daveg@sonic.net>
2016-11-30 06:16 ` Re: BUG #14441: trim function bug (hit a keyword somewhere internally) mark pether <mark.pether@gmail.com>
@ 2016-11-30 06:51 ` Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 0 replies; 5+ messages in thread
From: Tom Lane @ 2016-11-30 06:51 UTC (permalink / raw)
To: mark pether <mark.pether@gmail.com>; +Cc: David Gould <daveg@sonic.net>; pgsql-bugs
mark pether <mark.pether@gmail.com> writes:
> Sorry, I misread the document it's not a bug, doco was unclear.
> It replaces the largest match which may be all characters or simply
> individual characters. I realised this after I created the bug.
Hmm, yeah, docs fail to make clear what happens with a multicharacter
trim spec. I think we could fix this with a better example.
Will do something about it.
regards, tom lane
--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #14441: trim function bug (hit a keyword somewhere internally)
2016-11-30 01:17 BUG #14441: trim function bug (hit a keyword somewhere internally) mark.pether@gmail.com
@ 2016-11-30 06:24 ` Tom Lane <tgl@sss.pgh.pa.us>
1 sibling, 0 replies; 5+ messages in thread
From: Tom Lane @ 2016-11-30 06:24 UTC (permalink / raw)
To: mark.pether@gmail.com; +Cc: pgsql-bugs
mark.pether@gmail.com writes:
> select TRIM(LEADING 'EXCLUDE' FROM 'C001');
> Will incorrectly strip off the 'C' leaving '001'.
The code appears to believe that the first argument is a set of characters
that are to be removed from the front of the string argument until none
remain. For instance:
regression=# select TRIM(LEADING 'ABC' FROM 'CAABC001');
ltrim
-------
001
(1 row)
So the behavior you mention is expected.
I think what you're expecting is to strip off one or more occurrences of a
substring, but that's not what our version of TRIM does.
A look at the SQL standard indicates that it requires the first argument
to be *exactly one* character in length, and then TRIM strips leading
occurrence(s) of that character. So our existing implementation is a
plausible extension of that ... as is your expectation, but there's
certainly no reason to favor yours over what's there. Also:
* the PG code has acted this way since it was introduced in 1997.
* the code comments claim, and a bit of googling seems to confirm,
that this matches Oracle's behavior.
So I do not see us changing this.
You can easily build the behavior you want out of other spare parts.
Regexes would be my first weapon of choice:
regression=# select substring('C001' from '^(?:EXCLUDE)*(.*)$');
substring
-----------
C001
(1 row)
regression=# select substring('EXCLUDEC001' from '^(?:EXCLUDE)*(.*)$');
substring
-----------
C001
(1 row)
regression=# select substring('EXCLUDEEXCLUDEC001' from '^(?:EXCLUDE)*(.*)$');
substring
-----------
C001
(1 row)
See
https://www.postgresql.org/docs/current/static/functions-matching.html#FUNCTIONS-POSIX-REGEXP
regards, tom lane
--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs
^ permalink raw reply [nested|flat] 5+ messages in thread
end of thread, other threads:[~2016-11-30 06:51 UTC | newest]
Thread overview: 5+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2016-11-30 01:17 BUG #14441: trim function bug (hit a keyword somewhere internally) mark.pether@gmail.com
2016-11-30 06:09 ` David Gould <daveg@sonic.net>
2016-11-30 06:16 ` mark pether <mark.pether@gmail.com>
2016-11-30 06:51 ` Tom Lane <tgl@sss.pgh.pa.us>
2016-11-30 06:24 ` Tom Lane <tgl@sss.pgh.pa.us>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox