thswlsqls commented on PR #8758: URL: https://github.com/apache/paimon/pull/8758#issuecomment-5032846353
Thanks, good catch. You're right — `ArrowUtils.toArrowField()` forces the map key to `notNull()`, so the key writer has to stay non-nullable regardless of `keyType.isNullable()`; otherwise a null key gets written instead of failing loud, which contradicts the schema. Fixed — the key writer is back to non-nullable (the value, array-element, and row-field branches still mirror each nested type's own nullability, which is what the schema declares for them), and I added a null-key test. Just pushed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
