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
Attempt to drop non-existing generator produces bad error [CORE3114] #3492
Comments
Commented by: @aafemt Proposed patch. |
Modified by: @aafemtAttachment: dyn_del.diff [ 11710 ] |
Commented by: @hvlad It seems to me that the real issue is that DYN_error() don't put real error code into status-vector but isc_random + interpreted string instead. Regarding your patch - it is not good idea to "fix" one particular case of more generic problem. |
Commented by: @hvlad DYN_error() is not a half of the engine. |
Commented by: @aafemt Better patch for HEAD only. |
Modified by: @asfernandesstatus: Open [ 1 ] => Resolved [ 5 ] resolution: Fixed [ 1 ] Fix Version: 3.0 Alpha 1 [ 10331 ] assignee: Adriano dos Santos Fernandes [ asfernandes ] |
Modified by: @pcisarstatus: Resolved [ 5 ] => Closed [ 6 ] |
Modified by: @pavel-zotovQA Status: No test |
Modified by: @pavel-zotovstatus: Closed [ 6 ] => Closed [ 6 ] QA Status: No test => Deferred Test Details: Can`t reproduce on: SQL> set heading off; set term ^; execute block returns(e int) as begin execute statement 'drop sequence tratata'; when any do begin 335544351 // it's not isc_random = 335544382 |
Submitted by: @aafemt
Attachments:
dyn_del.diff
CORE-3114.zip
Attempt to drop not existing generator produces isc_random error that makes the error hard to recognize and also has other disadvantages, such as ignoring client locale.
Commits: e090671
====== Test Details ======
Can`t reproduce on:
WI-V2.0.6.13266
WI-V2.1.3.18185
SQL> set heading off; set term ^; execute block returns(e int) as begin execute statement 'drop sequence tratata'; when any do begin
e=gdscode; suspend; end end^ set term ;^
335544351 // it's not isc_random = 335544382
The text was updated successfully, but these errors were encountered: