apache / apache/arrow-java

[JDBC] Exception handling for non-simple types swallows actual error during processing

Ouverte
#94 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Java
Étoiles
94
Forks
152
Merge moyen
3 j 16 h
PR mergées (30 j)
11

Description

### Describe the bug, including details regarding any error messages, version, and platform.

It appears as though apache/arrow#39356 added support for better error messages when an error is encountered during JDBC processing. However, some `MinorType` such as Decimal (and various others) do not have an associated ArrowType and throw an exception trying to get the additional information leading to the original error being suppressed.

Decimal MinorType:
https://github.com/apache/arrow/blob/main/java/vector/src/main/java/org/apache/arrow/vector/types/Types.java#L583

Callsite for suppressing error:
https://github.com/apache/arrow/blob/main/java/vector/src/main/java/org/apache/arrow/vector/types/Types.java#L821

called from consumer (where minorType's arrow type is null)
https://github.com/apache/arrow/blob/main/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/CompositeJdbcConsumer.java#L55

```
java.lang.RuntimeException: Error occurred while consuming data.
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.consumeData(ArrowVectorIterator.java:121)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.load(ArrowVectorIterator.java:164)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.next(ArrowVectorIterator.java:183)
...
Caused by: java.lang.UnsupportedOperationException: Cannot get simple type for type DECIMAL
at org.apache.arrow.vector.types.Types$MinorType.getType(Types.java:821)
at org.apache.arrow.adapter.jdbc.consumer.CompositeJdbcConsumer.consume(CompositeJdbcConsumer.java:55)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.consumeData(ArrowVectorIterator.java:107)
... 95 more
```

suppressed error (via debugger):
```
java.lang.UnsupportedOperationException: BigDecimal precision cannot be greater than that in the Arrow vector: 39 > 38
at org.apache.arrow.vector.util.DecimalUtility.checkPrecisionAndScale(DecimalUtility.java:99)
at org.apache.arrow.vector.DecimalVector.set(DecimalVector.java:365)
at org.apache.arrow.adapter.jdbc.consumer.DecimalConsumer.set(DecimalConsumer.java:79)
at org.apache.arrow.adapter.jdbc.consumer.DecimalConsumer$NullableDecimalConsumer.consume(DecimalConsumer.java:101)
at org.apache.arrow.adapter.jdbc.consumer.CompositeJdbcConsumer.consume(CompositeJdbcConsumer.java:50)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.consumeData(ArrowVectorIterator.java:107)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.load(ArrowVectorIterator.java:164)
at org.apache.arrow.adapter.jdbc.ArrowVectorIterator.next(ArrowVectorIterator.java:183)
```

It is somewhat difficult to come up with a simple reproduction, but I getting this error by rounding decimals that should be too large to fit into a `decimal(38,10)` -- if a decimal exceeds this (such as the numeric value of `99999999999999999999999999999.9999999999` which requires 39 digits of precision) this error is thrown.

This is happening on version 15.0.0 and up. (was not occurring in 14.0.X and below).

### Component(s)

Java

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par CompositeJdbcConsumer.consume, puis examinez Types.java, DecimalConsumer.java et ArrowVectorIterator.consumeData pour retracer la façon dont l’erreur de précision Decimal d’origine est remplacée. Reproduisez une valeur JDBC dépassant decimal(38,10) et vérifiez que le traitement signale l’erreur de précision sous-jacente au lieu de l’UnsupportedOperationException provenant de MinorType.getType().

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
databases
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.