Github user xwq commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/849#discussion_r89429871
  
    --- Diff: core/sql/regress/compGeneral/EXPECTED006.SB ---
    @@ -1686,9 +1686,10 @@ d3d9446802a44259755d38e6d163e820   852952723  
b1d5781111d84f7b3fe45a0852e59758cd
     --- 1 row(s) selected.
     >>select sha2('the original data', 128) from dual;
     
    -*** ERROR[8428] The argument to function SHA2 is not valid.
    +*** ERROR[4045] The operand of function SHA2 must be numeric.
    --- End diff --
    
    @zellerh 
    
    Can I use 8428 instead of 4045 or use yyerror('The second operand expect 
224, 256, 384 or 512') to throw Internal parse error with message. I think we 
don't need to add a new error code since it only used once here. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to