Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 322
  • 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
Industry Leaders: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
Robert SilverAuthor Commented:
For the record what does DELIMITER //    {CODE} //   do?
0
 
Robert SilverAuthor 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

Vote for the Most Valuable Expert

It’s time to recognize experts that go above and beyond with helpful solutions and engagement on site. Choose from the top experts in the Hall of Fame or on the right rail of your favorite topic page. Look for the blue “Nominate” button on their profile to vote.

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