integer can have a length

[ondrajodas]

Integer může mít hodnotu délky uloženou v db - validace by tím měla projít

part of Jira task - https://keboola.atlassian.net/browse/COM-490

[michaljurecko]

@ondrajodas nezda sa mi to, ... integer ma pevnu dlzku 4 bytes -> 32 bits takze by som to ignoroval.
V manifeste/metadatach by dlzka nemala byt.
IMHO sa neda nastavit ale je pevna.

Su dalsie typy BIGINT, SMALLINT, ktore budu dlzku asi tiez reportovat.

https://docs.aws.amazon.com/redshift/latest/dg/r_Numeric_types201.html

[ondrajodas]

hmm to by ignorovat taky šlo... udělal jsem to podle mysql, tak tam bychom to měli taky přehodit

[michaljurecko]

v SNFK sa to ignoruje:

php-datatypes/src/Definition/Snowflake.php

Line 216 in c70b788

if (!is_null($length) && $length != "") {

Resp. ignorovat to treba asi skor ako sa to dostane do tejto kniznice.
Ak si nie si isty ci to ignorovat, tak sa skus spytat na Slacku.

[ondrajodas]

tímhle co posíláš to ale neignoruješ, ale vyhodíš exceptionu

[michaljurecko]

ano -> a kedze sa ziadne exceptions nevyhadzuju -> tak to znamena, ze pre INTEGER, ... typy nechodi do tejto kniznice dlzka.
... teda navrhujem, aby sa ta length unset-la uz v komponente pre INTEGER, BIGINT, ... a tu to uz vobec nebude potrebne riesit.

[ondrajodas]

ok, ani ve writerech se délka nezadává tak budeme ignorovat :)

[ondrajodas]

u mysql tomu je tak že když je INT(10) a je zerofill na true tak db doplní nuly zleva 🤦