Merge pull request #783 from daneshk/2201.12.x

[2201.12.X] Fix the Clob out parameter issue in stored procedure

Author: daneshk

ballerina/Ballerina.toml

@@ -1,7 +1,7 @@
 [package]
 org = "ballerina"
 name = "sql"
-version = "1.17.0"
+version = "1.17.1"
 authors = ["Ballerina"]
 keywords = ["database", "client", "network", "SQL", "RDBMS"]
 repository = "https://github.com/ballerina-platform/module-ballerina-sql"
@@ -15,11 +15,11 @@ graalvmCompatible = true
 [[platform.java21.dependency]]
 groupId = "io.ballerina.stdlib"
 artifactId = "sql-native"
-version = "1.17.0"
-path = "../native/build/libs/sql-native-1.17.0.jar"
+version = "1.17.1"
+path = "../native/build/libs/sql-native-1.17.1-SNAPSHOT.jar"
 
 [[platform.java21.dependency]]
-path = "../test-utils/build/libs/sql-test-utils-1.17.0.jar"
+path = "../test-utils/build/libs/sql-test-utils-1.17.1-SNAPSHOT.jar"
 scope = "testOnly"
 
 [[platform.java21.dependency]]

ballerina/CompilerPlugin.toml

@@ -3,4 +3,4 @@ id = "sql-compiler-plugin"
 class = "io.ballerina.stdlib.sql.compiler.SQLCompilerPlugin"
 
 [[dependency]]
-path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.17.0.jar"
+path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.17.1-SNAPSHOT.jar"

ballerina/Dependencies.toml

