Snowflake incremental import with PK - convert empty values to null

[ErikZigo]

Nasel jsem chybu v metode Keboola\Db\Import\Snowflake\ImportBase::insertOrUpdateTargetTable pri incrementalnim zapisu do tabulky s primarnim klicem

Pri aktualizaci existujicich zaznamu se ignoruje nastaveni $convertEmptyValuesToNull . Aplikuje se pouze pri vkladani novych zaznamu.

[ErikZigo]

Navic pokud tabulka muze obsahovat NULL hodnoty, tak pak je nefunkcni i podminka pro aktualizaci pouze zmenenych zaznamu.

Porovnavaji se tam zdrojova a cilova data. V tomhle pripade ale muze dojit k situaci cokoliv != NULL. To nikdy nevratitrue/false, ale vzdy NULL. Takze zmeny se do cilove databaze nepromitnou!

[ErikZigo]

@Halama koukl bys na ten PR? Jestli to neni ko.otina? Diky

RS testy padaj kvuli starym credentials na travisu. Moc nechapu proc failuje MySQL test. U me v dockeru to funguje.

[Halama]

Ty testy by mely prochazet vsechny.

[Halama]

Ten Redshift jsem nejspis asi zkurvil ja kdyz jsem tam mazal db, ten teda fixnu.

[ErikZigo]

https://travis-ci.org/keboola/php-db-import/builds/302491614

RS dava ze neexistuje databaze

[Halama]

Fixnu ted ty RS testy ale zkouknul bych to radsi az zitra rano.

[Halama]

Nastavil jsem nove RS credentials a restartnul to https://travis-ci.org/keboola/php-db-import/builds/302501478

[ErikZigo]

Zitrek asi v pohode. Me to neblokuje. Ja koukam ze stejny problem bude i u redshiftu. Tak na to delam taky test a fix.

[Halama]

ok super. Kdyztak prosim mrkni i na ty mysql testy proc failuji.

[Halama]

vypada to ze RS uz bezi.

[ErikZigo]

MySQL faily jsou zpusobeny novejsi verzi PhpUnitu. To nebude problem fixnout.

[Halama]

ok super, tak to prosim fixni asi v tomhle PR at je to zeleny :)