-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Oracle Timesten dialect updates #10273
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
base: 6.6
Are you sure you want to change the base?
Conversation
…LimitHandler and TimesTenSequenceSupport
Thanks for your pull request! This pull request does not follow the contribution rules. Could you have a look? ❌ All commit messages should start with a JIRA issue key matching pattern › This message was automatically generated. |
Hi @carlblan, thank you for the contribution.
As @yrodiere mentioned in this other pr #9629 (comment), we now have an |
Also please note that your contributions cannot, under any circumstance, be licensed under UPL, but must be licensed under ASL2 with the option to relicense under LGPL v2.1, as mentioned in your PR description:
And thanks for contributing :) |
return false; | ||
} | ||
|
||
public String getDual() { |
Check notice
Code scanning / CodeQL
Missing Override annotation Note
Dialect.getDual
return "dual"; | ||
} | ||
|
||
public String getFromDualForSelectOnly() { |
Check notice
Code scanning / CodeQL
Missing Override annotation Note
Some updates from an Oracle employee to the Oracle TimesTen Community dialect. ( Hibernate 6.6 )
In TimesTenDialect.java
In TimesTenSqlAstTranslator.java
In TimesTenLimitHandler.java
In TimesTenSequenceSupport.java
Testing:
SPECIAL NOTE:
Please review those and let me know If there's a problem with that.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.