uschindler commented on code in PR #872: URL: https://github.com/apache/solr/pull/872#discussion_r881383710
########## solr/solr-ref-guide/modules/query-guide/pages/dense-vector-search.adoc: ########## @@ -109,20 +109,20 @@ and the hyper-parameter of the HNSW algorithm make sure you set this configurati Here's how `DenseVectorField` can be configured with the advanced codec hyper-parameters: [source,xml] -<fieldType name="knn_vector" class="solr.DenseVectorField" vectorDimension="4" similarityFunction="cosine" codecFormat="Lucene90HnswVectorsFormat" hnswMaxConnections="10" hnswBeamWidth="40"/> +<fieldType name="knn_vector" class="solr.DenseVectorField" vectorDimension="4" similarityFunction="cosine" codecFormat="Lucene91HnswVectorsFormat" hnswMaxConnections="10" hnswBeamWidth="40"/> <field name="vector" type="knn_vector" indexed="true" stored="true"/> `codecFormat`:: + [%autowidth,frame=none] |=== -|Optional |Default: `Lucene90HnswVectorsFormat` +|Optional |Default: `Lucene91HnswVectorsFormat` |=== + (advanced) Specifies the knn codec implementation to use + -Accepted values: `Lucene90HnswVectorsFormat`. +Accepted values: `Lucene91HnswVectorsFormat`. Review Comment: Thanks. I did also not understand the backwards compatibility problem. The codec is written to index file format, so when the backwards-codec JAR file is there (shipped with Solr), it can open old index. The Codec definition in schema is only used for new segments in index. So thanks for removing the migration note. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
