Skip to content

Style edits of type mapping for Delta Lake connector#13120

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
jhlodin:jl/delta-type-mapping
Jul 25, 2022
Merged

Style edits of type mapping for Delta Lake connector#13120
ebyhr merged 1 commit intotrinodb:masterfrom
jhlodin:jl/delta-type-mapping

Conversation

@jhlodin
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin commented Jul 7, 2022

Description

Clean up type mapping documentation for Delta Lake

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

Improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Documentation

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

Edit the type mapping section of Delta Lake documentation for style and consistency

Related issues, pull requests, and links

Uses #11564 for format and style

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:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Jul 7, 2022
@jhlodin jhlodin requested review from ebyhr and mosabua July 7, 2022 21:18
@jhlodin jhlodin added the docs label Jul 7, 2022
Copy link
Copy Markdown
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

Looks good.

@ebyhr ebyhr merged commit 4c470ae into trinodb:master Jul 25, 2022
@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Jul 25, 2022

Merged, thanks!

@github-actions github-actions bot added this to the 392 milestone Jul 25, 2022
@jhlodin jhlodin deleted the jl/delta-type-mapping branch July 26, 2022 16:12
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