@@ -44,7 +44,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "crypto"
-version = "2.9.0"
+version = "2.9.2"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
@@ -54,7 +54,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "data.jsondata"
-version = "1.1.0"
+version = "1.1.3"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
@@ -79,7 +79,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "http"
-version = "2.14.0"
+version = "2.14.6"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "auth"},
@@ -129,7 +129,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "jwt"
-version = "2.15.0"
+version = "2.15.1"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "cache"},
@@ -280,7 +280,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "oauth2"
-version = "2.14.0"
+version = "2.14.1"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "cache"},
@@ -303,7 +303,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "os"
-version = "1.10.0"
+version = "1.10.1"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "io"},
@@ -316,7 +316,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "sql"
-version = "1.17.0"
+version = "1.17.1"
 dependencies = [
 	{org = "ballerina", name = "file"},
 	{org = "ballerina", name = "io"},

changelog.md

@@ -4,6 +4,11 @@ This file contains all the notable changes done to the Ballerina SQL package thr
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+### Changed
+- [Fix the CLOB retrieval issue as out parameter in OracleDB stored procedure calls](https://github.com/ballerina-platform/ballerina-library/issues/8386)
+
 ## [1.17.0] 2025-09-19
 
 ### Added

gradle.properties

@@ -22,9 +22,9 @@ stdlibUrlVersion=2.6.0
 
 # Level 02
 stdlibConstraintVersion=1.7.0
-stdlibCryptoVersion=2.9.0
+stdlibCryptoVersion=2.9.1
 stdlibLogVersion=2.12.0
-stdlibOsVersion=1.10.0
+stdlibOsVersion=1.10.1
 stdlibTaskVersion=2.7.0
 
 # Level 03
@@ -35,12 +35,12 @@ stdlibUuidVersion=1.10.0
 
 # Level 04
 stdlibAuthVersion=2.14.0
-stdlibDataJsonDataVersion=1.1.0
-stdlibJwtVersion=2.15.0
-stdlibOAuth2Version=2.14.0
+stdlibDataJsonDataVersion=1.1.3
+stdlibJwtVersion=2.15.1
+stdlibOAuth2Version=2.14.1
 
 # Level 05
-stdlibHttpVersion=2.14.0
+stdlibHttpVersion=2.14.6
 
 # Level 06
 stdlibTransactionVersion=1.12.0

native/src/main/java/io/ballerina/stdlib/sql/nativeimpl/CallProcessor.java

@@ -310,9 +310,9 @@ private static int getOutParameterType(BObject typedValue,
             case Constants.OutParameterTypes.CLOB:
             case Constants.OutParameterTypes.NCLOB:
                 if (typedValue instanceof BString) {
-                    sqlTypeValue = Types.CLOB;
-                } else {
                     sqlTypeValue = Types.LONGVARCHAR;
+                } else {
+                    sqlTypeValue = Types.CLOB;
                 }
                 break;
             case Constants.OutParameterTypes.DATE:

native/src/test/java/io/ballerina/stdlib/sql/TestUtils.java

@@ -26,6 +26,9 @@
 import io.ballerina.runtime.api.types.PredefinedTypes;
 import io.ballerina.runtime.api.types.RecordType;
 import io.ballerina.runtime.api.types.Type;
+import io.ballerina.runtime.api.types.semtype.BasicTypeBitSet;
+import io.ballerina.runtime.api.types.semtype.Context;
+import io.ballerina.runtime.api.types.semtype.SemType;
 import io.ballerina.runtime.api.values.BArray;
 import io.ballerina.runtime.api.values.BFuture;
 import io.ballerina.runtime.api.values.BInitialValueEntry;
@@ -62,8 +65,8 @@
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.Map;
+import java.util.Optional;
 import java.util.Properties;
 import java.util.Set;
 import java.util.concurrent.Executor;
@@ -111,6 +114,26 @@ public ObjectType getType() {
                 return TypeCreator.createObjectType(name, emptyModule, 0);
             }
 
+            @Override
+            public SemType widenedType(Context cx) {
+                return BObject.super.widenedType(cx);
+            }
+
+            @Override
+            public Optional<SemType> inherentTypeOf(Context cx) {
+                return BObject.super.inherentTypeOf(cx);
+            }
+
+            @Override
+            public BasicTypeBitSet getBasicType() {
+                return BObject.super.getBasicType();
+            }
+
+            @Override
+            public Type getOriginalType() {
+                return BObject.super.getOriginalType();
+            }
+
             @Override
             public Object get(BString bString) {
                 return null;
@@ -893,6 +916,11 @@ public String stringValue(BLink bLink) {
                 return null;
             }
 
+            @Override
+            public String informalStringValue(BLink parent) {
+                return BTypedesc.super.informalStringValue(parent);
+            }
+
             @Override
             public String expressionStringValue(BLink bLink) {
                 return null;
@@ -902,6 +930,21 @@ public String expressionStringValue(BLink bLink) {
             public Type getType() {
                 return null;
             }
+
+            @Override
+            public SemType widenedType(Context cx) {
+                return BTypedesc.super.widenedType(cx);
+            }
+
+            @Override
+            public Optional<SemType> inherentTypeOf(Context cx) {
+                return BTypedesc.super.inherentTypeOf(cx);
+            }
+
+            @Override
+            public BasicTypeBitSet getBasicType() {
+                return BTypedesc.super.getBasicType();
+            }
         };
     }
 
@@ -1138,22 +1181,6 @@ public Type getType() {
                 return type;
             }
 
-            @Override
-            public Iterator<?> getJavaIterator() {
-                BIterator<?> iterator = getIterator();
-                return new Iterator<>() {
-
-                    @Override
-                    public boolean hasNext() {
-                        return false;
-                    }
-
-                    @Override
-                    public Object next() {
-                        return null;
-                    }
-                };
-            }
         };
     }
 
@@ -1323,22 +1350,6 @@ public Type getType() {
                 return TestUtils.getBooleanStructRecord();
             }
 
-            @Override
-            public Iterator<?> getJavaIterator() {
-                BIterator<?> iterator = getIterator();
-                return new Iterator<>() {
-
-                    @Override
-                    public boolean hasNext() {
-                        return false;
-                    }
-
-                    @Override
-                    public Object next() {
-                        return null;
-                    }
-                };
-            }
         };
     }
 }