aiguofer commented on code in PR #840:
URL: https://github.com/apache/arrow-java/pull/840#discussion_r2384205659
##########
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:
ohhh I copy pasted this from navigating to the decompiled class in
`AvaticaResultSet`... I'll change it :)
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
