fix(storage): respect STORAGE_READ_FROM env var regardless of backend mode#9
Merged
nitaibezerra merged 2 commits intomainfrom Dec 26, 2025
Merged
Conversation
… mode The StorageAdapter was ignoring STORAGE_READ_FROM when STORAGE_BACKEND was not set to dual_write. Now STORAGE_READ_FROM takes precedence when explicitly set, allowing read source to be configured independently. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Use proper SQL date casting with ::date for date range filtering - Remove dependency on Django-style filter operators (__gte, __lte) - Query now properly includes all records within the date range 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
5 tasks
mauriciomendonca
pushed a commit
that referenced
this pull request
Mar 20, 2026
Correcoes baseadas no code review: - #4 (ALTO): Remover BEGIN/COMMIT explicito do rollback SQL 005 que quebrava o commit atomico do runner (COMMIT prematuro antes do registro em migration_history) - #1 (MEDIO): Remover server-side cursor desnecessario em 006_migrate_unique_ids.py — fetchall() carrega tudo em memoria de qualquer forma, o name= so adiciona confusao semantica - #9 (MEDIO): Sanitizar target_version com regex ^[0-9]{3}$ no workflow e substituir eval $CMD por execucao direta com bash array para prevenir command injection - #5 (BAIXO): Extrair _execute_with_history() em migrate.py para eliminar ~60% de duplicacao entre execute_migration() e execute_rollback(), reduzindo risco de divergencia futura Todos os 44 testes continuam passando. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This fixes the upload-to-cogfy step which was still reading from HuggingFace despite STORAGE_READ_FROM=postgres.
Test plan
🤖 Generated with Claude Code