(kyuubi) branch master updated: [KYUUBI #6828][FOLLOWUP] Fix NPE in KyuubiBaseResultSet::getBigDecimal

[feiwang]

This is an automated email from the ASF dual-hosted git repository.

feiwang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new f0c31e2f78 [KYUUBI #6828][FOLLOWUP] Fix NPE in 
KyuubiBaseResultSet::getBigDecimal
f0c31e2f78 is described below

commit f0c31e2f78d991f9b36329499e48db54ff8a5a5a
Author: Wang, Fei <fwan...@ebay.com>
AuthorDate: Tue Apr 22 20:46:43 2025 -0700

    [KYUUBI #6828][FOLLOWUP] Fix NPE in KyuubiBaseResultSet::getBigDecimal
    
    ### Why are the changes needed?
    
    It is missed in #6828
    
https://github.com/apache/hive/blob/733d4f09011f27bea17a5301537a9c6174f28874/jdbc/src/java/org/apache/hive/jdbc/HiveBaseResultSet.java#L151-L159
    
    ### How was this patch tested?
    
    Minor change.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #7038 from turboFei/fixNPE.
    
    Closes #6828
    
    2785e97be [Wang, Fei] Fix NPE
    
    Authored-by: Wang, Fei <fwan...@ebay.com>
    Signed-off-by: Wang, Fei <fwan...@ebay.com>
---
 .../src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiBaseResultSet.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git 
a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiBaseResultSet.java
 
b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiBaseResultSet.java
index d9576a2986..ffc3008cf3 100644
--- 
a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiBaseResultSet.java
+++ 
b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiBaseResultSet.java
@@ -67,6 +67,9 @@ public abstract class KyuubiBaseResultSet implements 
SQLResultSet {
   @Override
   public BigDecimal getBigDecimal(int columnIndex) throws SQLException {
     final Object val = getObject(columnIndex);
+    if (val == null) {
+      return null;
+    }
     if (val instanceof BigDecimal) {
       return (BigDecimal) val;
     }