uschindler commented on PR #11834:
URL: https://github.com/apache/lucene/pull/11834#issuecomment-1264657879
I added a issue link as TODO:
https://github.com/policeman-tools/forbidden-apis/issues/207
(the problem is that parsing the Java 19 classes leads to warnings about the
same mis
uschindler merged PR #11834:
URL: https://github.com/apache/lucene/pull/11834
--
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 opened a new pull request, #11835:
URL: https://github.com/apache/lucene/pull/11835
This is mostly a cleanup after we added a MR-JAR. The problem was some hacks
in the validation tasks, As long as soureSets are named "mainXX", with XX a
feature version, we check everything automa
uschindler merged PR #11835:
URL: https://github.com/apache/lucene/pull/11835
--
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.
dsmiley commented on PR #11807:
URL: https://github.com/apache/lucene/pull/11807#issuecomment-1264749687
+1 works for me; this change is the right thing to do, even if it surfaces
issues that can be separately fixed.
--
This is an automated message from the Apache Git Service.
To respond
uschindler commented on issue #11819:
URL: https://github.com/apache/lucene/issues/11819#issuecomment-1264782928
I have a solution for Intellij Idea: #11836
This variant disables compilation of the MR-JAR sourceSets (like in Eclipse).
--
This is an automated message from the Apache Git
uschindler opened a new pull request, #11836:
URL: https://github.com/apache/lucene/pull/11836
This closes #11819.
This is a bit of hack, but after trying a few times with Intellij Idea: When
Gradle is running in ideaSync mode, we do some late evaluation:
- remove all source folder
romseygeek merged PR #11807:
URL: https://github.com/apache/lucene/pull/11807
--
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.
romseygeek closed issue #11490: UnifiedHighlighter can miss terms because of
query rewrites [LUCENE-10454]
URL: https://github.com/apache/lucene/issues/11490
--
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 t
risdenk merged PR #2671:
URL: https://github.com/apache/lucene-solr/pull/2671
--
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 code in PR #11836:
URL: https://github.com/apache/lucene/pull/11836#discussion_r986079639
##
gradle/ide/intellij-idea.gradle:
##
@@ -37,6 +37,22 @@ if (isIdea) {
}
}
+if (isIdeaSync) {
+ allprojects {
+// disable all MR-JAR folders by hiding th
mdmarshmallow commented on issue #11761:
URL: https://github.com/apache/lucene/issues/11761#issuecomment-1269018833
I updated the PR to change the default as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
zhaih opened a new issue, #11838:
URL: https://github.com/apache/lucene/issues/11838
### Description
There're several queries relying on rewrite to do the most heavy work, such
as KnnVectorQuery and AutomatonQuery, should we make rewrite step being able to
run under multi-threading?
zhaih merged PR #11831:
URL: https://github.com/apache/lucene/pull/11831
--
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.apach
zhaih commented on PR #11831:
URL: https://github.com/apache/lucene/pull/11831#issuecomment-1269059590
Thanks @mdmarshmallow! I merged and backported it!
--
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
stefanvodita commented on PR #11780:
URL: https://github.com/apache/lucene/pull/11780#issuecomment-1269159465
Thinking some more about this change in light of the comments, I see 3 ways
it could go:
1. Do nothing. The user can already instantiate new reader states after a
refresh if they
rmuir merged PR #11837:
URL: https://github.com/apache/lucene/pull/11837
--
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.apach
rmuir opened a new issue, #11839:
URL: https://github.com/apache/lucene/issues/11839
### Description
Currently, when running the build (e.g. `gradlew coverage`) you get a
per-module report.
Ideally we'd produce an aggregate report (similar to what black magic
jenkins is doing:
zhaih commented on issue #11750:
URL: https://github.com/apache/lucene/issues/11750#issuecomment-1269218912
I **think** this looks like a bug (but probably not hurting a lot? Because
the segment is dropping anyway?) but I'm not 100% sure. @mikemccand do you have
any ideas?
--
This is an
zhaih commented on issue #11839:
URL: https://github.com/apache/lucene/issues/11839#issuecomment-1269234855
@rmuir I can work on it if you don't want to do it right now?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
20 matches
Mail list logo