odiszapc / c5-db-migration

Automatically exported from code.google.com/p/c5-db-migration
0 stars 0 forks source link

MySQL functions with comments not created #16

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
[ERROR] Error executing: /*!50003 CREATE*/ /*!50020 DEFINER=`reuters`@`
%`*/ /*!50003 FUNCTION `error_message_key`(code INTEG
ER(11)) RETURNS varchar(2000) CHARSET utf8     READS SQL DATA begin   
declare error_message varchar(2000)
com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an 
error in your SQL syntax; check the manual that corre
sponds to your MySQL server version for the right syntax to use near '' at 
line 1
        at com.carbonfive.db.jdbc.ScriptRunnerImpl.doExecute
(ScriptRunnerImpl.java:131)
        at com.carbonfive.db.jdbc.ScriptRunnerImpl.execute
(ScriptRunnerImpl.java:26)
        at com.carbonfive.db.jdbc.ScriptRunner.execute
(ScriptRunner.java:25)
        at com.carbonfive.db.migration.SQLScriptMigration.migrate
(SQLScriptMigration.java:30)

Original issue reported on code.google.com by tumakha on 27 Jan 2009 at 3:32

Attachments:

GoogleCodeExporter commented 8 years ago
This was tested on the 0.9.6 build.

Original comment by tumakha on 27 Jan 2009 at 3:52

GoogleCodeExporter commented 8 years ago
Thanks for the submission! -Christian

Original comment by christia...@gmail.com on 29 Jan 2009 at 5:27

GoogleCodeExporter commented 8 years ago
MySQL command DELIMITER not supported also for simple functions

Original comment by tumakha on 26 Feb 2009 at 11:57

Attachments:

GoogleCodeExporter commented 8 years ago
Reset the delimiter back to the default not needed.
New delimiter must be set by new command "DELIMITER ;"
This is normally behaviour of MySQL database.
Patch for fixing this issue in attachment.

Original comment by tumakha on 26 Feb 2009 at 12:39

Attachments:

GoogleCodeExporter commented 8 years ago
Fixed in svn, look for 0.9.7 release to include changes.  Thanks for the patch!

Original comment by christia...@gmail.com on 27 Feb 2009 at 4:40