Re: [I] Eclipse - one or more cycles were detected [lucene]

2024-08-23 Thread via GitHub
plutext commented on issue #13676: URL: https://github.com/apache/lucene/issues/13676#issuecomment-2306547017 Yes, I imported as an existing project (Import > General > Existing Projects into Workspace) rather than Gradle > Existing Gradle Project. Here is one set of projects affected

Re: [PR] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
uschindler commented on PR #13570: URL: https://github.com/apache/lucene/pull/13570#issuecomment-2306601362 We may need to change the changes entry to tell that to enduser. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
ChrisHegarty opened a new pull request, #13682: URL: https://github.com/apache/lucene/pull/13682 Backport of * #13570 The backport only supports the JDK 21 version of the code - JDK 19 and 20 specific versions do not aggregate files so therefore remain the same. I modified

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
ChrisHegarty commented on code in PR #13682: URL: https://github.com/apache/lucene/pull/13682#discussion_r1728853946 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -312,6 +361,21 @@ public boolean getPreload() { return preload == ALL_FILES; }

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
uschindler commented on code in PR #13682: URL: https://github.com/apache/lucene/pull/13682#discussion_r1728877786 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -312,6 +361,21 @@ public boolean getPreload() { return preload == ALL_FILES; } +

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
uschindler commented on code in PR #13682: URL: https://github.com/apache/lucene/pull/13682#discussion_r1728883679 ## lucene/core/src/java/org/apache/lucene/store/MappedByteBufferIndexInputProvider.java: ## @@ -62,7 +63,13 @@ public MappedByteBufferIndexInputProvider() { }

Re: [I] Eclipse - one or more cycles were detected [lucene]

2024-08-23 Thread via GitHub
dweiss commented on issue #13676: URL: https://github.com/apache/lucene/issues/13676#issuecomment-2306989813 I just tried and it imports and builds fine for me? ![image](https://github.com/user-attachments/assets/62a498e3-e6da-4c61-b4d1-9f07ae2d4017) -- This is an automated message

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
ChrisHegarty commented on code in PR #13682: URL: https://github.com/apache/lucene/pull/13682#discussion_r1729023896 ## lucene/core/src/java/org/apache/lucene/store/MappedByteBufferIndexInputProvider.java: ## @@ -62,7 +63,13 @@ public MappedByteBufferIndexInputProvider() { }

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
ChrisHegarty merged PR #13682: URL: https://github.com/apache/lucene/pull/13682 -- 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...@lucen

Re: [I] Test TestIndexWriterWithThreads#testIOExceptionDuringWriteSegmentWithThreadsOnlyOnce Failed [lucene]

2024-08-23 Thread via GitHub
aoli-al commented on issue #13552: URL: https://github.com/apache/lucene/issues/13552#issuecomment-2307194412 Gentle ping @jpountz and @atris, I saw your previous discussion about #10112. This failure might be related -- This is an automated message from the Apache Git Service. To respo

Re: [PR] nocommit: demonstrate how a minor change in IndexSearcher can have an inexplicable performance impact [lucene]

2024-08-23 Thread via GitHub
epotyom commented on PR #13657: URL: https://github.com/apache/lucene/pull/13657#issuecomment-2307371884 I've been running some experiments, so far this is what I observe: It is not important in which order we init collection of `Collector`s and call `createWeight`, but it is important that

Re: [I] Find a way to remove IndexSearcher#search(Query query, CollectorOwner collectorOwner) before 10.0 [lucene]

2024-08-23 Thread via GitHub
epotyom commented on issue #13671: URL: https://github.com/apache/lucene/issues/13671#issuecomment-2307498253 The main reason for adding `CollectorOwner` class was to allow different drill-sideways dimensions to use `CollectorManager`s with different Collector `C` and result `T` types. Exce

Re: [PR] [GENE-2434] - Load vault token from file [lucene-solr]

2024-08-23 Thread via GitHub
Unity-Technologies closed pull request #2685: [GENE-2434] - Load vault token from file URL: https://github.com/apache/lucene-solr/pull/2685 -- 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 specif

Re: [PR] [GENE-2434] - Load vault token from file [lucene-solr]

2024-08-23 Thread via GitHub
itygh commented on PR #2685: URL: https://github.com/apache/lucene-solr/pull/2685#issuecomment-2307509471 这是来自QQ邮箱的假期自动回复邮件。您好,我最近正在休假中,无法亲自回复您的邮件。我将在假期结束后,尽快给您回复。 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [9.x] Aggregate files from the same segment into a single Arena [lucene]

2024-08-23 Thread via GitHub
uschindler commented on PR #13682: URL: https://github.com/apache/lucene/pull/13682#issuecomment-2307516673 🙏🥳 -- 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 unsubscrib

[PR] HNSW BP reorder tool [lucene]

2024-08-23 Thread via GitHub
msokolov opened a new pull request, #13683: URL: https://github.com/apache/lucene/pull/13683 This is a working tool for re-ordering an index using BP (binary partitioning) over a vector field. It will renumber an existing HNSW graph over the field without the need to re-index it. There are

Re: [PR] HNSW BP reorder tool [lucene]

2024-08-23 Thread via GitHub
msokolov commented on PR #13683: URL: https://github.com/apache/lucene/pull/13683#issuecomment-2307756410 addresses https://github.com/apache/lucene/issues/13565 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL