AliRana30 commented on PR #49086: URL: https://github.com/apache/arrow/pull/49086#issuecomment-3841896779
@pitrou @HuaHuaY I've updated the implementation to use std::any_of as suggested by @HuaHuaY for cleaner null checking. However, I understand @pitrou's concern that this may not be a real bug if there's no evidence in CI. The fundamental question remains: should Table::Make validate null inputs, or is that the caller's responsibility? I'm happy to close this if the team decides validation isn't needed. Just wanted to address the code style feedback first. -- 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]
