aiguofer commented on code in PR #840:
URL: https://github.com/apache/arrow-java/pull/840#discussion_r2404179033
##########
flight/flight-sql-jdbc-core/src/main/java/org/apache/arrow/driver/jdbc/ArrowFlightJdbcVectorSchemaRootResultSet.java:
##########
@@ -102,6 +106,33 @@ void populateData(final VectorSchemaRoot vectorSchemaRoot,
final Schema schema)
execute2(new ArrowFlightJdbcCursor(vectorSchemaRoot),
this.signature.columns);
}
+ /**
+ * The default method in AvaticaResultSet does not properly handle
TIMESTASMP_WITH_TIMEZONE, so we
+ * override here to add support.
+ *
+ * @param columnIndex the first column is 1, the second is 2, ...
+ * @return
+ * @throws SQLException
+ */
+ @Override
+ public Object getObject(int columnIndex) throws SQLException {
+ this.checkOpen();
+
+ Cursor.Accessor accessor;
+ try {
+ accessor = accessorList.get(columnIndex - 1);
+ } catch (IndexOutOfBoundsException var3) {
Review Comment:
@lidavidm btw, this has been fixed already
--
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]
