fix(cdk): Surface HTTP cache operational errors#1018
fix(cdk): Surface HTTP cache operational errors#1018devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: bot_apk <apk@cognition.ai>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksTesting This CDK VersionYou can test this version of the CDK using the following: # Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1778321821-gitlab-cache-error-message#egg=airbyte-python-cdk[dev]' --help
# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1778321821-gitlab-cache-error-messagePR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
Runtime test resultsTested the Python CDK library changes locally with shell-only runtime assertions; no browser recording was needed. Escalation / limitation: I did not run a live GitLab sync. I directly exercised the CDK runtime paths changed by the PR by forcing the exception states that would otherwise require a flaky requests-cache SQLite concurrency failure.
Commands |
|
↪️ Triggering Reason: CDK PR is linked to a connector oncall issue, CI is passing, and no AI review marker is present. |
Summary
OperationalErrorfailures from the cached HTTP session with a deterministicAirbyteTracedExceptionmessage:Internal HTTP response cache failed.internal_messageand classify the failure assystem_error.system_error.Review & Testing Checklist for Human
OperationalErrorwrapper is scoped narrowly enough to requests-cache SQLite failures surfaced through the HTTP session.use_cache: trueunder concurrent reads.Notes
config_erroronly when underlying failures are not config-only.poetry run ruff check .poetry run mypy airbyte_cdkpoetry run pytest unit_tests/sources/streams/http/test_http_client.py unit_tests/sources/streams/concurrent/test_concurrent_read_processor.py -qLink to Devin session: https://app.devin.ai/sessions/214802548503443494c17e90e631e677