Skip to content

Improve split weight documentation#12684

Closed
pettyjamesm wants to merge 1 commit intotrinodb:masterfrom
pettyjamesm:split-weight-docs
Closed

Improve split weight documentation#12684
pettyjamesm wants to merge 1 commit intotrinodb:masterfrom
pettyjamesm:split-weight-docs

Conversation

@pettyjamesm
Copy link
Copy Markdown
Member

Description

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

Documentation improvement

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

Only to documentation.

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

This change only adds additional context to documentation.

Documentation

( ) No documentation is needed.
( ) 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 Jun 3, 2022
@pettyjamesm pettyjamesm requested a review from findepi June 3, 2022 17:32
@pettyjamesm pettyjamesm marked this pull request as ready for review June 3, 2022 17:32
@github-actions github-actions bot added the docs label Jun 3, 2022
@findepi findepi requested review from alexjo2144, martint and sopel39 and removed request for findepi June 3, 2022 20:45
@pettyjamesm
Copy link
Copy Markdown
Member Author

@sopel39 / @alexjo2144 / @martint - any chance you can review / merge this documentation-only change PR?

Comment thread core/trino-spi/src/main/java/io/trino/spi/SplitWeight.java
@martint
Copy link
Copy Markdown
Member

martint commented Jun 11, 2022

Merged as 682f5ed

@martint martint closed this Jun 11, 2022
@pettyjamesm pettyjamesm deleted the split-weight-docs branch June 13, 2022 14:08
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.

2 participants