-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[PLSQL] Single statement without ending semicolon #3556
Conversation
@KvanTTT To let single statement working without semicolon required lot of changes. Can I gently ask for a review? |
@KvanTTT Can you please help with
I need some guidance on how to figure out the issue here. Thanks. |
It looks like just a problem with CSharp test runner. |
Co-authored-by: Ivan Kochurkin <kvanttt@gmail.com>
@KvanTTT thanks for your suggestions. I have applied it. |
@kaby76 it looks like there is a problem with coverage CI task. Could you investigate please? |
Seems same issue with CSharp. Any suggestioni? |
I mentioned this for another PR. #3612 (comment) And I opened an Issue for it. #3597 I will turn off voverage computations for now as this will require a change to trcover, and will take time to make sure it works. |
I should probably open an issue for this but SQLPlus commands do not require a terminating semicolon at all. See section 4.6.1 of the SQLPlus user guide here: https://docs.oracle.com/en/database/oracle/oracle-database/23/sqpug/SQL-Plus-basics.html#GUID-4F029548-6DD2-468F-B6E1-E6D6AA59014B So if you have a file that starts with:
These changes make the parser bail out immediately on the |
In case of single statement without ending semicolon, do you think this should pass?
I think yes, since this query is valid. The statement should fail if is followed by another one.