alessandrobenedetti commented on code in PR #910:
URL: https://github.com/apache/solr/pull/910#discussion_r913065377
##########
solr/core/src/java/org/apache/solr/schema/DenseVectorField.java:
##########
@@ -42,40 +40,33 @@
* Provides a field type to support Lucene's {@link
org.apache.lucene.document.KnnVectorField}. See
* {@link org.apache.lucene.search.KnnVectorQuery} for more details. It
supports a fixed cardinality
* dimension for the vector and a fixed similarity function. The default
similarity is
- * EUCLIDEAN_HNSW (L2). The default index codec format is specified in the
Lucene Codec constructor.
- * For Lucene 9.1 e.g. See {@link
org.apache.lucene.codecs.lucene91.Lucene91Codec} Currently {@link
- * org.apache.lucene.backward_codecs.lucene90.Lucene90HnswVectorsFormat} and
{@link
- * org.apache.lucene.codecs.lucene91.Lucene91HnswVectorsFormat} are supported
for advanced
- * hyper-parameter customisation. See {@link
org.apache.lucene.util.hnsw.HnswGraph} for more details
- * about the implementation. <br>
+ * EUCLIDEAN_HNSW (L2). The default algorithm is HNSW. For Lucene 9.1 e.g. See
{@link
+ * org.apache.lucene.util.hnsw.HnswGraph} for more details about the
implementation. <br>
* Only {@code Indexed} and {@code Stored} attributes are supported.
*/
public class DenseVectorField extends FloatPointField {
+ public static final String HNSW_ALGORITHM = "hnsw";
static final String KNN_VECTOR_DIMENSION = "vectorDimension";
static final String KNN_SIMILARITY_FUNCTION = "similarityFunction";
- static final String CODEC_FORMAT = "codecFormat";
+ static final String ALGORITHM = "knnAlgorithm";
static final String HNSW_MAX_CONNECTIONS = "hnswMaxConnections";
static final String HNSW_BEAM_WIDTH = "hnswBeamWidth";
private int dimension;
private VectorSimilarityFunction similarityFunction;
private VectorSimilarityFunction DEFAULT_SIMILARITY =
VectorSimilarityFunction.EUCLIDEAN;
- private String codecFormat;
+ private String algorithm;
Review Comment:
let's be consistent and use KNN_ALGORITHM and knnAlgorithm everywhere?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
