Re-enable SQL test
- Dominant language
- C++
- Stars
- 60
- Forks
- 100
- Avg merge
- 1h 50m
- Merged PRs (30d)
- 25
Description
As of now SQL test for the extension is skipped because certain data is deleted from external website: https://github.com/duckdb/duckdb-httpfs/blob/85ac4667bcb0d868199e156f8dd918b0278db7b9/test/sql/copy/csv/test_csv_httpfs.test#L11-L12
When I was working on cached httpfs (which adds caching and parallel read wrapper around httpfs), I checked the CSV file into repository so it's hermetic.
Reference: https://github.com/dentiny/duck-read-cache-fs/tree/main/test/data
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with test/sql/copy/csv/test_csv_httpfs.test at the skipped lines referenced in the issue, then inspect the test/data approach in the linked duck-read-cache-fs repository. Add the required CSV fixture to this repository and re-enable the SQL test; done means the test runs hermetically without relying on deleted external data.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, sql
- Domain
- databases, testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100