dgvj-work commented on PR #1243: URL: https://github.com/apache/arrow-go/pull/1243#issuecomment-5504313756
Pushed a follow-up for the remaining unwrap cases. `storageType` now walks extension layers (with cycle protection) and is nil-safe. Double-wrapped Null is rejected in the direct, one-field, two-field, and shredded paths. `NewVariantType(NewOpaqueType(nil, ...))` returns `arrow.ErrInvalid`, and `NewShreddedVariantType` still returns the default type instead of panicking. @zeroshade this should be ready for another look. -- 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]
