Skip to content

Fix inaccurate references to prepared statement header#12641

Merged
martint merged 1 commit intotrinodb:masterfrom
colebow:colebow/client-protocol-docs
Jun 1, 2022
Merged

Fix inaccurate references to prepared statement header#12641
martint merged 1 commit intotrinodb:masterfrom
colebow:colebow/client-protocol-docs

Conversation

@colebow
Copy link
Copy Markdown
Member

@colebow colebow commented Jun 1, 2022

Description

The header here is X-Trino-Prepared-Statement, but a couple references say that it is X-Trino-Prepared-Statements, which is wrong. This fixes that.

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

Fix

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

Docs

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 Jun 1, 2022
@colebow colebow requested a review from mosabua June 1, 2022 19:30
@github-actions github-actions bot added the docs label Jun 1, 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.

lgtm - confirmed in the code.

@martint martint merged commit 011993f into trinodb:master Jun 1, 2022
@github-actions github-actions bot added this to the 383 milestone Jun 1, 2022
@colebow colebow deleted the colebow/client-protocol-docs branch June 2, 2022 14:53
v-jizhang added a commit to v-jizhang/presto that referenced this pull request Jun 2, 2022
Cherry-pick of trinodb/trino#12641

Co-authored-by: Colebow <cole.bowden@starburstdata.com>
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