lidavidm commented on code in PR #857:
URL: https://github.com/apache/arrow-java/pull/857#discussion_r2377837429
##########
adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/CompositeJdbcConsumer.java:
##########
@@ -46,9 +45,9 @@ public void consume(ResultSet rs) throws SQLException,
IOException {
BaseConsumer consumer = (BaseConsumer) consumers[i];
JdbcFieldInfo fieldInfo =
new JdbcFieldInfo(rs.getMetaData(),
consumer.columnIndexInResultSet);
- ArrowType arrowType = consumer.vector.getMinorType().getType();
+
throw new JdbcConsumerException(
- "Exception while consuming JDBC value", e, fieldInfo, arrowType);
+ "Exception while consuming JDBC value", e, fieldInfo,
consumer.vector.getMinorType());
Review Comment:
ah, oops, sorry about that :grimacing:
To be fair, I do think it's weird that we have a `getMinorType` but not a
`getType`
--
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]
