Merge pull request #196 from milderhc/exceptions

Milder Hernandez · web-flow · commit 86983f720af5 · 2024-08-26T09:52:33.000-07:00
Update to SKException
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/azureaisearch/AzureAISearchVectorStoreCollectionCreateMapping.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/azureaisearch/AzureAISearchVectorStoreCollectionCreateMapping.java
@@ -13,6 +13,8 @@
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDataField;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordKeyField;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordVectorField;
+import com.microsoft.semantickernel.exceptions.SKException;
+
 import java.time.OffsetDateTime;
 import java.util.List;
 import javax.annotation.Nonnull;
@@ -41,7 +43,7 @@ private static VectorSearchAlgorithmMetric getAlgorithmMetric(
             case EUCLIDEAN:
                 return VectorSearchAlgorithmMetric.EUCLIDEAN;
             default:
-                throw new IllegalArgumentException(
+                throw new SKException(
                     "Unsupported distance function: " + vectorField.getDistanceFunction());
         }
     }
@@ -62,7 +64,7 @@ private static VectorSearchAlgorithmConfiguration getAlgorithmConfig(
                     .setParameters(
                         new ExhaustiveKnnParameters().setMetric(getAlgorithmMetric(vectorField)));
             default:
-                throw new IllegalArgumentException(
+                throw new SKException(
                     "Unsupported index kind: " + vectorField.getIndexKind());
         }
     }
