Bugs item #1714829, was opened at 2007-05-08 12:07 Message generated for change (Comment added) made by romulog You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1714829&group_id=56967
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: SQL/Core Group: SQL CVS Head >Status: Closed Resolution: None Priority: 5 Private: No Submitted By: Romulo Goncalves (romulog) Assigned to: Niels Nes (nielsnes) Summary: SQL: CHECK CONSTRAINT... Initial Comment: I created the table: create table t1 (id int CHECK (f1() = 1)); MAPI = [EMAIL PROTECTED]:50000 QUERY = create table t3 (id int CHECK (f1() = 1)); ERROR = !unknown constraint (20874664)->token = CHECK I think the example says everything. Regards, Romulo ---------------------------------------------------------------------- >Comment By: Romulo Goncalves (romulog) Date: 2007-05-08 19:33 Message: Logged In: YES user_id=1498628 Originator: YES This bug can be closed. The checkin at 19:31 by Romulo contains the fix. The test to cover it is: sql/src/test/BugConstraints/Tests/check_constraint.SF-1714829 Regards, Romulo PS: The test check.sql added by Martin was removed. ---------------------------------------------------------------------- Comment By: Romulo Goncalves (romulog) Date: 2007-05-08 17:48 Message: Logged In: YES user_id=1498628 Originator: YES I aggree with you, I will change the parser. It is an implemented feature not handle correctly by Niels. Regards, Romulo ---------------------------------------------------------------------- Comment By: Fabian (mr-meltdown) Date: 2007-05-08 17:36 Message: Logged In: YES user_id=963970 Originator: NO It *is* a bug, because the message produced is completely garbage. If the feature is not supported, then the parser shouldn't accept it, or an appropriate message conveyed. ---------------------------------------------------------------------- Comment By: Romulo Goncalves (romulog) Date: 2007-05-08 17:34 Message: Logged In: YES user_id=1498628 Originator: YES Why did you add a test file if Niels said that it is not a bug? Regards, Romulo ---------------------------------------------------------------------- Comment By: Martin Kersten (mlkersten) Date: 2007-05-08 17:29 Message: Logged In: YES user_id=490798 Originator: NO Test file added to repository. 'BugConstraints/check.sql' ---------------------------------------------------------------------- Comment By: Romulo Goncalves (romulog) Date: 2007-05-08 12:43 Message: Logged In: YES user_id=1498628 Originator: YES Well, If the message error is weak it is not a bug. We already agree that the error system of M5-SQL has to be improved. The problem is that we need time. I do not close the bug, it should be closed by Niles. He is the big boss of SQL module. Regards, Romulo ---------------------------------------------------------------------- Comment By: Fabian (mr-meltdown) Date: 2007-05-08 12:22 Message: Logged In: YES user_id=963970 Originator: NO I disagree. The message could have been in Chinese, I don't think it's clear at all. ---------------------------------------------------------------------- Comment By: Romulo Goncalves (romulog) Date: 2007-05-08 12:19 Message: Logged In: YES user_id=1498628 Originator: YES it tells you that this feature isn't supported! So this is not a bug report but a feature request which Niels kindly ignore. Regards, Romulo ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1714829&group_id=56967 ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Monetdb-bugs mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/monetdb-bugs
