Skip to content

Document ALTER TABLE EXECUTE OPTIMIZE for Delta Lake connector#11526

Merged
findepi merged 1 commit intotrinodb:masterfrom
jhlodin:jl/delta-optimize
Mar 25, 2022
Merged

Document ALTER TABLE EXECUTE OPTIMIZE for Delta Lake connector#11526
findepi merged 1 commit intotrinodb:masterfrom
jhlodin:jl/delta-optimize

Conversation

@jhlodin
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin commented Mar 16, 2022

Description

Add documentation for the OPTIMIZE command to the Delta Lake connector doc, and the corresponding ALTER TABLE support as well. Reuses similar content from the Hive connector doc - https://trino.io/docs/current/connector/hive.html#alter-table-execute

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?

Add documentation explaining how the ALTER TABLE EXECUTE OPTIMIZE command works in Delta Lake catalogs.

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:

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

@cla-bot cla-bot bot added the cla-signed label Mar 16, 2022
@jhlodin jhlodin requested review from hashhar and mosabua March 16, 2022 20:56
@hashhar hashhar requested review from alexjo2144 and removed request for hashhar March 16, 2022 21:07
@github-actions github-actions bot added the docs label Mar 16, 2022
@findepi findepi requested a review from findinpath March 18, 2022 11:08
Copy link
Copy Markdown
Contributor

@findinpath findinpath 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. Thank you.
I've verified that the look & feel is the same as for the Iceberg connector.

@jhlodin
Copy link
Copy Markdown
Contributor Author

jhlodin commented Mar 23, 2022

@findepi @findinpath If this looks good, can we have this merged?

@mosabua
Copy link
Copy Markdown
Member

mosabua commented Mar 23, 2022

@ebyhr willing to merge this?

@findepi
Copy link
Copy Markdown
Member

findepi commented Mar 24, 2022

@ebyhr willing to merge this?

Alex did not review yet.

@jhlodin jhlodin force-pushed the jl/delta-optimize branch from a7eb612 to c73e97f Compare March 24, 2022 16:46
@findepi findepi merged commit d94f22e into trinodb:master Mar 25, 2022
@jhlodin jhlodin deleted the jl/delta-optimize branch March 25, 2022 15:49
@github-actions github-actions bot added this to the 375 milestone Mar 25, 2022
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.

5 participants