-
Notifications
You must be signed in to change notification settings - Fork 176
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
Bug 470284: Do not log a stack trace when creating a schema is no sequence #394
base: master
Are you sure you want to change the base?
Conversation
…ady exist Signed-off-by: Jonathan Coustick <[email protected]>
...e.persistence.core/src/org/eclipse/persistence/tools/schemaframework/SequenceDefinition.java
Outdated
Show resolved
Hide resolved
Also formatting Signed-off-by: Jonathan Coustick <[email protected]>
…into PAYARA-3474-master
@rfelcman Could you re-review please? |
Sorry, I don't think, that this good. There are two reasons why I can't approve this PR:
I don't see there any stacktrace as You mentioned in PR comment. After fix:
There just unnecessary messages -> high amount of log messages -> resources consumption -> there is problem with Travis build too -> see in the last Travis-CI build (attached with last commit in this PR) https://travis-ci.org/eclipse-ee4j/eclipselink/jobs/609266315?utm_medium=notification&utm_source=github_status . Maybe there is something about persistence.xml. Could You please provide it. Could You please provide mentioned stack trace output too. |
When useing the schema generation option of JPA, eclipselink tries to read from an Oracle Database sequence to determine if it should be altered or created.
The checkIfExist() uses the general code for executing queries on the database. And thus when the sequence does not exist, it logs a stacktrace.
The exception is later on swallowed and a false is returned as the result for checkIfExist.
The exception is very irritating as it is just part of the normal processing and doe not indicate a problem with the deploy or schema generation, so this PR hides it at FINEST level.