• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 324
  • Last Modified:

MySQL Stored Function Question note the table and columns are verified

Can anyone tell me what is wrong with this simple stored function in MySQL?
CREATE FUNCTION lup_t1(sid int(12) ) RETURNS text DETERMINISTIC
BEGIN
        DECLARE a1 text;
        SELECT  abstract into a1 from notestbl where Id=sid;
        RETURN a1;
END;

Does MySQL 5.x not allow returning Text types? What is going on
Does MySQL 5.x not allow returning certain large text types
0
Robert Silver
Asked:
Robert Silver
  • 3
  • 2
  • 2
1 Solution
 
pivarCommented:
Hi,

Try the following:

DELIMITER //
CREATE FUNCTION lup_t1(sid int(12) ) RETURNS text DETERMINISTIC
BEGIN
        DECLARE a1 text;
      SELECT  abstract into a1 from notestbl where Id=sid;
        RETURN a1;
END
//

/peter
0
 
Kevin CrossChief Technology OfficerCommented:
I have seen a bug report from long ago on TEXT that was too long. How large is the data?
0
 
Kevin CrossChief Technology OfficerCommented:
I could not reproduce the old bug in 5.5 MySQL by the way. Without further information, I would have to agree with Peter that likely you just need to add the DELIMITER change from and to ;
0
The new generation of project management tools

With monday.com’s project management tool, you can see what everyone on your team is working in a single glance. Its intuitive dashboards are customizable, so you can create systems that work for you.

 
Robert SilverSr. Software EngineerAuthor Commented:
For the record what does DELIMITER //    {CODE} //   do?
0
 
Robert SilverSr. Software EngineerAuthor Commented:
I beleieve this is due to a semicolon confusing things so by making // the delimiter
we open and close the code in  // marks.
I believe this is explained in the Section 17.1 of the pdf file: mysql-refman-5.5-en.pdf file
reference manual found at
 http://dev.mysql.com/doc/

Would that be correct?
0
 
Kevin CrossChief Technology OfficerCommented:
Yes. Since the semicolon is the delimiter and as such is likely to be in your code of the function, you have to change the delimiter temporarily, then change it back when you are done.

-- ; used here since we have not changed delimiter yet
USE mysql;

DELIMITER //

-- // used here since we already changed delimiter
DROP FUNCTION IF EXISTS lup_t1 //

CREATE FUNCTION lup_t1(sid int(12) ) RETURNS text DETERMINISTIC
BEGIN
        DECLARE a1 text;
        SELECT  abstract into a1 from notestbl where Id=sid;
        RETURN a1;
END
//
DELIMITER ;
0
 
pivarCommented:
You have to show where the procedure ends, at the same time you have to show where each statement ends. So to not confuse the parser, you need to have different delimiters.
delimiter // changes the delimiter showing where the procedure ends, while the delimiter for statements stays as ;

If you have an one line procedure you don't need to change delimiter:

CREATE FUNCTION hello (s CHAR(20)) RETURNS CHAR(50) DETERMINISTIC
RETURN CONCAT('Hello, ',s,'!');

0

Featured Post

The new generation of project management tools

With monday.com’s project management tool, you can see what everyone on your team is working in a single glance. Its intuitive dashboards are customizable, so you can create systems that work for you.

  • 3
  • 2
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now