-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added replace_id attrib to sqlFileStore #58
Conversation
python-lib/tc_etl_lib/README.md
Outdated
- :param `replace_id` opcional: diccionario `tipo de entidad` => `lista de atributos replace_id`. | ||
Reemplaza el ID de las entidades del tipo o tipos especificados, por un valor construido a partir de la lista de atributos indicados en este parámetro, separados por `_`. | ||
Imita el comportamiento del atributo `replaceId` de los flujos históricos de URBO-DEPLOYER, para poder usar este *store* en ETLs que alimenten *singletons*. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incluir entrada en el changelog (de tipo "Add") por encima de la marca de 0.7.0 (December 233rd, 2022)
(de paso, corregir el typo en ese 233 ;)
¿Cómo de urgente/necesario es? ¿Cerramos versión 0.8.0 tras mezclar este fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix en c8a6a95.
En las ETLs de accommodation-vertical necesitábamos esta funcionalidad. Si se cierra versión lo usamos ya, si no mantenemos el workaround que tenemos en esas ETLs.
Co-authored-by: Fermín Galán Márquez <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM |
Actualmente el store
sqlFileStore
no puede usarse para generar un fichero SQL de carga de entidades singleton (tal como las define URBO-DEPLOYER, conreplaceId
), porque esas entidades necesitan que se sobrescriba elid
de la entidad antes de guardar en base de datos.Esta PR soluciona el problema añadiendo al
sqlFileStore
un parámetro opcionalreplace_id
que simula el comportamiento de un flujo histórico configurado con unreplaceId
. El sqlFileStore genera un id de entidad a partir de los atributos especificados, antes de construir la orden SQL.