I think you're right that this was intended to skip NULL fields; however, I don't think this is a problem anymore.

This check was needed because we used to walk the Avro schema to create the converter. Now, we walk the Parquet GroupType to create the converter. Unless I'm misreading the code, I don't think we need to worry about this check anymore.