Dear Fujii-san,

> > Overall: The author found the case pre-compilation could succeed but the 
> > actual
> > compilation would fail, with SET/GET DESCRIPTOR. This patch tries to detect
> the
> > pre-compilation phase and raise a failure.
> 
> The patch looks good to me.
> Barring any objections, I'm thinking to commit it.

Thanks!

> 
> > Based on the SQL standard/other
> > DBMSs/history, it's not meaningful to allow the statement.
> > This does not contain tests because we cannot compile such a test program in
> the
> > first place.
> 
> I'm not sure this case warrants regression tests, but if we need them,
> the attached patch adds them.

Oh, I didn't recognized that we had TAP tests to detect error/warnings by the 
pre compiler.
I do not have strong opinions for it.

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Reply via email to