Revisión de código para SQL

## Lista de verificación de revisión de PR de SQL (copiar en el comentario de revisión)
 
### Migraciones
- [ ] Versión única; el nombre del archivo tiene el ID del ticket
- [ ] Riesgo de bloqueo: bajo / medio / alto (Habilidad de seguridad de migración)
- [ ] CONCURRENTLY para índice de producción en tablas grandes
- [ ] SQL de rollback en db/rollback/ o cuerpo del PR
- [ ] Orden de despliegue: migración antes de la aplicación
 
### Consultas
- [ ] EXPLAIN (ANALYZE, BUFFERS) adjunto desde staging
- [ ] Sin regresión de escaneo secuencial en tablas grandes
- [ ] Consultas parametrizadas ($1 / binds nombrados)
- [ ] LIMIT en puntos finales de lista
 
### Seguridad
- [ ] Privilegio mínimo del rol (no superusuario)
- [ ] search_path establecido o nombres con prefijo de esquema
- [ ] Políticas RLS si la tabla es de inquilino

EXPLAIN (ANALYZE, BUFFERS)
SELECT id, total, created_at
FROM orders
WHERE tenant_id = $1 AND status = 'open'
ORDER BY created_at DESC
LIMIT 100;

Index Scan using idx_orders_tenant_status on orders
  Buffers: shared hit=120 read=4
  Execution Time: 1.2 ms

SELECT * FROM orders WHERE tenant_id = $1 OR tenant_id = $2;
-- Sin EXPLAIN, sin LIMIT, OR previene el índice en algunos planificadores

Solicitar cambios:
1. Adjuntar EXPLAIN (ANALYZE, BUFFERS) desde staging
2. Reemplazar OR con el patrón IN (...) o UNION ALL
3. Listar columnas explícitamente; eliminar SELECT *
4. Ver la Habilidad de Revisión EXPLAIN para la rúbrica de aprobación/rechazo

# MALO: N+1
for order in orders:
    items = session.query(Item).filter_by(order_id=order.id).all()
 
# BUENO: evidencia de joinedload en el PR
orders = session.query(Order).options(joinedload(Order.items)).filter(...)

-- rollback/V20260709_128__orders_priority_down.sql
DROP INDEX CONCURRENTLY IF EXISTS idx_orders_priority;
ALTER TABLE orders DROP COLUMN IF EXISTS priority;