kezhuoquan opened a new pull request, #6424:
URL: https://github.com/apache/hive/pull/6424
## What changes were proposed in this pull request?
`JdbcColumn.columnClassName()` currently maps `Types.DECIMAL` to
`BigInteger.class.getName()`, which is incorrect for DECIMAL semantics.
This PR changes the mapping to `BigDecimal.class.getName()`.
## Why are the changes needed?
JDBC `DECIMAL` should map to `java.math.BigDecimal`, not `BigInteger`.
The current behavior may cause type mismatch and incorrect metadata
interpretation by JDBC clients.
## Does this PR introduce _any_ user-facing change?
Yes.
For DECIMAL columns, JDBC metadata now reports
`java.math.BigDecimal` as the column class name.
## How was this patch tested?
- Built JDBC module successfully:
- `mvn -pl jdbc -DskipTests compile`
- Verified code change in:
- `jdbc/src/java/org/apache/hive/jdbc/JdbcColumn.java`
--
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]
