Re: Delete expression causing SqlValidatorException

[Julian Hyde <jh...@apache.org>]

Are you saying that the fix for CALCITE-546 (i.e. commit 1c292a9d) broke something? Or that a subsequent commit broke something?

In any case, it seems pretty clear that something is broken. In software engineering terms, the cause of this issue is clear: we don’t have a test for “delete from tbl”. Let’s put that right.

Can you log a JIRA case and provide a test case (a fix would be even better!)

Julian


> On Nov 12, 2015, at 7:11 AM, Yuri Au Yong <yu...@persistent.com> wrote:
> 
> SQL “delete from tbl” causes SqlValidatorException :
>  
> Nov 12, 2015 10:47:47 PM org.apache.calcite.sql.validate.SqlValidatorException <init>
> SEVERE: org.apache.calcite.sql.validate.SqlValidatorException: Column '*' not found in any table
>  
> It seems some of https://issues.apache.org/jira/browse/CALCITE-546 <https://issues.apache.org/jira/browse/CALCITE-546> fix code is reverted. So far I have just seen this in the latest SqlValidatorImpl.java code:
>  
> SqlIdentifier.star(starPos)
>  
> Reverted to
>  
> new SqlIdentifier("*", SqlParserPos.ZERO)
>  
>  
>  
> Thanks & Regards,
>  
> Yuri Au Yong
> Software Engineer - NPM | yuri.auyong@persistent.my <ma...@persistent.my> | Desk: +60 3 7663 8372
> Persistent Systems Ltd. | Partners in Innovation | www.persistentsys.com <http://www.persistentsys.com/>
>  <https://tnpmsupport.persistentsys.com/training>
>  
> DISCLAIMER ========== This e-mail may contain privileged and confidential information which is the property of Persistent Systems Ltd. It is intended only for the use of the individual or entity to which it is addressed. If you are not the intended recipient, you are not authorized to read, retain, copy, print, distribute or use this message. If you have received this communication in error, please notify the sender and delete all copies of this message. Persistent Systems Ltd. does not accept any liability for virus infected mails.
>