rok commented on code in PR #37298:
URL: https://github.com/apache/arrow/pull/37298#discussion_r1422635671
##########
cpp/src/arrow/c/bridge_test.cc:
##########
@@ -3694,7 +3695,10 @@ TEST_F(TestSchemaRoundtrip, Dictionary) {
}
TEST_F(TestSchemaRoundtrip, UnregisteredExtension) {
- TestWithTypeFactory(uuid, []() { return fixed_size_binary(16); });
+ TestWithTypeFactory(complex128, []() {
Review Comment:
Some of these were checking registering / deregistering mechanics for
extension types and it wasn't practical to use UUID type for that so I either
removed them or replaced with something equivalent. These tests typically
assumed uuid was not globally available extension at the start.
--
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