@@ -75,7 +77,7 @@ public static SearchField mapKeyField(VectorStoreRecordKeyField keyField) {
 
     public static SearchField mapDataField(VectorStoreRecordDataField dataField) {
         if (dataField.getFieldType() == null) {
-            throw new IllegalArgumentException(
+            throw new SKException(
                 "Field type is required: " + dataField.getEffectiveStorageName());
         }
 
@@ -97,7 +99,7 @@ public static void updateVectorSearchParameters(
         List<VectorSearchProfile> profiles,
         VectorStoreRecordVectorField vectorField) {
         if (vectorField.getDimensions() <= 0) {
-            throw new IllegalArgumentException("Vector field dimensions must be greater than 0");
+            throw new SKException("Vector field dimensions must be greater than 0");
         }
 
         algorithms.add(getAlgorithmConfig(vectorField));
@@ -121,7 +123,7 @@ public static SearchFieldDataType getSearchFieldDataType(Class<?> fieldType) {
         } else if (fieldType == OffsetDateTime.class) {
             return SearchFieldDataType.DATE_TIME_OFFSET;
         } else {
-            throw new IllegalArgumentException("Unsupported field type: " + fieldType.getName());
+            throw new SKException("Unsupported field type: " + fieldType.getName());
         }
     }
 }
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/azureaisearch/AzureAISearchVectorStoreRecordCollectionOptions.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/azureaisearch/AzureAISearchVectorStoreRecordCollectionOptions.java
@@ -5,6 +5,8 @@
 import com.microsoft.semantickernel.data.VectorStoreRecordCollectionOptions;
 import com.microsoft.semantickernel.data.VectorStoreRecordMapper;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDefinition;
+import com.microsoft.semantickernel.exceptions.SKException;
+
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
@@ -128,7 +130,7 @@ public Builder<Record> withRecordDefinition(VectorStoreRecordDefinition recordDe
          */
         public AzureAISearchVectorStoreRecordCollectionOptions<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass must be provided");
+                throw new SKException("recordClass must be provided");
             }
 
             return new AzureAISearchVectorStoreRecordCollectionOptions<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStore.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStore.java
@@ -162,7 +162,7 @@ public JDBCVectorStore build() {
          */
         public Mono<JDBCVectorStore> buildAsync() {
             if (dataSource == null) {
-                throw new IllegalArgumentException("dataSource is required");
+                throw new SKException("dataSource is required");
             }
 
             JDBCVectorStore vectorStore = new JDBCVectorStore(dataSource, options);
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreDefaultQueryProvider.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreDefaultQueryProvider.java
@@ -182,7 +182,7 @@ public void prepareVectorStore() {
      * Checks if the types of the record class fields are supported.
      *
      * @param recordDefinition the record definition
-     * @throws IllegalArgumentException if the types are not supported
+     * @throws SKException if the types are not supported
      */
     @Override
     public void validateSupportedTypes(VectorStoreRecordDefinition recordDefinition) {
@@ -403,13 +403,13 @@ public void deleteRecords(String collectionName, List<String> keys,
      *
      * @param identifier the identifier
      * @return the identifier if it is valid
-     * @throws IllegalArgumentException if the identifier is invalid
+     * @throws SKException if the identifier is invalid
      */
     public static String validateSQLidentifier(String identifier) {
         if (identifier.matches("[a-zA-Z_][a-zA-Z0-9_]*")) {
             return identifier;
         }
-        throw new IllegalArgumentException("Invalid SQL identifier: " + identifier);
+        throw new SKException("Invalid SQL identifier: " + identifier);
     }
 
     /**
@@ -455,7 +455,7 @@ public Builder withPrefixForCollectionTables(String prefixForCollectionTables) {
         @Override
         public JDBCVectorStoreDefaultQueryProvider build() {
             if (dataSource == null) {
-                throw new IllegalArgumentException("DataSource is required");
+                throw new SKException("DataSource is required");
             }
 
             return new JDBCVectorStoreDefaultQueryProvider(dataSource, collectionsTable,
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordCollection.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordCollection.java
@@ -329,13 +329,13 @@ public Builder<Record> withOptions(JDBCVectorStoreRecordCollectionOptions<Record
         @Override
         public JDBCVectorStoreRecordCollection<Record> build() {
             if (dataSource == null) {
-                throw new IllegalArgumentException("dataSource is required");
+                throw new SKException("dataSource is required");
             }
             if (collectionName == null) {
-                throw new IllegalArgumentException("collectionName is required");
+                throw new SKException("collectionName is required");
             }
             if (options == null) {
-                throw new IllegalArgumentException("options is required");
+                throw new SKException("options is required");
             }
 
             return new JDBCVectorStoreRecordCollection<>(dataSource, collectionName, options);
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordCollectionOptions.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordCollectionOptions.java
@@ -4,6 +4,7 @@
 import com.microsoft.semantickernel.data.VectorStoreRecordCollectionOptions;
 import com.microsoft.semantickernel.data.VectorStoreRecordMapper;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDefinition;
+import com.microsoft.semantickernel.exceptions.SKException;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 import java.sql.ResultSet;
@@ -180,7 +181,7 @@ public Builder<Record> withPrefixForCollectionTables(String prefixForCollectionT
          */
         public JDBCVectorStoreRecordCollectionOptions<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass is required");
+                throw new SKException("recordClass is required");
             }
 
             return new JDBCVectorStoreRecordCollectionOptions<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordMapper.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/jdbc/JDBCVectorStoreRecordMapper.java
@@ -97,10 +97,10 @@ public Builder<Record> withObjectMapper(ObjectMapper objectMapper) {
          */
         public JDBCVectorStoreRecordMapper<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass is required");
+                throw new SKException("recordClass is required");
             }
             if (vectorStoreRecordDefinition == null) {
-                throw new IllegalArgumentException("vectorStoreRecordDefinition is required");
+                throw new SKException("vectorStoreRecordDefinition is required");
             }
 
             return new JDBCVectorStoreRecordMapper<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/postgres/PostgreSQLVectorStoreRecordMapper.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/postgres/PostgreSQLVectorStoreRecordMapper.java
@@ -14,9 +14,6 @@
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.postgresql.util.PGobject;
 
-import java.lang.reflect.Constructor;
-import java.lang.reflect.Field;
-import java.lang.reflect.InvocationTargetException;
 import java.sql.ResultSet;
 import java.sql.ResultSetMetaData;
 import java.sql.SQLException;
@@ -94,10 +91,10 @@ public Builder<Record> withObjectMapper(ObjectMapper objectMapper) {
          */
         public PostgreSQLVectorStoreRecordMapper<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass is required");
+                throw new SKException("recordClass is required");
             }
             if (vectorStoreRecordDefinition == null) {
-                throw new IllegalArgumentException("vectorStoreRecordDefinition is required");
+                throw new SKException("vectorStoreRecordDefinition is required");
             }
 
             return new PostgreSQLVectorStoreRecordMapper<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisHashSetVectorStoreRecordCollectionOptions.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisHashSetVectorStoreRecordCollectionOptions.java
@@ -4,6 +4,7 @@
 import com.microsoft.semantickernel.data.VectorStoreRecordCollectionOptions;
 import com.microsoft.semantickernel.data.VectorStoreRecordMapper;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDefinition;
+import com.microsoft.semantickernel.exceptions.SKException;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -155,7 +156,7 @@ public Builder<Record> withPrefixCollectionName(boolean prefixCollectionName) {
          */
         public RedisHashSetVectorStoreRecordCollectionOptions<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass must be provided");
+                throw new SKException("recordClass must be provided");
             }
 
             return new RedisHashSetVectorStoreRecordCollectionOptions<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisHashSetVectorStoreRecordMapper.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisHashSetVectorStoreRecordMapper.java
@@ -84,10 +84,10 @@ public Builder<Record> withVectorStoreRecordDefinition(
         @Override
         public RedisHashSetVectorStoreRecordMapper<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass is required");
+                throw new SKException("recordClass is required");
             }
             if (recordDefinition == null) {
-                throw new IllegalArgumentException("vectorStoreRecordDefinition is required");
+                throw new SKException("vectorStoreRecordDefinition is required");
             }
 
             ObjectMapper mapper = new ObjectMapper();
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisJsonVectorStoreRecordCollectionOptions.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisJsonVectorStoreRecordCollectionOptions.java
@@ -4,6 +4,7 @@
 import com.microsoft.semantickernel.data.VectorStoreRecordCollectionOptions;
 import com.microsoft.semantickernel.data.VectorStoreRecordMapper;
 import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDefinition;
+import com.microsoft.semantickernel.exceptions.SKException;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -154,7 +155,7 @@ public Builder<Record> withPrefixCollectionName(boolean prefixCollectionName) {
          */
         public RedisJsonVectorStoreRecordCollectionOptions<Record> build() {
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass must be provided");
+                throw new SKException("recordClass must be provided");
             }
 
             return new RedisJsonVectorStoreRecordCollectionOptions<>(
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisJsonVectorStoreRecordMapper.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisJsonVectorStoreRecordMapper.java
@@ -87,10 +87,10 @@ public Builder<Record> withObjectMapper(ObjectMapper objectMapper) {
         @Override
         public RedisJsonVectorStoreRecordMapper<Record> build() {
             if (keyFieldName == null) {
-                throw new IllegalArgumentException("keyFieldName is required");
+                throw new SKException("keyFieldName is required");
             }
             if (recordClass == null) {
-                throw new IllegalArgumentException("recordClass is required");
+                throw new SKException("recordClass is required");
             }
 
             return new RedisJsonVectorStoreRecordMapper<>(record -> {
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStore.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStore.java
@@ -122,11 +122,11 @@ public Builder withOptions(RedisVectorStoreOptions options) {
         @Override
         public RedisVectorStore build() {
             if (client == null) {
-                throw new IllegalArgumentException("client is required");
+                throw new SKException("client is required");
             }
 
             if (options == null) {
-                throw new IllegalArgumentException("options is required");
+                throw new SKException("options is required");
             }
 
             return new RedisVectorStore(client, options);
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStoreCollectionCreateMapping.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStoreCollectionCreateMapping.java
@@ -10,6 +10,8 @@
 import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
+
+import com.microsoft.semantickernel.exceptions.SKException;
 import redis.clients.jedis.search.Schema;
 
 public class RedisVectorStoreCollectionCreateMapping {
@@ -42,7 +44,7 @@ private static String getAlgorithmMetric(
             case EUCLIDEAN:
                 return RedisVectorDistanceMetric.EUCLIDEAN;
             default:
-                throw new IllegalArgumentException(
+                throw new SKException(
                     "Unsupported distance function: " + vectorField.getDistanceFunction());
         }
     }
@@ -59,7 +61,7 @@ private static Schema.VectorField.VectorAlgo getAlgorithmConfig(
             case FLAT:
                 return Schema.VectorField.VectorAlgo.FLAT;
             default:
-                throw new IllegalArgumentException(
+                throw new SKException(
                     "Unsupported index kind: " + vectorField.getIndexKind());
         }
     }
@@ -81,7 +83,7 @@ public static Schema mapToSchema(List<VectorStoreRecordField> fields) {
                 VectorStoreRecordDataField dataField = (VectorStoreRecordDataField) field;
 
                 if (dataField.getFieldType() == null) {
-                    throw new IllegalArgumentException(
+                    throw new SKException(
                         "Field type is required for filterable fields: "
                             + dataField.getEffectiveStorageName());
                 }
@@ -91,7 +93,7 @@ public static Schema mapToSchema(List<VectorStoreRecordField> fields) {
                 } else if (supportedFilterableNumericTypes.contains(dataField.getFieldType())) {
                     schema.addNumericField(getRedisPath(dataField.getEffectiveStorageName()));
                 } else {
-                    throw new IllegalArgumentException(
+                    throw new SKException(
                         "Unsupported field type for numeric filterable fields: "
                             + dataField.getEffectiveStorageName());
                 }
@@ -102,7 +104,7 @@ public static Schema mapToSchema(List<VectorStoreRecordField> fields) {
                 VectorStoreRecordVectorField vectorField = (VectorStoreRecordVectorField) field;
 
                 if (vectorField.getDimensions() < 1) {
-                    throw new IllegalArgumentException(
+                    throw new SKException(
                         "Dimensions must be greater than 0 for vector fields: "
                             + vectorField.getEffectiveStorageName());
                 }
diff --git a/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStoreOptions.java b/semantickernel-experimental/src/main/java/com/microsoft/semantickernel/connectors/data/redis/RedisVectorStoreOptions.java
@@ -1,6 +1,8 @@
 // Copyright (c) Microsoft. All rights reserved.
 package com.microsoft.semantickernel.connectors.data.redis;
 
+import com.microsoft.semantickernel.exceptions.SKException;
+
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
@@ -98,7 +100,7 @@ public Builder withStorageType(RedisStorageType storageType) {
          */
         public RedisVectorStoreOptions build() {
             if (storageType == null) {
-                throw new IllegalArgumentException("storageType is required");
+                throw new SKException("storageType is required");
             }
 
             return new RedisVectorStoreOptions(storageType, vectorStoreRecordCollectionFactory);

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,8 @@`
`13`	`13`	`import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordDataField;`
`14`	`14`	`import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordKeyField;`
`15`	`15`	`import com.microsoft.semantickernel.data.recorddefinition.VectorStoreRecordVectorField;`
	`16`	`+import com.microsoft.semantickernel.exceptions.SKException;`
	`17`	`+`
`16`	`18`	`import java.time.OffsetDateTime;`
`17`	`19`	`import java.util.List;`
`18`	`20`	`import javax.annotation.Nonnull;`
`@@ -41,7 +43,7 @@ private static VectorSearchAlgorithmMetric getAlgorithmMetric(`
`41`	`43`	`case EUCLIDEAN:`
`42`	`44`	`return VectorSearchAlgorithmMetric.EUCLIDEAN;`
`43`	`45`	`default:`
`44`		`- throw new IllegalArgumentException(`
	`46`	`+ throw new SKException(`
`45`	`47`	`"Unsupported distance function: " + vectorField.getDistanceFunction());`
`46`	`48`	`}`
`47`	`49`	`}`
`@@ -62,7 +64,7 @@ private static VectorSearchAlgorithmConfiguration getAlgorithmConfig(`
`62`	`64`	`.setParameters(`
`63`	`65`	`new ExhaustiveKnnParameters().setMetric(getAlgorithmMetric(vectorField)));`
`64`	`66`	`default:`
`65`		`- throw new IllegalArgumentException(`
	`67`	`+ throw new SKException(`
`66`	`68`	`"Unsupported index kind: " + vectorField.getIndexKind());`
`67`	`69`	`}`
`68`	`70`	`}`
`@@ -75,7 +77,7 @@ public static SearchField mapKeyField(VectorStoreRecordKeyField keyField) {`
`75`	`77`
`76`	`78`	`public static SearchField mapDataField(VectorStoreRecordDataField dataField) {`
`77`	`79`	`if (dataField.getFieldType() == null) {`
`78`		`- throw new IllegalArgumentException(`
	`80`	`+ throw new SKException(`
`79`	`81`	`"Field type is required: " + dataField.getEffectiveStorageName());`
`80`	`82`	`}`
`81`	`83`
`@@ -97,7 +99,7 @@ public static void updateVectorSearchParameters(`
`97`	`99`	`List<VectorSearchProfile> profiles,`
`98`	`100`	`VectorStoreRecordVectorField vectorField) {`
`99`	`101`	`if (vectorField.getDimensions() <= 0) {`
`100`		`- throw new IllegalArgumentException("Vector field dimensions must be greater than 0");`
	`102`	`+ throw new SKException("Vector field dimensions must be greater than 0");`
`101`	`103`	`}`
`102`	`104`
`103`	`105`	`algorithms.add(getAlgorithmConfig(vectorField));`
`@@ -121,7 +123,7 @@ public static SearchFieldDataType getSearchFieldDataType(Class<?> fieldType) {`
`121`	`123`	`} else if (fieldType == OffsetDateTime.class) {`
`122`	`124`	`return SearchFieldDataType.DATE_TIME_OFFSET;`
`123`	`125`	`} else {`
`124`		`- throw new IllegalArgumentException("Unsupported field type: " + fieldType.getName());`
	`126`	`+ throw new SKException("Unsupported field type: " + fieldType.getName());`
`125`	`127`	`}`
`126`	`128`	`}`
`127`	`129`	`}`
Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ public JDBCVectorStore build() {`
`162`	`162`	`*/`
`163`	`163`	`public Mono<JDBCVectorStore> buildAsync() {`
`164`	`164`	`if (dataSource == null) {`
`165`		`- throw new IllegalArgumentException("dataSource is required");`
	`165`	`+ throw new SKException("dataSource is required");`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`JDBCVectorStore vectorStore = new JDBCVectorStore(dataSource, options);`
Original file line number	Diff line number	Diff line change
`@@ -329,13 +329,13 @@ public Builder<Record> withOptions(JDBCVectorStoreRecordCollectionOptions<Record`
`329`	`329`	`@Override`
`330`	`330`	`public JDBCVectorStoreRecordCollection<Record> build() {`
`331`	`331`	`if (dataSource == null) {`
`332`		`- throw new IllegalArgumentException("dataSource is required");`
	`332`	`+ throw new SKException("dataSource is required");`
`333`	`333`	`}`
`334`	`334`	`if (collectionName == null) {`
`335`		`- throw new IllegalArgumentException("collectionName is required");`
	`335`	`+ throw new SKException("collectionName is required");`
`336`	`336`	`}`
`337`	`337`	`if (options == null) {`
`338`		`- throw new IllegalArgumentException("options is required");`
	`338`	`+ throw new SKException("options is required");`
`339`	`339`	`}`
`340`	`340`
`341`	`341`	`return new JDBCVectorStoreRecordCollection<>(dataSource, collectionName, options);`
Original file line number	Diff line number	Diff line change
`@@ -97,10 +97,10 @@ public Builder<Record> withObjectMapper(ObjectMapper objectMapper) {`
`97`	`97`	`*/`
`98`	`98`	`public JDBCVectorStoreRecordMapper<Record> build() {`
`99`	`99`	`if (recordClass == null) {`
`100`		`- throw new IllegalArgumentException("recordClass is required");`
	`100`	`+ throw new SKException("recordClass is required");`
`101`	`101`	`}`
`102`	`102`	`if (vectorStoreRecordDefinition == null) {`
`103`		`- throw new IllegalArgumentException("vectorStoreRecordDefinition is required");`
	`103`	`+ throw new SKException("vectorStoreRecordDefinition is required");`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`return new JDBCVectorStoreRecordMapper<>(`
Original file line number	Diff line number	Diff line change
`@@ -84,10 +84,10 @@ public Builder<Record> withVectorStoreRecordDefinition(`
`84`	`84`	`@Override`
`85`	`85`	`public RedisHashSetVectorStoreRecordMapper<Record> build() {`
`86`	`86`	`if (recordClass == null) {`
`87`		`- throw new IllegalArgumentException("recordClass is required");`
	`87`	`+ throw new SKException("recordClass is required");`
`88`	`88`	`}`
`89`	`89`	`if (recordDefinition == null) {`
`90`		`- throw new IllegalArgumentException("vectorStoreRecordDefinition is required");`
	`90`	`+ throw new SKException("vectorStoreRecordDefinition is required");`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`ObjectMapper mapper = new ObjectMapper();`