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-issue-importer opened this issue Mar 4, 2016 · 8 comments

Comments

@firebird-issue-importer
Copy link

firebird-issue-importer commented Mar 4, 2016

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-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 4, 2016

Modified by: @dyemanov

Fix Version: 3.0 RC2 [ 10048 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 4, 2016

Modified by: @dyemanov

Version: 3.0 RC2 [ 10048 ]

Fix Version: 3.0 RC2 [ 10048 ] =>

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 4, 2016

Commented by: @dyemanov

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

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 4, 2016

Modified by: @dyemanov

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 8, 2016

Modified by: @dyemanov

Fix Version: 3.0.0 [ 10740 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 9, 2016

Modified by: @asfernandes

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

resolution: Fixed [ 1 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 10, 2016

Modified by: @pavel-zotov

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

QA Status: No test => Done successfully

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 10, 2016

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