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

[Fix] Allow CallProcedure to specify mode #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcinc
Copy link

@marcinc marcinc commented Oct 6, 2022

This PR addresses #74.

There are few possible solutions to this so I'm happy to address it in a different way if required.

Possible solutions:

  1. (Current one) g.CallProcedure takes an additional mode parameter that tells it whether to run RW or RO query. This however requires the knowledge of what procedure mode is. It can be easily retrieved from the instance by running CALL dbms.procedures() (in read-only mode) which will show the mode required for a specific procedure.

  2. g.CallProcedure can build available procedures lookup by calling dbms.procedures() internally and automatically setting up the mode required for a given procedure call. This however has a downside of additional call for each g.CallProcedure invocation. I've got that solution ready to go if there is preference for it.

@marcinc marcinc changed the title Allow CallProcedure to specify mode [Fix] Allow CallProcedure to specify mode Oct 10, 2022
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.

1 participant