From cf845d2a4d0051251dd71a1c769d6aab72b4ee19 Mon Sep 17 00:00:00 2001 From: Kathleen DeRusso Date: Thu, 3 Apr 2025 16:31:23 -0400 Subject: [PATCH] Ensure sentence overlap is considered in SentenceBoundaryChunkingSettings equals/hashCode --- .../inference/chunking/SentenceBoundaryChunkingSettings.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/chunking/SentenceBoundaryChunkingSettings.java b/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/chunking/SentenceBoundaryChunkingSettings.java index 9d6f5bb89218f..0033cc9ee2bef 100644 --- a/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/chunking/SentenceBoundaryChunkingSettings.java +++ b/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/chunking/SentenceBoundaryChunkingSettings.java @@ -134,11 +134,11 @@ public boolean equals(Object o) { if (this == o) return true; if (o == null || getClass() != o.getClass()) return false; SentenceBoundaryChunkingSettings that = (SentenceBoundaryChunkingSettings) o; - return Objects.equals(maxChunkSize, that.maxChunkSize); + return Objects.equals(maxChunkSize, that.maxChunkSize) && Objects.equals(sentenceOverlap, that.sentenceOverlap); } @Override public int hashCode() { - return Objects.hash(maxChunkSize); + return Objects.hash(maxChunkSize, sentenceOverlap); } }