Skip to content

[PLUGIN-1100] description of role property in oracle batch sink markdown#215

Merged
itsankit-google merged 1 commit intodevelopfrom
PLUGIN-1100
Feb 17, 2022
Merged

[PLUGIN-1100] description of role property in oracle batch sink markdown#215
itsankit-google merged 1 commit intodevelopfrom
PLUGIN-1100

Conversation

@itsankit-google
Copy link
Copy Markdown
Contributor

Comment thread oracle-plugin/docs/Oracle-batchsink.md Outdated

**SID/Service Name:** Oracle connection point (Database name or Service name).

**Role** Login role of the user when connecting to the database.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there an example we can give - like what kind of roles?

Copy link
Copy Markdown
Contributor Author

@itsankit-google itsankit-google Feb 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ack, added example of roles.

Copy link
Copy Markdown
Contributor

@avirmani avirmani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@itsankit-google itsankit-google merged commit 231df85 into develop Feb 17, 2022
@itsankit-google itsankit-google deleted the PLUGIN-1100 branch February 17, 2022 20:59

**SID/Service Name:** Oracle connection point (Database name or Service name).

**Role** Login role of the user when connecting to the database. For eg, NORMAL, SYSDBA, SYSOPER, etc.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@itsankit-google If you want to be consistent with the Oracle connection and Oracle batch source markdown they say "Login role of the user when connecting to the database."

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.

3 participants