Skip to content

Improve type mapping documentation for Oracle connector#13659

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
tlblessing:tb/oracle-type-mapping
Aug 18, 2022
Merged

Improve type mapping documentation for Oracle connector#13659
ebyhr merged 1 commit intotrinodb:masterfrom
tlblessing:tb/oracle-type-mapping

Conversation

@tlblessing
Copy link
Copy Markdown
Member

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Minor documentation improvement for Oracle connector for consistency.

How would you describe this change to a non-technical end user or system administrator?

Added some boilerplate intro text only for reading and writing data type sections. Fixed table format for less wrapping.

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
(x) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label Aug 12, 2022
@tlblessing tlblessing requested review from ebyhr and jhlodin August 12, 2022 20:02
@github-actions github-actions bot added the docs label Aug 12, 2022
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin left a comment

Choose a reason for hiding this comment

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

Please add "No other types are supported" below each table

@tlblessing tlblessing force-pushed the tb/oracle-type-mapping branch from 9d3847d to bc3c7f4 Compare August 15, 2022 18:09
@tlblessing
Copy link
Copy Markdown
Member Author

Please add "No other types are supported" below each table

Done in commit bc3c7f4

@tlblessing tlblessing requested a review from jhlodin August 15, 2022 18:10
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin left a comment

Choose a reason for hiding this comment

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

LGTM

@tlblessing tlblessing force-pushed the tb/oracle-type-mapping branch from bc3c7f4 to d092110 Compare August 17, 2022 17:08
@tlblessing
Copy link
Copy Markdown
Member Author

@ebyhr I think this PR is ready to merge now. Thank you!

@ebyhr ebyhr merged commit 59618ec into trinodb:master Aug 18, 2022
@github-actions github-actions bot added this to the 394 milestone Aug 18, 2022
@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Aug 18, 2022

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants