tustvold commented on PR #7160: URL: https://github.com/apache/arrow-rs/pull/7160#issuecomment-2674197099
I wonder if supporting PartialEq for extensions is all that important, my understanding is the PartialEq support is a utility for testing. As such I wonder if we just return `None` if either argument has extensions present? IMO it would be nice if we didn't have to export an Extension trait publicly and could just support registering anything `Send + Sync + Debug`, with any necessary trait machinery kept private. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
