Skip to content
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

Feature : Oracle integration #340

Merged
merged 11 commits into from
Mar 10, 2023
Merged

Conversation

RomainWilbert
Copy link
Contributor

Implemented with Oracle-XE (gvenzl/oracle-xe:18-slim).

@RomainWilbert
Copy link
Contributor Author

RomainWilbert commented Oct 10, 2022

Hi @badgerwithagun !

I have trouble to make the acceptance test with Oracle working. When i launch my tests inside IDE it's fine, when i make a Maven build, test is stuck in an inifite loop.

Seems that -Doracle.jdbc.javaNetNio=false argument helped with the execution. Can't exactly figure why :(

@RomainWilbert
Copy link
Contributor Author

Build is failing because of missing javadoc on lombok generated code :(

@RomainWilbert RomainWilbert deleted the feat-oracle branch November 10, 2022 14:22
@RomainWilbert RomainWilbert restored the feat-oracle branch November 10, 2022 14:24
@RomainWilbert RomainWilbert reopened this Nov 10, 2022
@skastenholz
Copy link
Contributor

@badgerwithagun Any chance this nicely done PR will get an approval by this awesome project once failing checks are resolved? Thanks.

@badgerwithagun
Copy link
Member

Absolutely @skastenholz!

If someone can fix the build I'll be happy to progress. I'll need to review it properly obviously

@skastenholz
Copy link
Contributor

Hey @RomainWilbert and @badgerwithagun, I provided the following PR RomainWilbert#4 to avoid the current build error. I assume it works this way. Otherwise please provide some guidance. Thanks.

@skastenholz
Copy link
Contributor

@badgerwithagun As discussed the checks are fixed now. Please let us know if anything (else) prevents the merging of this PR. Thank you!

@badgerwithagun
Copy link
Member

Great stuff, thanks both!

Merging.

@badgerwithagun badgerwithagun merged commit a8817b9 into gruelbox:master Mar 10, 2023
@skastenholz
Copy link
Contributor

Thanks a lot for merging! I am wondering how and when it will be available after releasing 5.2.337 on Maven Central (https://mvnrepository.com/artifact/com.gruelbox/transactionoutbox-core) since https://github.com/gruelbox/transaction-outbox/actions/runs/4383418970 failed.

@kzkvv
Copy link
Contributor

kzkvv commented Apr 6, 2023

Hi, @RomainWilbert I found that you have rewritten my previous fix for postgres, now the previous bug I fixed in this PR appears again.
Could you explain please, was it a mistake while merging or the planned change?

image

badgerwithagun pushed a commit that referenced this pull request May 14, 2023
Co-authored-by: Romain Wilbert <romain.wilbert-ext@pole-emploi.fr>
Co-authored-by: skastenholz <stefan.kastenholz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants