Skip to content

Conversation

@ssheikin
Copy link
Contributor

This change simply reflects the current reality,
as the session property is already part of the public documentation. https://trino.io/docs/current/admin/properties-client-protocol.html#protocol-spooling-enabled

Description

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

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

@cla-bot cla-bot bot added the cla-signed label Jan 19, 2026
@ssheikin ssheikin force-pushed the ssheikin/52/trino/spooling_enabled branch from 3261575 to 9206a0f Compare January 19, 2026 11:02
@ssheikin ssheikin changed the title Make SPOOLING_ENABLED public session property Make SPOOLING_ENABLED session property public Jan 19, 2026
@wendigo
Copy link
Contributor

wendigo commented Jan 19, 2026

Technically speaking it's not about "public" or "private" but "hidden" or "not hidden". Please update commit message accordingly

This change simply reflects the current reality,
as the session property is already part of the public documentation.
https://trino.io/docs/current/admin/properties-client-protocol.html#protocol-spooling-enabled
@ssheikin ssheikin force-pushed the ssheikin/52/trino/spooling_enabled branch from 9206a0f to 87c5f7a Compare January 19, 2026 13:46
@ssheikin ssheikin changed the title Make SPOOLING_ENABLED session property public Make SPOOLING_ENABLED session property not hidden Jan 19, 2026
@wendigo wendigo merged commit 85bfae5 into trinodb:master Jan 19, 2026
100 checks passed
@github-actions github-actions bot added this to the 480 milestone Jan 19, 2026
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