-
Notifications
You must be signed in to change notification settings - Fork 3k
Update the Oracle Database and Oracle Client requirements for n8n #3838
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
Conversation
Updated the note on Oracle Client Libraries for thick mode.
Clarified requirements for Oracle Database and Client Libraries.
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
|
||
| Requires Oracle Database **19c or later**. | ||
| For thick mode, use Oracle Client Libraries **19c or later**. | ||
| For advanced Oracle Database features like High Availability (HA) and Sharding, also requires Oracle Client Libraries **19c or later**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no single feature called 'High Availability'. Perhaps say AC & TAC here instead? However these work with older clients so I'm not sure what the intent of the clarification is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to give an example of what the 'Oracle Database advanced features' can entail. But I will remove it if it does not add any value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is benefit mentioning features that are gained, but "HA" is not a feature. There are other things that make up HA that are part of Thin mode already, e.g connection re-establishment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood.
I have replaced HA with TAC
sudarshan12s
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes LGTM
|
@HumanistSerif Request your approval for this doc change for Oracle Database |
|
@Joffcom Request your help to merge this, if everything is fine |
Update the documentations with specific Oracle Database and Oracle Client requirements for n8n.
Note
Clarifies Oracle Database 19c requirement and that advanced features (TAC, Sharding) need Oracle Client Libraries 19c, plus minor credential wording/format updates.
docs/integrations/builtin/app-nodes/n8n-nodes-base.oracledb/index.md:docs/integrations/builtin/credentials/oracledb.md:Written by Cursor Bugbot for commit 6634927. This will update automatically on new commits. Configure here.