[ 
https://issues.apache.org/jira/browse/HIVE-28358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17861355#comment-17861355
 ] 

Valentino Pinna commented on HIVE-28358:
----------------------------------------

h3. What changes were proposed in this pull request?

This PR proposes an implementation for the getClob(int) and getClob(String) 
methods of 
[HiveBaseResultSet|https://github.com/apache/hive/blob/fe2e17c3ad4773a4b1066ac525f7de2a86572eca/jdbc/src/java/org/apache/hive/jdbc/HiveBaseResultSet.java].
h3. Why are the changes needed?

please see HIVE-28358.
h3. Does this PR introduce _any_ user-facing change?

Yes. The current getClob() methods of HiveBaseResultSet just throw an 
exception. The proposed change returns a 
[SerialClob|https://docs.oracle.com/javase/8/docs/api/javax/sql/rowset/serial/SerialClob.html]
 already initialized with a char array obtained from the corresponding 
getString() methods.
h3. Is the change a dependency upgrade?

No
h3. How was this patch tested?

Testing was done inside the "poorly designed application". Other additional 
tests were not deemed useful since the functionality is not present in the 
master release.

> Enable JDBC getClob retrieval from String columns
> -------------------------------------------------
>
>                 Key: HIVE-28358
>                 URL: https://issues.apache.org/jira/browse/HIVE-28358
>             Project: Hive
>          Issue Type: Improvement
>          Components: JDBC
>            Reporter: Valentino Pinna
>            Priority: Trivial
>
> To allow a "poorly designed application" that retrieve Strings via getClob() 
> method/s to work, this small improvement supports the two methods via 
> delegation to getString() and construction of a SerialClob instance.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to