8000 Add `ExternalFileCache` validation as option for `ExtendedOpenFileInfo` by lnkuiper · Pull Request #17205 · duckdb/duckdb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add ExternalFileCache validation as option for ExtendedOpenFileInfo #17205

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

Merged
merged 2 commits into from
Apr 22, 2025

Conversation

lnkuiper
Copy link
Contributor

This PR adds functionality to bypass the built-in cache validation for ExternalFileCache by adding the key "validate_external_file_cache" with value false, to the ExtendedOpenFileInfo of OpenFileInfo. This is not exposed to users, so it can only be used internally. This avoids doing a HEAD request for files that have been queried before.

The Iceberg/Delta extensions should be able to set this to false safely. cc @Tmonster

@duckdb-draftbot duckdb-draftbot marked this pull request as draft April 22, 2025 11:43
@lnkuiper lnkuiper marked this pull request as ready for review April 22, 2025 14:06
@Mytherin Mytherin merged commit 84db85c into duckdb:main Apr 22, 2025
49 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 18, 2025
Add `ExternalFileCache` validation as option for `ExtendedOpenFileInfo` (duckdb/duckdb#17205)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 18, 2025
Add `ExternalFileCache` validation as option for `ExtendedOpenFileInfo` (duckdb/duckdb#17205)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 19, 2025
Add `ExternalFileCache` validation as option for `ExtendedOpenFileInfo` (duckdb/duckdb#17205)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0