shrirangmhalgi commented on PR #56904: URL: https://github.com/apache/spark/pull/56904#issuecomment-4889706144
@MaxGekk could you please review. While implementing the array tests, discovered that the base `JDBCValueGetter.ArrayGetter` and `JdbcUtils.makeSetter` had no `TimeType` case - PostgreSQL returns `java.sql.Time` for `time[]` elements which loses sub-second precision. Fixed the same by using `getResultSet() + getObject(classOf[LocalTime])` per element (same approach as the scalar TimeGetter), and added a `TimeType` case to `makeSetter` for array writes. All Docker integration tests are passing. Thankyou. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
