bigquery-firestore-export: clearing PARTITIONING_FIELD is blocked but the DTS API allows it
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend, data-engineering
Línea de trabajo
Empieza en functions/src/dts.ts, en la guardia de PARTITIONING_FIELD y su comentario, y luego compara la implementación de extensión upstream correspondiente heredada en el commit a34ba233. Reproduce el PATCH en un proyecto real en la ubicación us y verifica que borrar el campo tenga éxito y cambie una ejecución posterior; se considera terminado cuando ambas implementaciones permiten el valor vacío sin la constante de error obsoleta.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
bigquery-firestore-export refuses to clear PARTITIONING_FIELD, but the Data Transfer API allows it.
Setting PARTITIONING_FIELD and later clearing it fails the update with:
Cannot remove partitioning_field from an existing transfer config. The BigQuery Data Transfer API does not support clearing this parameter once it has been set. To change partitioning, you must create a new transfer config with the desired partitioning settings.
The API does support it. Creating a new transfer config is a costly workaround, since run history is stored under the old config ID and does not come with you.
Reproduction
Run against a real project, us location:
- Create a scheduled query with
partitioning_field: created_atand destination templateprobe_{run_time|"%H%M%S"}. Two runs produced tables partitionedDAY (field: created_at). - Clear it:
PATCH /v1/{name}?updateMask=paramswith"partitioning_field": "". Returns 200, and a follow-upGETreads it back as"". - Trigger another run. It carries
partitioning_field: "", reachesSUCCEEDED, and reports no error. - Compare the destination tables:
probe_152800 DAY (field: created_at) <- partitioning set
probe_152909 DAY (field: created_at) <- partitioning set
probe_153056 (none) <- after clearing
So the clear is accepted, persists, runs, and changes the output. It is not silently ignored and it does not fail.
The "you cannot un-partition an existing table" concern does not apply here: the destination template gives every run a new time-suffixed table, so nothing is mutated in place.
Where it comes from
The guard and its comment are in functions/src/dts.ts in the upstream extension:
// The BigQuery Data Transfer API does not support clearing this parameter
throw new Error(PARTITIONING_FIELD_REMOVAL_ERROR);
The kit inherited both in a34ba233. The kit and the extension differ only in what happens after the throw, so both are affected.
Suggested fix
Drop the guard and let an empty partitioning_field through the update mask, and remove PARTITIONING_FIELD_REMOVAL_ERROR along with the comment asserting the limitation. Worth confirming against the extension too, since the belief originated there.
Not verified: behaviour in locations other than us, and whether a destination table without a time-suffixed template behaves differently.
- Lenguaje dominante
- TypeScript
- Estrellas
- 978
- Forks
- 433
- Merge medio
- 20 h 44 min
- PR fusionados (30 d)
- 17
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de firebase/extensions
-
decision(storage-resize-images): kit/extension parity – ALLOWED_EVENT_TYPES is ignored, the kit reads EXT_SELECTED_EVENTSPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodecision kits
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
firebase/extensions#3273 ·
Los mantenedores suelen responder en 1 día
-
docs(storage-resize-images): kit README's Multiple instances section fails in an installed projectAbiertokits type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
firebase/extensions#3271 ·
Los mantenedores suelen responder en 1 día
-
kits type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
firebase/extensions#3270 ·
Los mantenedores suelen responder en 1 día
-
fix(storage-resize-images): kit/extension parity – BUCKET_REGION offers europe-north2 and northamerica-south1, which can't deployPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertokits type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
firebase/extensions#3268 ·
Los mantenedores suelen responder en 1 día
-
fix(storage-resize-images): kit/extension parity – onCompletion event no longer carries the object namePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertokits type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
firebase/extensions#3266 ·
Los mantenedores suelen responder en 1 día
Todos los issues de firebase/extensions
Issues similares
-
ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
xavxyz/bei-shan-lin-shui#57 ·
Los mantenedores suelen responder en 1 día
-
[BUG] rag: sentence content can cross the reported sentence boundariesPosiblemente ocupada @LJK-ikun la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
README: the Resource URI Schemas example's file://documents/* pattern only matches that literal URIAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
platformatic/mcp#361 ·
Los mantenedores suelen responder en 1 día
-
最近文档这个弹窗,左右宽度没有最小宽度限制Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
siyuan-note/siyuan#20799 · 7 comentarios ·
Los mantenedores suelen responder en 1 día