Javadod and comments + map UUID to string and other small changes

Author: fmeheust

data/semantickernel-data-oracle/pom.xml

@@ -56,38 +56,10 @@
             <artifactId>jackson-datatype-jsr310</artifactId>
             <version>2.18.0</version>
         </dependency>
-        <dependency>
-            <groupId>com.github.jknack</groupId>
-            <artifactId>handlebars</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>com.google.code.findbugs</groupId>
-            <artifactId>jsr305</artifactId>
-            <scope>provided</scope>
-        </dependency>
-        <dependency>
-            <groupId>com.fasterxml.jackson.dataformat</groupId>
-            <artifactId>jackson-dataformat-yaml</artifactId>
-            <scope>compile</scope>
-        </dependency>
         <dependency>
             <groupId>com.github.spotbugs</groupId>
             <artifactId>spotbugs-annotations</artifactId>
         </dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-text</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.postgresql</groupId>
-            <artifactId>postgresql</artifactId>
-            <version>42.7.4</version>
-        </dependency>
-        <dependency>
-            <groupId>org.xerial</groupId>
-            <artifactId>sqlite-jdbc</artifactId>
-            <version>3.47.0.0</version>
-        </dependency>
         <dependency>
             <groupId>com.oracle.database.jdbc</groupId>
             <artifactId>ojdbc11</artifactId>
@@ -98,6 +70,7 @@
             <artifactId>ojdbc-provider-jackson-oson</artifactId>
             <version>1.0.4</version>
         </dependency>
+        <!-- Tests use TestContainers to create an Oracle Database -->
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>
@@ -108,7 +81,6 @@
             <artifactId>junit-jupiter-api</artifactId>
             <scope>test</scope>
         </dependency>
-        <!-- Tests use TestContainers to create an Oracle Database -->
         <dependency>
             <groupId>org.testcontainers</groupId>
             <artifactId>testcontainers</artifactId>

data/semantickernel-data-oracle/src/main/java/com/microsoft/semantickernel/data/jdbc/oracle/OracleDataTypesMapping.java

@@ -1,22 +1,74 @@
+/*
+ ** Semantic Kernel Oracle connector version 1.0.
+ **
+ ** Copyright (c) 2025 Oracle and/or its affiliates.
+ ** Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/
+ */
 package com.microsoft.semantickernel.data.jdbc.oracle;
 
 /**
- * Defines oracle database type constants for supported field types.
+ * Defines oracle database type constants for supported java types.
  */
 public class OracleDataTypesMapping {
-    public static final String STRING_VARCHAR = "NVARCHAR2(%s)";
+
+    /**
+     * Oracle database type used when strings are mapped to VARCHAR
+     */
+    public static final String STRING_VARCHAR = "VARCHAR2(%s)";
+    /**
+     * Oracle database type used when strings are mapped to CLOB
+     */
     public static final String STRING_CLOB = "CLOB";
+    /**
+     * Oracle database type used to map booleans
+     */
     public static final String BOOLEAN = "BOOLEAN";
+    /**
+     * Oracle database type used to map bytes
+     */
     public static final String BYTE = "NUMBER(3)";
+    /**
+     * Oracle database type used to map byte arrays
+     */
     public static final String BYTE_ARRAY = "RAW(2000)";
+    /**
+     * Oracle database type used to map shorts
+     */
     public static final String SHORT = "NUMBER(5)";
+    /**
+     * Oracle database type used to map ints
+     */
     public static final String INTEGER = "NUMBER(10)";
+    /**
+     * Oracle database type used to map longs
+     */
     public static final String LONG = "NUMBER(19)";
+    /**
+     * Oracle database type used to map float
+     */
     public static final String FLOAT = "BINARY_FLOAT";
+    /**
+     * Oracle database type used to map double
+     */
     public static final String DOUBLE = "BINARY_DOUBLE";
+    /**
+     * Oracle database type used to map BigDecimal
+     */
     public static final String DECIMAL = "NUMBER";
+    /**
+     * Oracle database type used to map offset date time
+     */
     public static final String OFFSET_DATE_TIME = "TIMESTAMP(7) WITH TIME ZONE";
-    public static final String UUID = "RAW(16)";
+    /**
+     * Oracle database type used to map UUID
+     */
+    public static final String UUID = "VARCHAR2(36)";
+    /**
+     * Oracle database type used to map lists
+     */
     public static final String JSON = "JSON";
+    /**
+     * Oracle database type used to map vectors (the parameter is the dimension of the vector)
+     */
     public static final String VECTOR_FLOAT = "VECTOR(%s, FLOAT32)";
 }