lidavidm commented on code in PR #868:
URL: https://github.com/apache/arrow-adbc/pull/868#discussion_r1250838355
##########
c/driver/postgresql/connection.cc:
##########
@@ -1025,6 +1025,18 @@ AdbcStatusCode PostgresConnection::SetOption(const char*
key, const char* value,
}
return ADBC_STATUS_OK;
}
+
+ if (std::strcmp(key, "TIME ZONE") == 0) {
+ std::string query = std::string("SET TIME ZONE '") + std::string(value) +
"'";
Review Comment:
If we do want an option for this, it should be something like
`adbc.postgresql.time_zone` based on other drivers
But if we just need it for ingestion, maybe we could do everything inside a
transaction, and set/restore the timezone within the transaction?
--
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]
