-
Notifications
You must be signed in to change notification settings - Fork 66
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
Question / Feature Request : Oracle DB Support #541
Comments
Hi @andye2004, From an implementation POV I don't imagine this is too much of a big deal. I am sure Oracle has a good blob api. At the time I originally implemented the code in the Spring Content JPA module and was considering which DBs to support and which not to. I actually wanted to do Oracle - why wouldn't you - but for testing I was relying on freely available hosted things and a freely available hosted Oracle thing wasn't a thing! At least, not that I could find. Now-a-days I have a slightly more sane container-based testing strategy and there seems to be plenty of oracle containers out there. So, if you have the bandwidth to implement Oracle support then yeah please do. I'd love a contribution like that. That would be awesome. |
Hi @paulcwarren, just forked the 1.2.x branch and checked it out and when I try to run |
Are you running |
Completely fresh checkout of the source to a newly created directory. Running just standard TLDR; Attached output from two different runs, second one has S3 test disabled to highlight Azure failures. Error from first test run:-
Disabling that particular test I then get:-
I stopped at this point and just skipped tests to build what I needed for the Oracle changes. |
Apologies for the inconvenience. I see the problem. Some of these tests require environment variables to be set . Setting Probably these tests should be classed as integration tests so I will see if I can move them tomorrow. In the meantime I would suggest just commenting them out of the modules section in spring-content/pom.xml. You could also create a branch, push it and submit a prospective PR as that will trigger all of the tests to run in CI but, I guess, you probably don't want to do that just yet as you are just getting up and running. |
No worries my friend! The joys of software development :) |
Sorry Paul, question for you. Do you have any objections to me using TestContainers for the Oracle integration tests as opposed to having to run containers externally? |
No, not at all. I am not too familiar with github actions and test containers so I would be interested to see it. |
Hi @paulcwarren, I'm wondering if you, or anyone else, has attempted to add support for Oracle DB at any point? If so, were there any issues that prevented the functionality from being added?
If not, this is something I may be interested in providing a PR for. Please let me know.
Cheers, Andy.
The text was updated successfully, but these errors were encountered: