Skip to content

Conversation

@walkabout21
Copy link

@walkabout21 walkabout21 commented May 31, 2023

Add support for sqlcl in Oracle adapter by checking for an executable sql binary and defaulting to sqlplus if it doesn't exist.

@walkabout21
Copy link
Author

@tpope , just checking if you need any revisions on this. Would have like to have included some kind of "opt out" for sqlcl, but not sure how it could be implemented.

@tpope
Copy link
Owner

tpope commented Jun 6, 2023

Seeing this code reminded me, all you need to do to opt in is:

let g:dbext_default_ORA_bin = 'sql'

Given that, I think we can probably leave the default as is? Changing the default would mean that Dadbod and dbext would have different defaults, which isn't the worst thing in the world, but also I'm not sure it's worth it just to save one line of configuration.

@walkabout21
Copy link
Author

That makes sense. I didn't see anything about that in the docs so I could roll back my code revision and just add the opt in config info to the oracle adapter section of the documentation for the PR?

@walkabout21
Copy link
Author

walkabout21 commented Jun 26, 2023

@tpope I rolled back the code change and added a note in the documentation explaining how to override sqlplus as the default binary

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.

2 participants