easyice commented on PR #12782:
URL: https://github.com/apache/lucene/pull/12782#issuecomment-1958968291
> @easyice Hi, I have doubt that the encoding data result using group-varint
encoding is different from the old way, so is this way compatible with the old
index format data?
believezzd commented on PR #174:
URL: https://github.com/apache/lucene/pull/174#issuecomment-1959010131
@jpountz
## Description
* There are two place code that i can't understand in this commit, could you
please give me help.
## First one
* To
benwtrent commented on PR #13124:
URL: https://github.com/apache/lucene/pull/13124#issuecomment-1959856815
> For instance the merge scheduler could return a custom Executor that
dynamically decides to run a new task in the current thread or to fork to a
separate thread depending on how
jpountz commented on PR #13124:
URL: https://github.com/apache/lucene/pull/13124#issuecomment-1959736838
> So, we would have to update the MergeScheduler to have some methods to
return the executor for us to use and pass to MergeState (which is only created
via the SegmentMerger object).
ChrisHegarty commented on PR #13119:
URL: https://github.com/apache/lucene/pull/13119#issuecomment-1959890082
Thanks for the reviews. All comments have been addressed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
ChrisHegarty merged PR #13119:
URL: https://github.com/apache/lucene/pull/13119
--
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:
azagniotov commented on PR #12517:
URL: https://github.com/apache/lucene/pull/12517#issuecomment-1960013536
@uschindler yes: I have rebased from the latest `main` branch and ran the
`./gradlew clean regenerate`. The following is the (partial) output:
```
...
...
> Task
azagniotov commented on PR #12517:
URL: https://github.com/apache/lucene/pull/12517#issuecomment-1960013241
@uschindler yes: I have rebased from the latest `main` branch and ran the
`./gradlew clean regenerate`. The following is the (partial) output:
```
...
...
> Task
benwtrent merged PR #76:
URL: https://github.com/apache/lucene-site/pull/76
--
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:
benwtrent commented on code in PR #13124:
URL: https://github.com/apache/lucene/pull/13124#discussion_r1499916055
##
lucene/core/src/java/org/apache/lucene/codecs/lucene99/Lucene99HnswVectorsFormat.java:
##
@@ -152,7 +153,25 @@ public Lucene99HnswVectorsFormat() {
* @param
rmuir commented on PR #12753:
URL: https://github.com/apache/lucene/pull/12753#issuecomment-1960300191
@ChrisHegarty @uschindler @mikemccand i think we still need to address
these python scripts, sorry I haven't gotten to it! been busy unfortunately:
> still need to update
rmuir commented on PR #12753:
URL: https://github.com/apache/lucene/pull/12753#issuecomment-1960301699
I mean, they could be a followup ticket. But it is nice to keep the `main`
branch releasable. Until we fix those python scripts, we can't release because
they want to use java 17
--
mocobeta commented on PR #12517:
URL: https://github.com/apache/lucene/pull/12517#issuecomment-1960500994
@azagniotov Sorry, I've not been available for a while. Let me take a look;
I will try to find time next week...
--
This is an automated message from the Apache Git Service.
To
github-actions[bot] commented on PR #12980:
URL: https://github.com/apache/lucene/pull/12980#issuecomment-1960551634
This PR has not had activity in the past 2 weeks, labeling it as stale. If
the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you
for your
github-actions[bot] commented on PR #12872:
URL: https://github.com/apache/lucene/pull/12872#issuecomment-1960551768
This PR has not had activity in the past 2 weeks, labeling it as stale. If
the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you
for your
github-actions[bot] commented on PR #12985:
URL: https://github.com/apache/lucene/pull/12985#issuecomment-1960551613
This PR has not had activity in the past 2 weeks, labeling it as stale. If
the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you
for your
azagniotov commented on PR #12517:
URL: https://github.com/apache/lucene/pull/12517#issuecomment-1960506371
Thank you, @mocobeta
--
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
benwtrent opened a new pull request, #76:
URL: https://github.com/apache/lucene-site/pull/76
(no comment)
--
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,
ChrisHegarty commented on PR #12753:
URL: https://github.com/apache/lucene/pull/12753#issuecomment-1960191939
This is ready to merge, right? @rmuir @uschindler @mikemccand
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
kaivalnp commented on PR #12820:
URL: https://github.com/apache/lucene/pull/12820#issuecomment-1959568841
Thanks for checking @benwtrent!
We primarily improve cases of using a high topK + a selective filter (good
rate of fallback, large number of duplicate computations). I notice
wjp719 commented on PR #12782:
URL: https://github.com/apache/lucene/pull/12782#issuecomment-1959222579
> Have you got specific errors? could you give some detailed message? Thanks!
I have no errors,I didn't realize the new format was used, Thanks.
--
This is an automated
benwtrent commented on code in PR #13124:
URL: https://github.com/apache/lucene/pull/13124#discussion_r1499311351
##
lucene/core/src/java/org/apache/lucene/codecs/lucene99/Lucene99HnswVectorsFormat.java:
##
@@ -152,7 +153,25 @@ public Lucene99HnswVectorsFormat() {
* @param
benwtrent commented on code in PR #13119:
URL: https://github.com/apache/lucene/pull/13119#discussion_r1499170841
##
lucene/core/src/java/org/apache/lucene/codecs/lucene99/Lucene99HnswVectorsReader.java:
##
@@ -171,15 +172,25 @@ private void validateFieldEntry(FieldInfo info,
benwtrent commented on PR #13124:
URL: https://github.com/apache/lucene/pull/13124#issuecomment-1959551147
> So I think it is still better to have two separate thread pools for
inter-segment merge and inner segment merge, but I wonder whether we can have a
ThreadPoolManager which
24 matches
Mail list logo