madrob merged pull request #2531:
URL: https://github.com/apache/lucene-solr/pull/2531
--
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:
cpoerschke commented on pull request #202:
URL: https://github.com/apache/lucene/pull/202#issuecomment-877709067
Thanks @msokolov for copying across the comments from the JIRA! I'll convert
the pull request to "draft" to indicate the status of the deprecation process.
--
This is an
cpoerschke commented on pull request #201:
URL: https://github.com/apache/lucene/pull/201#issuecomment-877708925
Thanks @msokolov for copying across the comments from the JIRA! I'll convert
the pull request to "draft" to indicate the status of the deprecation process.
--
This is an
zacharymorn commented on pull request #180:
URL: https://github.com/apache/lucene/pull/180#issuecomment-877676142
> Yes; javadocs will need to warn people. This is _already_ a trap, not a
new one. The method `org.apache.lucene.index.IndexReader#getTermVector` is
tempting but can be bad
[
https://issues.apache.org/jira/browse/LUCENE-9614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17378497#comment-17378497
]
Michael Sokolov edited comment on LUCENE-9614 at 7/10/21, 5:03 PM:
---
[
https://issues.apache.org/jira/browse/LUCENE-9614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17378497#comment-17378497
]
Michael Sokolov commented on LUCENE-9614:
-
Doing nn vector search during rewrite has one
[
https://issues.apache.org/jira/browse/LUCENE-10016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17378489#comment-17378489
]
Michael Sokolov commented on LUCENE-10016:
--
I posted a PR removing fanout. For the question of
msokolov opened a new pull request #210:
URL: https://github.com/apache/lucene/pull/210
I think we can remove this parameter to simplify the nn search api and make
it more generic. For the HNSW algorithm, searching for top K=M and fanout=N is
basically the same as searching with top K=M+N
mocobeta commented on pull request #207:
URL: https://github.com/apache/lucene/pull/207#issuecomment-877639170
Thanks @msokolov for reviewing. I'll wait for a few more days to let others
give comments on this then merge it to the upstream if there is no another
feedback.
--
This is an
mocobeta commented on a change in pull request #207:
URL: https://github.com/apache/lucene/pull/207#discussion_r667339213
##
File path: lucene/CHANGES.txt
##
@@ -7,7 +7,7 @@ http://s.apache.org/luceneversions
New Features
-* LUCENE-9322: Vector-valued fields, Lucene90
mocobeta commented on a change in pull request #207:
URL: https://github.com/apache/lucene/pull/207#discussion_r667338807
##
File path: lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
##
@@ -2284,28 +2284,28 @@ static void checkImpacts(Impacts impacts, int
mocobeta commented on a change in pull request #207:
URL: https://github.com/apache/lucene/pull/207#discussion_r667338802
##
File path: lucene/core/src/java/org/apache/lucene/index/CheckIndex.java
##
@@ -2284,28 +2284,28 @@ static void checkImpacts(Impacts impacts, int
12 matches
Mail list logo