lidavidm commented on code in PR #4775:
URL: https://github.com/apache/arrow-adbc/pull/4775#discussion_r4023360177
##########
c/driver/postgresql/postgres_type.h:
##########
@@ -581,6 +586,8 @@ class PostgresTypeResolver {
}
private:
+ friend class PostgresTypeTest_BuiltinResolver_Test;
+
std::unordered_map<uint32_t, PostgresType> mapping_;
// We can't use PostgresTypeId as an unordered map key because there is no
// built-in hasher for an enum on gcc 4.8 (i.e., R 3.6 on Windows).
Review Comment:
I'll fix this separately
##########
c/driver/postgresql/postgres_type.h:
##########
@@ -581,6 +586,8 @@ class PostgresTypeResolver {
}
private:
+ friend class PostgresTypeTest_BuiltinResolver_Test;
Review Comment:
Added a getter
--
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]