vigyasharma opened a new pull request, #963:
URL: https://github.com/apache/lucene/pull/963
In [LUCENE-10583](https://issues.apache.org/jira/browse/LUCENE-10583), our
users ran into a deadlock for IndexWriter threads. This was because the
application had acquired a lock on the MMapDirectory
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1157282173
I'll try to paste excerpts of the code here:
```
public static LongRange fromLongs(
long min, boolean minInclusive, long max, boolean maxInclusive) {
// here comes
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1157279381
> Also another thing I realized is that `RangeFacetSetMatcher` is a bit
awkward now, as it actually is only specific to `long`'s
It's actually not only specific to longs. It uses longs t
zhaih commented on code in PR #961:
URL: https://github.com/apache/lucene/pull/961#discussion_r898701671
##
lucene/core/src/java/org/apache/lucene/search/BooleanWeight.java:
##
@@ -346,47 +346,100 @@ public BulkScorer bulkScorer(LeafReaderContext context)
throws IOException {
LuXugang opened a new pull request, #962:
URL: https://github.com/apache/lucene/pull/962
Backport for https://github.com/apache/lucene/pull/960 and make same change
for Lucene70DocValues
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
[
https://issues.apache.org/jira/browse/LUCENE-10577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554900#comment-17554900
]
Robert Muir commented on LUCENE-10577:
--
I don't understand why we are doing this a
[
https://issues.apache.org/jira/browse/LUCENE-10600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554898#comment-17554898
]
ASF subversion and git services commented on LUCENE-10600:
--
Co
LuXugang merged PR #960:
URL: https://github.com/apache/lucene/pull/960
--
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.apache
[
https://issues.apache.org/jira/browse/LUCENE-10577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554835#comment-17554835
]
Julie Tibshirani commented on LUCENE-10577:
---
Last thought: would it make sens
[
https://issues.apache.org/jira/browse/LUCENE-10577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554832#comment-17554832
]
Julie Tibshirani edited comment on LUCENE-10577 at 6/16/22 12:20 AM:
[
https://issues.apache.org/jira/browse/LUCENE-10577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554832#comment-17554832
]
Julie Tibshirani edited comment on LUCENE-10577 at 6/16/22 12:19 AM:
[
https://issues.apache.org/jira/browse/LUCENE-10577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554832#comment-17554832
]
Julie Tibshirani commented on LUCENE-10577:
---
[~sokolov] this is a really exci
mdmarshmallow commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156990982
I like the `getComparableLongs()` approach better as well.
I also think a demo is a good idea, I can create one later today or tomorrow
if you don't get around to it before then.
jtibshirani commented on code in PR #958:
URL: https://github.com/apache/lucene/pull/958#discussion_r898465271
##
lucene/core/src/test/org/apache/lucene/search/TestKnnVectorQuery.java:
##
@@ -498,7 +498,7 @@ public void testRandom() throws IOException {
/** Tests with rando
jtibshirani commented on code in PR #958:
URL: https://github.com/apache/lucene/pull/958#discussion_r898455193
##
lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraphSearcher.java:
##
@@ -87,10 +87,14 @@ public static NeighborQueue search(
int numVisited = 0;
for
[
https://issues.apache.org/jira/browse/LUCENE-10584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554757#comment-17554757
]
ASF subversion and git services commented on LUCENE-10584:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10584?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Greg Miller resolved LUCENE-10584.
--
Fix Version/s: 9.3
Resolution: Fixed
> SSDV facets should support hierarchical paths i
[
https://issues.apache.org/jira/browse/LUCENE-10600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554764#comment-17554764
]
Michael McCandless commented on LUCENE-10600:
-
OK thanks.
I guess because
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156833064
@gsmiller @mdmarshmallow in the last commit I refactored `FacetSet` to be an
abstract class with a concrete `LongFacetSet` impl. I struggled back-and-forth
between introducing a `public final
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156829025
> I think we should also have some subpackages (maybe a `decoders`,
`matchers`, and `sets` (for `FacetSets`))?
Take a look at the last commit: not sure there's a reason for `decoders` at
shahrs87 commented on PR #907:
URL: https://github.com/apache/lucene/pull/907#issuecomment-1156823060
@jpountz Made one more attempt to fix test failures. Can you please take a
look again ? Thank you fir being patient with me.
--
This is an automated message from the Apache Git Service.
[
https://issues.apache.org/jira/browse/LUCENE-10584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554758#comment-17554758
]
Greg Miller commented on LUCENE-10584:
--
Fixed and backported. Resolving.
> SSDV f
mdmarshmallow commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156814974
Yeah I think having their own FSD would make it clear. I think we should
also have some subpackages (maybe a `decoders`, `matchers`, and `sets` (for
`FacetSets`))?
--
This is an aut
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156805329
> Maybe one thing would be that we want to make it mandatory for a
`FacetSet` to implement its own `decode()` function?
I've been there :). The reason I went w/ the static method and fun
mdmarshmallow commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156800419
I took a look at the API and seeing if we could do something there with
generics, but that overcomplicates the API haha. It looks good to me! Maybe one
thing would be that we want to m
[
https://issues.apache.org/jira/browse/LUCENE-10584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554746#comment-17554746
]
ASF subversion and git services commented on LUCENE-10584:
--
Co
gsmiller merged PR #929:
URL: https://github.com/apache/lucene/pull/929
--
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.apache
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156786518
> I wonder if we shouldn't make `FacetSet` abstract and create a
`LongFacetSet` for the long-specific impl
I've contemplated on it when I wrote it. I like the idea of an explicit
`LongF
shaie commented on code in PR #841:
URL: https://github.com/apache/lucene/pull/841#discussion_r898280617
##
lucene/facet/src/java/org/apache/lucene/facet/facetset/FacetSetsField.java:
##
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
shaie commented on code in PR #841:
URL: https://github.com/apache/lucene/pull/841#discussion_r898279453
##
lucene/facet/src/java/org/apache/lucene/facet/facetset/FloatFacetSet.java:
##
@@ -0,0 +1,56 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+
gsmiller commented on PR #929:
URL: https://github.com/apache/lucene/pull/929#issuecomment-1156775532
Thanks @mdmarshmallow!
--
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.
tangdh created LUCENE-10619:
---
Summary: Optimize the writeBytes in TermsHashPerField
Key: LUCENE-10619
URL: https://issues.apache.org/jira/browse/LUCENE-10619
Project: Lucene - Core
Issue Type: Impr
gsmiller commented on PR #954:
URL: https://github.com/apache/lucene/pull/954#issuecomment-1156721533
Sure, no problem @jpountz. Thanks for the suggestion. Any concerns with
merging this PR as it is?
--
This is an automated message from the Apache Git Service.
To respond to the message, p
gsmiller commented on code in PR #841:
URL: https://github.com/apache/lucene/pull/841#discussion_r898212017
##
lucene/facet/src/java/org/apache/lucene/facet/facetset/FloatFacetSet.java:
##
@@ -0,0 +1,56 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or mor
jpountz opened a new pull request, #961:
URL: https://github.com/apache/lucene/pull/961
As suggested by @zhaih on #950, we could support more cases in
`BooleanWeight#count`. This PR adds support for these cases specifically:
- Pure disjunctions where only one clause has a non-zero coun
gsmiller commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156708360
@shaie I like this API approach to supporting other data types. Thanks! I
wonder if we shouldn't make `FacetSet` abstract and create a `LongFacetSet` for
the long-specific impl (and `Double
[
https://issues.apache.org/jira/browse/LUCENE-10603?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Lu Xugang reassigned LUCENE-10603:
--
Assignee: Lu Xugang
> Improve iteration of ords for SortedSetDocValues
> ---
[
https://issues.apache.org/jira/browse/LUCENE-10618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554645#comment-17554645
]
Adrien Grand commented on LUCENE-10618:
---
sure, feel free to give it a try and pin
[
https://issues.apache.org/jira/browse/LUCENE-10618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554640#comment-17554640
]
fang hou commented on LUCENE-10618:
---
hi [~jpountz] i'm new to lucene, do you mind if
LuXugang opened a new pull request, #960:
URL: https://github.com/apache/lucene/pull/960
Detailed discussion see:
https://issues.apache.org/jira/projects/LUCENE/issues/LUCENE-10600
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
[
https://issues.apache.org/jira/browse/LUCENE-10600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554626#comment-17554626
]
Lu Xugang commented on LUCENE-10600:
{quote}SortedSetDocValues#docValueCount should
[
https://issues.apache.org/jira/browse/LUCENE-10600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554625#comment-17554625
]
Adrien Grand commented on LUCENE-10600:
---
[~mikemccand] I don't think so, we still
mayya-sharipova commented on PR #947:
URL: https://github.com/apache/lucene/pull/947#issuecomment-1156532534
@msokolov Thanks for your work, this is a very exciting feature. Really
looking forward to it. Extra benefit – less disk space used for vector values.
Overall, this PR looks v
mikemccand commented on PR #738:
URL: https://github.com/apache/lucene/pull/738#issuecomment-1156523966
> On a related note, since writeBytes() is the only API that doesn't pause,
it may be useful to add this note in comments or docstring somewhere, perhaps
with a reference to the jira comm
[
https://issues.apache.org/jira/browse/LUCENE-10600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554620#comment-17554620
]
Michael McCandless commented on LUCENE-10600:
-
Can this be resolved as {{No
mikemccand commented on code in PR #922:
URL: https://github.com/apache/lucene/pull/922#discussion_r898026016
##
lucene/CHANGES.txt:
##
@@ -67,6 +67,8 @@ Other
* LUCENE-10493: Factor out Viterbi algorithm in Kuromoji and Nori to
analysis-common. (Tomoko Uchida)
+* Remove r
mikemccand commented on PR #922:
URL: https://github.com/apache/lucene/pull/922#issuecomment-1156516445
Yeah I think these attributes of `FacetField.TYPE` are not used, but it is
trappy looking. This `TYPE` is really just a marker/singleton so
`FacetsConfig.build` can know which fields in
[
https://issues.apache.org/jira/browse/LUCENE-10557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael McCandless updated LUCENE-10557:
Summary: Migrate to GitHub issue from Jira (was: Migrate to GitHub issue
from Ji
[
https://issues.apache.org/jira/browse/LUCENE-10557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554603#comment-17554603
]
Michael McCandless commented on LUCENE-10557:
-
Thanks [~tomoko] for driving
mikemccand commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156474464
> Can the returned order of documents from the (parallel) searcher be
different? For score-sorted cases it may not be a problem but whenever there
are ties, and the assertions somehow dep
mayya-sharipova commented on code in PR #947:
URL: https://github.com/apache/lucene/pull/947#discussion_r897925136
##
lucene/core/src/java/org/apache/lucene/codecs/lucene93/OffHeapVectorValues.java:
##
@@ -41,11 +42,11 @@ abstract class OffHeapVectorValues extends VectorValues
javanna commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156400157
Yep I see what you mean, I've seen that in the past that some tests may rely
on assumptions that are no longer true if you run the search concurrently. Some
may be test bugs, that may be fix
dweiss commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156324489
Just wondering what the impact is going to be, that's all. Can the returned
order of documents from the (parallel) searcher be different? For score-sorted
cases it may not be a problem but wh
javanna commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156306626
> It's also going to make tests less likely to reproduce though, isn't it?
Do you mean if there are new failures caused by running concurrent searches
more often, these are going to be
jpountz commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156227500
This is my expectation as well, similarly to tests that use
ConcurrentMergeScheduler.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
dweiss commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156207808
It's also going to make tests less likely to reproduce though, isn't it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
javanna commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156179895
> I wonder if this has potential to slow down tests due to repeated
instantiations of threads.
Yes I wonder the same, this needs some runs before we can evaluate how it
affects tests
jpountz commented on PR #959:
URL: https://github.com/apache/lucene/pull/959#issuecomment-1156171875
I like the idea of better exercising concurrent search. I wonder if this has
potential to slow down tests due to repeated instantiations of threads. I'm
inclined to merge and watch how long
[
https://issues.apache.org/jira/browse/LUCENE-10608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554455#comment-17554455
]
ASF subversion and git services commented on LUCENE-10608:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554456#comment-17554456
]
ASF subversion and git services commented on LUCENE-10608:
--
Co
Adrien Grand created LUCENE-10618:
-
Summary: Implement BooleanQuery rewrite rules based for
minimumShouldMatch
Key: LUCENE-10618
URL: https://issues.apache.org/jira/browse/LUCENE-10618
Project: Lucene
shaie commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156131175
> Rather than do this, I think that it makes more sense for the impls to
just have the ability of returning their own values as a `long[]` (something
like `getComparableLongs()` that was in th
[
https://issues.apache.org/jira/browse/LUCENE-10078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1755#comment-1755
]
ASF subversion and git services commented on LUCENE-10078:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554445#comment-17554445
]
ASF subversion and git services commented on LUCENE-10617:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554442#comment-17554442
]
ASF subversion and git services commented on LUCENE-10078:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554443#comment-17554443
]
ASF subversion and git services commented on LUCENE-10617:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554440#comment-17554440
]
Adrien Grand commented on LUCENE-10617:
---
Neither of these seeds reproduce for me
mdmarshmallow commented on PR #841:
URL: https://github.com/apache/lucene/pull/841#issuecomment-1156101105
> To complement that on the aggregation side we will need to pass a reader
which can convert the BytesRef to a long[]. I'm thinking that the
Int/Float/Long/DoubleFacetSet impls will do
[
https://issues.apache.org/jira/browse/LUCENE-10078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554423#comment-17554423
]
ASF subversion and git services commented on LUCENE-10078:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17554422#comment-17554422
]
ASF subversion and git services commented on LUCENE-10078:
--
Co
70 matches
Mail list logo
