Delal jsem to podle ukazky ale prehlidl jsem ze Common writer v4 pouziva incremental flag pro temporary tabulku, kdezdo v5 pouziva temporary.
V tomhle writeru jsem to nakonec udelal tak ze jsem vytvoril novou metodu createStaging ktera zaklada docasnou tabulku.
Zaroven
jsem to pokryl v testech
vyhodil jsem nepouzivane/obsolete metody
do testu doplnil test vsech public metod Writer\Snowflake ktere aplikace pouziva
taky jsem upravil generovani tmp name pro staging tabulky po vzoru Common writeru
Puvodni PR je sice mergnuty, ale neni nasazeny. Takze zadny problem neni.
V mym poslednim PR https://github.com/keboola/db-writer-snowflake/pull/47 byl bug kdy pri incrementalnim zapisu to cilovou tabulku zakladalo jako
temporary
.Delal jsem to podle ukazky ale prehlidl jsem ze Common writer v4 pouziva
incremental
flag pro temporary tabulku, kdezdo v5 pouzivatemporary
. V tomhle writeru jsem to nakonec udelal tak ze jsem vytvoril novou metoducreateStaging
ktera zaklada docasnou tabulku.Zaroven
Writer\Snowflake
ktere aplikace pouzivaPuvodni PR je sice mergnuty, ale neni nasazeny. Takze zadny problem neni.