Skip to content

Oracle DB Config support #145

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

Merged
merged 16 commits into from
Jun 7, 2022
Merged

Oracle DB Config support #145

merged 16 commits into from
Jun 7, 2022

Conversation

sanagaraj-pivotal
Copy link
Contributor

Work in this PR:

👉 Imports component necessary jaxb model

👉 Adds support to <db:oracle-config /> component

👉 Brings in Oracle driver automatically upon recognising <db:oracle-config /> component being used

👉 Auto fills application.properties with template data that is required by spring app for oracle database connection
For example, recognising <db:oracle-config /> element, following data is filled in application.properties

spring.datasource.url=--INSERT--DB-URL-HERE-Example:jdbc:oracle:thin:@localhost:1521:XE
spring.datasource.username=--INSERT-USER-NAME--
spring.datasource.password=--INSERT-PASSWORD--
spring.datasource.driverClassName=oracle.jdbc.OracleDriver
spring.jpa.show-sql=true

👉 Removes the use of LIMIT which used to limit the query result count, since different DBs have different way of doing this limiting, a document has been created and linked in the code (Decide on how this document should live in the future) the link can be found here

@sanagaraj-pivotal sanagaraj-pivotal changed the title Adding scripting xsd Oracle DB Config support Jun 7, 2022
@fabapp2 fabapp2 merged commit e59dd6e into spring-projects-experimental:main Jun 7, 2022
@sanagaraj-pivotal sanagaraj-pivotal deleted the adding-scripting-xsd branch July 21, 2022 09:11
@fabapp2 fabapp2 added type: enhancement New feature or request Mule-3.9 labels Sep 23, 2022
@fabapp2 fabapp2 added this to the v0.12.0 milestone Sep 23, 2022
@sbrannen
Copy link

@fabapp2, did you intentionally assign this issue to me?

If so, can you please explain what you expect from me?

@fabapp2
Copy link
Contributor

fabapp2 commented Sep 28, 2022

Hi @sbrannen, I'd certainly love to get you involved ❤️
But it was my fat finger selecting you. Thought I removed you again 🤔 will do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants