aloneguid / parquet-dotnet

Fully managed Apache Parquet implementation
https://aloneguid.github.io/parquet-dotnet/
MIT License
542 stars 141 forks source link

[BUG]: Scale must be less than the precision - should be "cannot be larger than" #416

Closed sbrickey closed 5 months ago

sbrickey commented 8 months ago

Library Version

3.10.0

OS

Windows

OS Architecture

64 bit

How to reproduce?

new Parquet.Data.DecimalDataField("abc", 4, 4);

in this case, the decimal does not allow for values >= 1.0 but DOES allow for values < 1.0; in the example above, the values must exist between 0.0000 and 0.9999 as there are only 4 decimal digits available.

Failing test

No response

sbrickey commented 8 months ago

note that I am INTENTIONALLY using the 3.x version not the 4.x series, as I am handling the multithreaded writing of rowgroups myself.

aloneguid commented 5 months ago

I will check latest version, but it's unlikely a fix will be added to v3 unless you are willing to do it yourself.

aloneguid commented 5 months ago

latest v4 seems to be ok