diff --git a/changelog.md b/changelog.md index 14207233a..9945fc634 100644 --- a/changelog.md +++ b/changelog.md @@ -1,3 +1,5 @@ +- Fixed error with KNN queries against vectors that are stored in nested fields, e.g. `outer.inner.vec`. +--- - Switched LSH parameter names to more canonical equivalents: `bands -> L`, `rows -> k`, based on the [LSH wikipedia article](https://en.wikipedia.org/wiki/Locality-sensitive_hashing#LSH_algorithm_for_nearest_neighbor_search) and material from Indyk, et. al, e.g. [these slides](http://people.csail.mit.edu/indyk/mmds.pdf). diff --git a/plugin/src/main/scala/com/klibisz/elastiknn/query/KnnQueryBuilder.scala b/plugin/src/main/scala/com/klibisz/elastiknn/query/KnnQueryBuilder.scala index af9f9f3c3..03a83ca06 100644 --- a/plugin/src/main/scala/com/klibisz/elastiknn/query/KnnQueryBuilder.scala +++ b/plugin/src/main/scala/com/klibisz/elastiknn/query/KnnQueryBuilder.scala @@ -134,9 +134,10 @@ final case class KnnQueryBuilder(query: NearestNeighborsQuery) extends AbstractQ } ) } catch { - // Pass along only the cause instead of the ExecutionException. + // Guava cache wraps any exceptions in an ExecutionException. Pass along only the cause. case e: java.util.concurrent.ExecutionException => throw new RuntimeException(s"Failed to retrieve mapping at index [$index] field [${query.field}]", e.getCause) + case e: Throwable => throw e } } diff --git a/version b/version index fc8bc9246..4e4c1314d 100644 --- a/version +++ b/version @@ -1 +1 @@ -0.1.0-PRE23 +0.1.0-PRE24