Skip to content

Commit

Permalink
netsuite doesn't appear to allow you to change your session time zone
Browse files Browse the repository at this point in the history
  • Loading branch information
ericcj committed Nov 9, 2022
1 parent ca666a3 commit f061f04
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/metabase/driver/netsuite.clj
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,9 @@
[driver details]
((get-method driver/can-connect? :sql-jdbc) driver details))

; netsuite doesn't appear to allow you to change your session time zone and even reading it as per https://timdietrich.me/blog/netsuite-suiteql-dates-times/ doesn't work over JDBC
(defmethod sql-jdbc.execute/set-timezone-sql :netsuite [_] nil)

; TIMESTAMP columns (e.g. item.createddate) were causing "Receiver class com.netsuite.jdbc.oabase.oacb does not define or inherit an implementation of the resolved method 'abstract java.lang.Object getObject(int, java.lang.Class)' of interface java.sql.ResultSet"
; maybe this is a similar concern to how the oracle driver handles TIMESTAMPTZ?
(defmethod sql-jdbc.execute/read-column-thunk [:netsuite Types/TIMESTAMP]
Expand Down

0 comments on commit f061f04

Please sign in to comment.