I reviewed my changes again, and it looks like the code already correctly plumbs type_length for non-FLBA fields through. Besides, from what I understand, specifying type_length on a non-FLBA field is an uncommon use case since Parquet currently does not specify type_length for those fields by default.

If this turns out to be problematic later, let's revisit this then. Is that fine?