mrkm4ntr commented on PR #13066:
URL: https://github.com/apache/lucene/pull/13066#issuecomment-1943217368
@jpountz Could you take a look?
--
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 speci
github-actions[bot] commented on PR #12947:
URL: https://github.com/apache/lucene/pull/12947#issuecomment-1942886749
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 #13060:
URL: https://github.com/apache/lucene/pull/13060#issuecomment-1942886567
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 #13035:
URL: https://github.com/apache/lucene/pull/13035#issuecomment-1942886656
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
sabi0 commented on PR #13100:
URL: https://github.com/apache/lucene/pull/13100#issuecomment-1942263565
Should I do anything about CHANGES.txt here?
Cherry-pick the commit but inject another PR number?
Should the line still go to the 9.10 section?
--
This is an automated message from
sabi0 commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1942257037
PR#13100
--
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 unsubscri
sabi0 opened a new pull request, #13100:
URL: https://github.com/apache/lucene/pull/13100
(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, e-mai
dweiss commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1942211430
Your fork should contain all the branches of the original repository. You
should typically have two remotes - one to your fork (origin), one to lucene's
upstream git. Let's say the upstre
sabi0 commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1942196592
I will happily do that. No problem.
(Just need to figure out how to get the 9x branch into my fork..)
--
This is an automated message from the Apache Git Service.
To respond to the mes
dweiss commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1488370861
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +53,31 @@
public abstract class BackwardsCompat
dweiss commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1488367811
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +53,31 @@
public abstract class BackwardsCompat
dweiss commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1942131273
I've applied this to main but it didn't backport cleanly to branch_9x. Would
you have some time to go through the conflicts and create another PR that
targets branch_9x, please? These are
uschindler commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1488362426
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +53,31 @@
public abstract class BackwardsCo
dweiss merged PR #13092:
URL: https://github.com/apache/lucene/pull/13092
--
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.apac
dweiss merged PR #13098:
URL: https://github.com/apache/lucene/pull/13098
--
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.apac
dweiss commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1488345592
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +52,28 @@
public abstract class BackwardsCompat
s1monw commented on PR #13095:
URL: https://github.com/apache/lucene/pull/13095#issuecomment-1941931700
@uschindler I did separate out the specific indices and added a new file.
that makes maintenance simpler and moves the `-cfs` `-nocfs` duplication to the
code. Also applied you comments.
jpountz commented on issue #13099:
URL: https://github.com/apache/lucene/issues/13099#issuecomment-1941895525
The ability to configure Lucene file formats by adding tuning knobs has come
up a few times in the past and the answer has been to create a custom codec, to
keep the scope of what L
uschindler commented on PR #13098:
URL: https://github.com/apache/lucene/pull/13098#issuecomment-1941783874
Yeah, let's apply this for main branch. branch_9x is Java 11 only and I
think we can keep it as is, it won't survive for too long anymore (I hope).
--
This is an automated message f
sarthakaggarwal97 opened a new issue, #13099:
URL: https://github.com/apache/lucene/issues/13099
### Description
It was observed, for duplicated / similar type of data, the change in block
size from [60K to 8K](https://github.com/apache/lucene/commit/f1fdd24) results
in over 50% incr
dweiss commented on PR #13098:
URL: https://github.com/apache/lucene/pull/13098#issuecomment-1941517375
I could try to play with spaces, comments, etc. to align those two lines
but in the end it's just a hack - looking for cracks in gjf. What I suggested
as an alternative is guaranteed to
uschindler merged PR #13097:
URL: https://github.com/apache/lucene/pull/13097
--
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.
uschindler commented on PR #13097:
URL: https://github.com/apache/lucene/pull/13097#issuecomment-1941501048
I will commit this only to main branch, as the file is only read from there.
Or I am missing something (does it need to be backported for another reason?)
--
This is an automated me
uschindler commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1487826242
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -50,7 +52,7 @@
public abstract class BackwardsComp
uschindler commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1487821423
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +52,28 @@
public abstract class BackwardsCo
uschindler commented on PR #13096:
URL: https://github.com/apache/lucene/pull/13096#issuecomment-1941478421
Thanks!
--
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 unsub
uschindler commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1487811954
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/BackwardsCompatibilityTestBase.java:
##
@@ -48,25 +52,28 @@
public abstract class BackwardsCo
uschindler commented on code in PR #13095:
URL: https://github.com/apache/lucene/pull/13095#discussion_r1487803547
##
dev-tools/scripts/addBackcompatIndexes.py:
##
@@ -114,53 +113,25 @@ def find_version(x):
x = re.sub(strip_dash_suffix_re, '', x) # remove the -suffix if any
sabi0 commented on code in PR #13041:
URL: https://github.com/apache/lucene/pull/13041#discussion_r1487807932
##
lucene/queryparser/src/java/org/apache/lucene/queryparser/flexible/core/nodes/TokenizedPhraseQueryNode.java:
##
@@ -70,10 +70,8 @@ public QueryNode cloneTree() throws
dweiss commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1941440581
Wait until this is fixed, merge with main and then we'll take it from there.
Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
dweiss commented on PR #13095:
URL: https://github.com/apache/lucene/pull/13095#issuecomment-1941375395
I've broken it, I'll fix it in a sec.
--
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 s
dweiss commented on PR #13095:
URL: https://github.com/apache/lucene/pull/13095#issuecomment-1941331344
I pushed a simple patch replacing properties with txt files - no need to
worry about property file syntax and easier to read too, I think.
I like how python script got simpler too.
s1monw merged PR #13067:
URL: https://github.com/apache/lucene/pull/13067
--
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.apac
s1monw commented on PR #13095:
URL: https://github.com/apache/lucene/pull/13095#issuecomment-1941123474
@uschindler @dweiss I moved all version to property files and updated the
python script
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
stefanvodita commented on PR #13045:
URL: https://github.com/apache/lucene/pull/13045#issuecomment-1941119764
Hi @dianjifzm! Can you give more context for what you're trying to do? I
[passed the PR description through Google
Translate](https://translate.google.com/?sl=auto&tl=en&text=%E5%BC
stefanvodita commented on code in PR #13041:
URL: https://github.com/apache/lucene/pull/13041#discussion_r1487566325
##
lucene/queryparser/src/java/org/apache/lucene/queryparser/flexible/core/nodes/TokenizedPhraseQueryNode.java:
##
@@ -70,10 +70,8 @@ public QueryNode cloneTree()
jpountz merged PR #13096:
URL: https://github.com/apache/lucene/pull/13096
--
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.apa
jpountz commented on PR #13096:
URL: https://github.com/apache/lucene/pull/13096#issuecomment-1940963440
(which is what Simon is doing at #13095)
--
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 t
jpountz commented on PR #13096:
URL: https://github.com/apache/lucene/pull/13096#issuecomment-1940960455
@uschindler I'm not sure to understand what you mean. The script that adds
bw compat indices indeed needs updating to update the `oldVersions` array when
introducing bw compat indices fo
uschindler commented on PR #13096:
URL: https://github.com/apache/lucene/pull/13096#issuecomment-1940917313
With our last change, we may need to change the script that adds the
backwards indexes. Or is this now how it worked before?
--
This is an automated message from the Apache Git Serv
sabi0 commented on PR #13092:
URL: https://github.com/apache/lucene/pull/13092#issuecomment-1940885691
I see that `BackwardsCompatibilityTestBase` is failing after switching to
the fresh 'main':
```
AssertionError: Version: 9.11.0 missing [in thread "Test worker"]
at
jpountz commented on code in PR #13067:
URL: https://github.com/apache/lucene/pull/13067#discussion_r1487442233
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/TestIndexSortBackwardsCompatibility.java:
##
@@ -82,14 +84,20 @@ public void testSortedIndexAddDocB
jpountz commented on PR #13096:
URL: https://github.com/apache/lucene/pull/13096#issuecomment-1940726022
I pushed a new commit that revives the old logic that allowed up to one
version missing backward-compatibility data.
--
This is an automated message from the Apache Git Service.
To res
43 matches
Mail list logo
