Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update description of 'max_temp_directory_size' #13724

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

szarnyasg
Copy link
Collaborator

@szarnyasg szarnyasg commented Sep 3, 2024

Update description of configuration option

Copy link
Contributor

Sep 3, 2024

#10978

The description of that PR should explain what the default values are, it's a little complicated and this new description doesn't quite cover it. I think docs is a better place to address this

Essentially it defaults to the space available at the temp_directory path, or unlimited if temp_directory is not set

@szarnyasg szarnyasg requested a review from Tishj September 4, 2024 08:57
@szarnyasg szarnyasg force-pushed the max_temp_directory_size-description branch from d769b9f to 99b8793 Compare September 4, 2024 12:18
@duckdb-draftbot duckdb-draftbot marked this pull request as draft September 4, 2024 12:18
@szarnyasg szarnyasg marked this pull request as ready for review September 4, 2024 15:25
@szarnyasg szarnyasg requested a review from Mytherin September 7, 2024 16:05
@szarnyasg szarnyasg force-pushed the max_temp_directory_size-description branch from 99b8793 to ba3808c Compare September 11, 2024 13:21
@duckdb-draftbot duckdb-draftbot marked this pull request as draft September 11, 2024 13:21
@szarnyasg szarnyasg marked this pull request as ready for review September 11, 2024 13:22
@Mytherin
Copy link
Collaborator

Thanks!

@Mytherin Mytherin merged commit 1d7e05c into duckdb:main Sep 12, 2024
43 checks passed
github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
Update description of 'max_temp_directory_size' (duckdb/duckdb#13724)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
Update description of 'max_temp_directory_size' (duckdb/duckdb#13724)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants