Fix naming issue and leftover logging call

Signed-off-by: Łukasz Jernaś <[email protected]>

Author: deejay1

auto-configurations/models/chat/memory/spring-ai-autoconfigure-model-chat-memory-mongodb/src/main/java/org/springframework/ai/model/chat/memory/mongodb/autoconfigure/MongoDbChatMemoryIndexCreator.java

@@ -32,12 +32,12 @@
  * main index on the conversationId and timestamp fields, and a TTL index on the timestamp
  * field if the TTL is set in properties.
  *
- * @see MongoDbChatMemoryProperties
  * @author Łukasz Jernaś
+ * @see MongoDbChatMemoryProperties
  * @since 1.0.0
  */
 @Component
-@ConditionalOnProperty(value = "spring.ai.chat.memory.mongodb.create-indexes", havingValue = "true")
+@ConditionalOnProperty(value = "spring.ai.chat.memory.mongodb.create-indices", havingValue = "true")
 public class MongoDbChatMemoryIndexCreator {
 
 	private static final Logger logger = LoggerFactory.getLogger(MongoDbChatMemoryIndexCreator.class);
@@ -60,14 +60,12 @@ public void initIndicesAfterStartup() {
 			.ensureIndex(new Index().on("conversationId", Sort.Direction.ASC).on("timestamp", Sort.Direction.DESC));
 
 		createOrUpdateTtlIndex();
-
 	}
 
 	private void createOrUpdateTtlIndex() {
 		if (!this.mongoDbChatMemoryProperties.getTtl().isZero()) {
 			// Check for existing TTL index
 			mongoTemplate.indexOps(Conversation.class).getIndexInfo().forEach(idx -> {
-				logger.error("Index info: {}", idx);
 				if (idx.getExpireAfter().isPresent()
 						&& !idx.getExpireAfter().get().equals(this.mongoDbChatMemoryProperties.getTtl())) {
 					logger.warn("Dropping existing TTL index, because TTL is different");

auto-configurations/models/chat/memory/spring-ai-autoconfigure-model-chat-memory-mongodb/src/main/java/org/springframework/ai/model/chat/memory/mongodb/autoconfigure/MongoDbChatMemoryProperties.java

@@ -33,7 +33,7 @@ public class MongoDbChatMemoryProperties {
 	 * If the indexes should be automatically created on app startup. Note: Changing the
 	 * TTL value will drop the TTL index and recreate it.
 	 */
-	private boolean createIndexes = false;
+	private boolean createIndices = false;
 
 	/**
 	 * The time to live (TTL) for the conversation documents in the database. The default
@@ -49,12 +49,12 @@ public void setTtl(Duration ttl) {
 		this.ttl = ttl;
 	}
 
-	public boolean isCreateIndexes() {
-		return createIndexes;
+	public boolean isCreateIndices() {
+		return createIndices;
 	}
 
-	public void setCreateIndexes(boolean createIndexes) {
-		this.createIndexes = createIndexes;
+	public void setCreateIndices(boolean createIndices) {
+		this.createIndices = createIndices;
 	}
 
 }

auto-configurations/models/chat/memory/spring-ai-autoconfigure-model-chat-memory-mongodb/src/test/java/org/springframework/ai/model/chat/memory/mongodb/autoconfigure/MongoDbChatMemoryPropertiesTests.java

@@ -28,16 +28,17 @@ public class MongoDbChatMemoryPropertiesTests {
 	void defaultValues_set() {
 		var properties = new MongoDbChatMemoryProperties();
 		assertThat(properties.getTtl()).isEqualTo(Duration.ZERO);
-		assertThat(properties.isCreateIndexes()).isFalse();
+		assertThat(properties.isCreateIndices()).isFalse();
 	}
 
 	@Test
 	void overrideValues() {
 		var properties = new MongoDbChatMemoryProperties();
 		properties.setTtl(Duration.ofMinutes(1));
-		properties.setCreateIndexes(true);
+		properties.setCreateIndices(true);
 
 		assertThat(properties.getTtl()).isEqualTo(Duration.ofMinutes(1));
-		assertThat(properties.isCreateIndexes()).isTrue();
+		assertThat(properties.isCreateIndices()).isTrue();
 	}
+
 }