rmuir commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1710505009
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/MemorySegmentPostingDecodingUtil.java:
##
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Fo
rmuir commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1710505391
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/MemorySegmentPostingDecodingUtil.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Fou
rmuir commented on PR #13636:
URL: https://github.com/apache/lucene/pull/13636#issuecomment-2276947096
> This class in the Panama codebase is huge and hard to read, so splitting
into byte/float implementations looks like a good cleanup. No public API is
involved.
Cool idea, thanks fo
msokolov closed issue #12627: HnwsGraph creates disconnected components
URL: https://github.com/apache/lucene/issues/12627
--
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 un
msokolov commented on issue #12627:
URL: https://github.com/apache/lucene/issues/12627#issuecomment-2276936647
also backported to 9x
--
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 c
github-actions[bot] commented on PR #13165:
URL: https://github.com/apache/lucene/pull/13165#issuecomment-2276934277
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 contributi
github-actions[bot] commented on PR #13525:
URL: https://github.com/apache/lucene/pull/13525#issuecomment-2276933809
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 contributi
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710374307
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/taxonomy/TaxonomyChildrenOrdinalIterator.java:
##
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software F
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710371930
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/ComparableUtils.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710365017
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/FacetFieldLeafCollector.java:
##
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710360090
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/FacetFieldLeafCollector.java:
##
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710358112
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/recorders/CountFacetRecorder.java:
##
@@ -0,0 +1,189 @@
+/*
+ * Licensed to the Apache Software Foundation (
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710346627
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/abstracts/FacetRecorder.java:
##
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710344889
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/abstracts/FacetRecorder.java:
##
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710343109
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/abstracts/OrdinalIterator.java:
##
@@ -0,0 +1,43 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710338323
##
lucene/facet/src/java/org/apache/lucene/facet/taxonomy/FacetLabel.java:
##
@@ -178,6 +178,11 @@ public FacetLabel subpath(final int length) {
}
}
+ /** G
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710320207
##
lucene/facet/src/java/org/apache/lucene/facet/DrillSidewaysQuery.java:
##
@@ -45,58 +45,26 @@ class DrillSidewaysQuery extends Query {
final Query baseQuery;
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710320207
##
lucene/facet/src/java/org/apache/lucene/facet/DrillSidewaysQuery.java:
##
@@ -45,58 +45,26 @@ class DrillSidewaysQuery extends Query {
final Query baseQuery;
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710309544
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeFacetCutter.java:
##
@@ -0,0 +1,354 @@
+/*
+ * Licensed to the Apache Software Found
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710300104
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/LongValueFacetCutter.java:
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundation (
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710295193
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/LongValueFacetCutter.java:
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundation (
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710265556
##
lucene/facet/src/java/org/apache/lucene/facet/DrillSideways.java:
##
@@ -349,45 +344,142 @@ private DrillDownQuery getDrillDownQuery(
public ConcurrentDrillSide
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1710246000
##
lucene/facet/src/java/org/apache/lucene/facet/DrillSideways.java:
##
@@ -349,45 +344,142 @@ private DrillDownQuery getDrillDownQuery(
public ConcurrentDrillSide
msokolov merged PR #13566:
URL: https://github.com/apache/lucene/pull/13566
--
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...@lucene.ap
msokolov commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2276439009
I ran one more test using Cohere (768 dim) vectors. Looks OK to me, I'll
merge this; at least it does no harm.
test case | recall | t| M | beam | index time | connectivity
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709992304
##
lucene/CHANGES.txt:
##
@@ -133,6 +142,8 @@ New Features
DocValuesSkipper abstraction. A new flag is added to FieldType.java that
configures whether
to create
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709975382
##
lucene/core/src/java/org/apache/lucene/search/CollectorOwner.java:
##
@@ -0,0 +1,74 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709965505
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -37,15 +44,23 @@ private static T doPrivileged(Privilege
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709961906
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709952530
##
lucene/sandbox/src/java/module-info.java:
##
@@ -19,13 +19,21 @@
module org.apache.lucene.sandbox {
requires org.apache.lucene.core;
requires org.apache.luce
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709932857
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/DoubleRangeFacetCutter.java:
##
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Foun
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709932857
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/DoubleRangeFacetCutter.java:
##
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Foun
uschindler commented on issue #13638:
URL: https://github.com/apache/lucene/issues/13638#issuecomment-2276279148
Looks like this is caused by #13484
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
uschindler opened a new issue, #13638:
URL: https://github.com/apache/lucene/issues/13638
### Description
Hi @dweiss,
since your last cleanup, when trying to run `gradlew eclipse` it complains
because the task does not exist. Looks ike the reason is that the root project
no longer
benwtrent commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2276247855
> or maybe you're suggesting we change the way we search, not specifically
something about this patch? That does kind of make sense to me - I'm not sure
why the HNSW paper settled on a
benwtrent commented on issue #13281:
URL: https://github.com/apache/lucene/issues/13281#issuecomment-2276245454
agreed, I think we will miss the window
--
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 g
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709835247
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeNode.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709899394
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709866139
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
jpountz commented on issue #13281:
URL: https://github.com/apache/lucene/issues/13281#issuecomment-2276232929
@benwtrent Shall we close this issue as "won't fix"?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709853131
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/OverlappingLongRangeFacetCutter.java:
##
@@ -0,0 +1,272 @@
+/*
+ * Licensed to the Apache Sof
msokolov commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2276225956
> What do we think of increasing the number of entry points to the lower
level?
or maybe you're suggesting we change the way we search, not specifically
something about this patc
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709880508
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal.ve
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709770635
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeFacetCutter.java:
##
@@ -0,0 +1,413 @@
+/*
+ * Licensed to the Apache Software Found
msokolov commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2276098716
>>What do we think of increasing the number of entry points to the lower
level?
> My first attempt searched for 2 instead of 1, and reconnected both of them
...
Oh wait I
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709749808
##
lucene/core/src/java/org/apache/lucene/codecs/lucene912/ForUtil.java:
##
@@ -630,55 +637,55 @@ private static void decode1(DataInput in, long[] tmp,
long[] longs)
msokolov commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2276082590
> What do we think of increasing the number of entry points to the lower
level?
My first attempt searched for 2 instead of 1, and reconnected both of them,
but doing this led to
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709713833
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeFacetCutter.java:
##
@@ -0,0 +1,413 @@
+/*
+ * Licensed to the Apache Software Found
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709705545
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal.ve
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709691392
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeFacetCutter.java:
##
@@ -0,0 +1,413 @@
+/*
+ * Licensed to the Apache Software Found
stefanvodita commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709557365
##
lucene/facet/src/java/org/apache/lucene/facet/DrillSidewaysQuery.java:
##
@@ -45,58 +45,26 @@ class DrillSidewaysQuery extends Query {
final Query baseQue
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709520659
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
mikemccand commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1698434531
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/LongValueFacetCutter.java:
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundatio
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709403728
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal.ve
benwtrent commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2275704679
> So this patch tries to ensure that every node on each level is reachable
from one of the entry points of that level (which are just the nodes on the
next higher level). This is easy
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709378126
##
lucene/core/src/java/org/apache/lucene/internal/vectorization/DefaultPostingDecodingUtil.java:
##
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the Apache Software Foundatio
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709362066
##
lucene/core/src/java/org/apache/lucene/internal/vectorization/DefaultPostingDecodingUtil.java:
##
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the Apache Software Foundatio
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709359720
##
lucene/core/src/java/org/apache/lucene/codecs/lucene912/PostingIndexInput.java:
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709358369
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
msokolov commented on PR #13566:
URL: https://github.com/apache/lucene/pull/13566#issuecomment-2275670617
I posted an updated patch that uses an approach that is more aligned with
how we search. The idea is we don't need to care about strongly-connected
graphs; we just want to make sure the
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709340084
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal.ve
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709339018
##
lucene/core/src/java/org/apache/lucene/codecs/lucene912/PostingIndexInput.java:
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709336210
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -79,4 +113,15 @@ public VectorUtilSupport getVectorUtilSuppo
benwtrent commented on issue #13626:
URL: https://github.com/apache/lucene/issues/13626#issuecomment-2275622759
@david-sitsky I haven't been able to dig further into this yet. Could you
write a test that replicates the failure? I have yet to be able to replicate
the failure.
I *THIN
uschindler commented on PR #13636:
URL: https://github.com/apache/lucene/pull/13636#issuecomment-2275507708
> To help with @rmuir's fast integer vectors issues: Once this is merged, I
plan to split the VectorUtilSupport into ByteVectorUtilSupport and
FloatVectorUtilSupport and use 2 getters
uschindler commented on PR #13636:
URL: https://github.com/apache/lucene/pull/13636#issuecomment-2275500767
As discussed before: I'd move all static constants to a package-private
class, so it is initialized a bit delayed and not in class loading of
VectorizationProvider.
To help wit
uschindler commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1709095157
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -16,20 +16,56 @@
*/
package org.apache.lucene.internal
epotyom commented on code in PR #13568:
URL: https://github.com/apache/lucene/pull/13568#discussion_r1709023544
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/facet/cutters/ranges/LongRangeFacetCutter.java:
##
@@ -0,0 +1,413 @@
+/*
+ * Licensed to the Apache Software Found
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1708960612
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/PanamaVectorizationProvider.java:
##
@@ -79,4 +102,18 @@ public VectorUtilSupport getVectorUtilSuppo
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1708959969
##
lucene/core/src/java/org/apache/lucene/internal/vectorization/DefaultPostingDecodingUtil.java:
##
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the Apache Software Foundatio
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1708943154
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/MemorySegmentPostingDecodingUtil.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software F
jpountz commented on code in PR #13636:
URL: https://github.com/apache/lucene/pull/13636#discussion_r1708936620
##
lucene/core/src/java21/org/apache/lucene/internal/vectorization/MemorySegmentPostingDecodingUtil.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software F
72 matches
Mail list logo