From: | "Gauthier, Dave" <dave(dot)gauthier(at)intel(dot)com> |
---|---|
To: | David Johnston <polobo(at)yahoo(dot)com>, "pgsql-general(at)postgresql(dot)org" <pgsql-general(at)postgresql(dot)org> |
Subject: | Re: constraining chars for all cols of a table |
Date: | 2011-02-18 22:45:38 |
Message-ID: | 482E80323A35A54498B8B70FF2B879800482827314@azsmsx504.amr.corp.intel.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
Yes, I was doing that in the app that inserts/updates the records. But someone coming at this via direct sql would bypass the check in the app. :-(
From: David Johnston [mailto:polobo(at)yahoo(dot)com]
Sent: Friday, February 18, 2011 5:10 PM
To: Gauthier, Dave; pgsql-general(at)postgresql(dot)org
Subject: RE: [GENERAL] constraining chars for all cols of a table
I don't fully follow the issue or usage pattern. It may not be perfect database design but you should be most concerned with user generated data. If you are cascading within the database (or application) system and you have this kind of extreme 1-input/1000-output relationship then you should TEST and VERIFY that the "cascading" code generates valid data (given valid input) and restrict your focus to validating the original input.
From: pgsql-general-owner(at)postgresql(dot)org [mailto:pgsql-general-owner(at)postgresql(dot)org] On Behalf Of Gauthier, Dave
Sent: Friday, February 18, 2011 4:51 PM
To: David Johnston; pgsql-general(at)postgresql(dot)org
Subject: Re: [GENERAL] constraining chars for all cols of a table
>>Restrict access to the table (for inserts) to a function that does the verification and then executes the insert in addition to any kind of logging and >>"RAISE"ing you need.
Wouldn't that be akin to doing the checking in the insert and update before triggers? That's certainly possible, but I fear the performance hit if I have to do a metadata query to get all the column names, then check them all one by one against a regexp that each maps to. This should be no problem for single record inserts by users. But some of these insert triggers cascade the creation of hundreds or thousands of additional records recursively. If the added overhead is a hald second per for what becomes a 1000 record creation, that's over 8 minutes !
I was looking at enum, and something like that would work if I could replace the hardcoded list of values with something like a regexp.
Still looking :-)
From: David Johnston [mailto:polobo(at)yahoo(dot)com]
Sent: Friday, February 18, 2011 4:42 PM
To: Gauthier, Dave; pgsql-general(at)postgresql(dot)org
Subject: RE: [GENERAL] constraining chars for all cols of a table
Restrict access to the table (for inserts) to a function that does the verification and then executes the insert in addition to any kind of logging and "RAISE"ing you need.
If you need to validate existing data I'd probably just do some one-time verifications and updates where required.
A column "CHECK" constraint, however, seems like it should work just find if you use a regular expression - and I cannot imagine it would be that performance limiting.
Without a more specific model in mind choosing between different approaches is difficult.
David J.
From: pgsql-general-owner(at)postgresql(dot)org [mailto:pgsql-general-owner(at)postgresql(dot)org] On Behalf Of Gauthier, Dave
Sent: Friday, February 18, 2011 4:24 PM
To: pgsql-general(at)postgresql(dot)org
Subject: [GENERAL] constraining chars for all cols of a table
Hi:
I have to constrain the chars used for table columns. For example...
create table foo (col1 text, col2 text, col3 text);
... where
col1 has to be all uppercase, nothing but [A-Z]
col2 has to be all lowercase [a-z] plus [0-9] is also allowed
col3 can be mixed case plus [0-9] and sqr brackets (but nothing else).
I could put a check constraint on each/every table column, but that seems complicated and potentially slow.
I could do this check using the existing insert and update before triggers, but then I'd have to loop through all the columns and decide one by one how to
check them. Again, slow and complicated.
Is there a better way? Whatever I do, I'd have to be able to capture violations to provide informative feedback to the users through the perl script that'll actually be doing the insert/update.
Thanks in Advance !
From | Date | Subject | |
---|---|---|---|
Next Message | Rich Shepard | 2011-02-18 22:55:52 | Explicit NULL for no INTEGER data? |
Previous Message | Susan Cassidy | 2011-02-18 22:18:03 | Re: constraining chars for all cols of a table |