Repository navigation
Conversation
…HOME. DuckDB defaults to using $HOME for files, but serverless environments don't always set that. When that value is not available, we'll use a default.
|
One test is failing |
o-nikolas
left a comment
There was a problem hiding this comment.
I'd actually argue this is the responsibility of the admin to configure these environments with a HOME directory. I had planned to configure that as part of our CDK configuration for the Lambda executor, but it must have gotten dropped in the flurry of CRs I put together
|
I actually looked into this a little more based on the CI failure and I think my solution only works on Duck 1.5.0+, which is another argument against this fix; we'd at least have to gate it behind a version check AND still add the requirements to the docs (and fix the CDK for the system test runner on the AWS side) either way. Before 1.5 the So this fix would/could only work in 1.5.0+, we'd either need a more robust fix (like using But based on Niko's comment, I'll close this PR and put up a new one this afternoon which just adds some docs and makes it explicit that this is an environment requirement. We can revisit if we want to add this later, gate it behind a proper Duck version check, etc, but the docs should be updated either way and would be universal instead of only fixing |
DuckDB defaults to using $HOME for files, but serverless environments don't always set that. When that value is not available, we'll use a default.
Specifically noticed this when the DuckDB system, tests failed on Lambda Executor, but should apply to other serverless environments and maybe some other non-standard installations that don't set $HOME. If $HOME isn't set, then fall back on a temp file.
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.