Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compiler issues message about "invalid request BLR" when attempt to compile wrong DDL of view with both subquery and "WITH CHECK OPTION" in its DDL [CORE5130] #5414

Closed
firebird-automations opened this issue Mar 4, 2016 · 8 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @pavel-zotov

create or alter view v2 as select 1 id from rdb$database;
recreate table t1(id int, x int, y int);
commit;

alter view v1 as
select * from t1 a
where
not exists(select * from t1 r where r.x > a.x) ---------- yes, it's not allowed when using together with clause "with check option"
--a.x is not null
with check option
;

Output in 2.5.6:

Statement failed, SQLSTATE = 42000
Dynamic SQL Error
-SQL error code = -607
-Invalid command
-No subqueries permitted for VIEW WITH CHECK OPTION

(OK, expected)

Output in WI-V3.0.0.32375:

Statement failed, SQLSTATE = HY000
invalid request BLR at offset 79
-context already in use (BLR error)

The same in WI-V3.0.0.32366 Firebird 3.0 Release Candidate 2.

Commits: 152b6bd 0eef9d8 FirebirdSQL/fbt-repository@d7c76a4 FirebirdSQL/fbt-repository@cba41c2

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Fix Version: 3.0 RC2 [ 10048 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Version: 3.0 RC2 [ 10048 ]

Fix Version: 3.0 RC2 [ 10048 ] =>

@firebird-automations
Copy link
Collaborator Author

Commented by: @dyemanov

I see that the check for subqueries was removed. Looks like a minor regression.

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Fix Version: 3.0.0 [ 10740 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @asfernandes

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Done successfully

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment