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
cryptic error when providing import.sql file without ; at the end of each line #3817
Comments
The second issue is fixed but it would be nice if we could improve the error message somehow. |
hi @gsmet, I can have a go at fixing this issue |
hi @emmanuelbernard, what version of Quarkus were you using in this code example? I tried to reproduce the issue using 0.25.0 and also the latest 999-SNAPSHOT (as of writing this) and was not able to reproduce the issue -- the init script always ran fine for me. Here is the exact script I used: import.sql
|
I managed to reproduce it and I opened https://hibernate.atlassian.net/browse/HHH-13673 , working on it |
It ran Andy? That’s really surprising. I would expect it to fail and was
hoping for a more graceful failure. The official line is that we ask people
to use `;` between statements.
It’s possible the DB you use was letting this go through. I must have used
H2 or PostgreSQL. These are my natural choices.
…On Wed 16 2019 at 23:38, Andrew Guibert ***@***.***> wrote:
hi @emmanuelbernard <https://github.com/emmanuelbernard>, what version of
Quarkus were you using in this code example? I tried to reproduce the issue
using 0.25.0 and also the latest 999-SNAPSHOT (as of writing this) and was
not able to reproduce the issue -- the init script always ran fine for me.
Here is the exact script I used:
import.sql
insert into Gift(name, id) values ('Gift 1', nextval ('giftSeq'))
insert into Gift(name, id) values ('Gift 2', nextval ('giftSeq'))
insert into Gift(name, id) values ('Gift 3', nextval ('giftSeq'))
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3817?email_source=notifications&email_token=AACJNWBFCR4PA22WZCZRYMLQO6CXBA5CNFSM4ITFV6O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBOATPY#issuecomment-542902719>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACJNWAGIOAFBWSZ54MHTITQO6CXBANCNFSM4ITFV6OQ>
.
|
@emmanuelbernard the exception comes from the antlr parser, it looks for one of the following terminal chars ';' , ' ' , '\t ' , '\r' , '\n' |
@gsmet here the PR hibernate/hibernate-orm#3071 |
@emmanuelbernard yea it ran for me, I pushed the code to this repo to recreate. Looks like @dreab8 has it fixed though so no further need to investigate |
I confirm this was fixed by #4717 |
Adding
Leads to a cryptic
Also it seems adding the
;
while in live coding does not fix the issue. I had to stop and restart Quarkus.The text was updated successfully, but these errors were encountered: