[jira] [Commented] (LUCENE-9997) Revisit smoketester for 9.0 build

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17429869#comment-17429869 ] Dawid Weiss commented on LUCENE-9997: - Filed a PR here: https://github.com/apache/l

[GitHub] [lucene] rmuir commented on a change in pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
rmuir commented on a change in pull request #389: URL: https://github.com/apache/lucene/pull/389#discussion_r730728533 ## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene90/Lucene90DocValuesProducer.java ## @@ -1402,15 +1403,23 @@ public int docID() { @Ov

[GitHub] [lucene] rmuir commented on pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
rmuir commented on pull request #389: URL: https://github.com/apache/lucene/pull/389#issuecomment-945586019 Seems a lot easier to just test this via SORTED_NUMERIC. There isn't any point in testing SORTED_SET as it only makes things more difficult. The test should fail without the f

[GitHub] [lucene] jpountz commented on a change in pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
jpountz commented on a change in pull request #389: URL: https://github.com/apache/lucene/pull/389#discussion_r730745755 ## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene90/Lucene90DocValuesProducer.java ## @@ -1402,15 +1403,23 @@ public int docID() { @

[GitHub] [lucene] jpountz commented on pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
jpountz commented on pull request #389: URL: https://github.com/apache/lucene/pull/389#issuecomment-945593285 > Why do we have all this delegation? It was a way to have sorted set doc values automatically benefit from the specialization of sorted numeric doc values without duplicatin

[GitHub] [lucene] janhoy commented on pull request #392: LUCENE-9997: Collect signed maven artifacts if -Psign is passed.

2021-10-18 Thread GitBox
janhoy commented on pull request #392: URL: https://github.com/apache/lucene/pull/392#issuecomment-945617644 I run the build with this patch, but get errors ``` > Task :lucene:backward-codecs:javadocJar Caching disabled for task ':lucene:backward-codecs:javadocJar' because: Bui

[GitHub] [lucene] janhoy commented on pull request #392: LUCENE-9997: Collect signed maven artifacts if -Psign is passed.

2021-10-18 Thread GitBox
janhoy commented on pull request #392: URL: https://github.com/apache/lucene/pull/392#issuecomment-945628415 I made the build pass when trying the java-based signing (`-Psign -Psigning.keyId="" -Psigning.password=$GPGPASS -Psigning.secretKeyRingFile=$HOME/.gnupg/secring.gpg`), but

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Stefan Vodita (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17429927#comment-17429927 ] Stefan Vodita commented on LUCENE-10182: I pushed a change renaming {{RamUsageT

[jira] [Commented] (LUCENE-10054) Handle hierarchy in HNSW graph

2021-10-18 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17429958#comment-17429958 ] Michael McCandless commented on LUCENE-10054: - [~mayya] this looks like an

[GitHub] [lucene] jpountz commented on pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
jpountz commented on pull request #389: URL: https://github.com/apache/lucene/pull/389#issuecomment-945690914 > As far as Asserting, maybe you misunderstand the issue Sorry you are right indeed, AssertingDVFormat cannot help here. -- This is an automated message from the Apache Git

[GitHub] [lucene] dweiss commented on pull request #392: LUCENE-9997: Collect signed maven artifacts if -Psign is passed.

2021-10-18 Thread GitBox
dweiss commented on pull request #392: URL: https://github.com/apache/lucene/pull/392#issuecomment-945778193 Eh. Ok, I'll take another look. It's all a bit of a black-box with those plugins... -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [lucene] xiaoshi2013 commented on pull request #301: LUCENE-10100: same as 10091 Fix some old errors in the main branch

2021-10-18 Thread GitBox
xiaoshi2013 commented on pull request #301: URL: https://github.com/apache/lucene/pull/301#issuecomment-945829893 Hi @mikemccand , I've updated some of the descriptions, can you take a look at this pr? -- This is an automated message from the Apache Git Service. To respond to the mes

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430044#comment-17430044 ] Dawid Weiss commented on LUCENE-10182: -- I like this explicit version better. [~usc

[GitHub] [lucene] mocobeta commented on a change in pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
mocobeta commented on a change in pull request #391: URL: https://github.com/apache/lucene/pull/391#discussion_r731013013 ## File path: dev-tools/scripts/smokeTestRelease.py ## @@ -1033,7 +1030,7 @@ def confirmAllReleasesAreTestedForBackCompat(smokeVersion, unpackPath): os.

[GitHub] [lucene] mocobeta commented on a change in pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
mocobeta commented on a change in pull request #391: URL: https://github.com/apache/lucene/pull/391#discussion_r731013013 ## File path: dev-tools/scripts/smokeTestRelease.py ## @@ -1033,7 +1030,7 @@ def confirmAllReleasesAreTestedForBackCompat(smokeVersion, unpackPath): os.

[GitHub] [lucene] dweiss commented on pull request #392: LUCENE-9997: Collect signed maven artifacts if -Psign is passed.

2021-10-18 Thread GitBox
dweiss commented on pull request #392: URL: https://github.com/apache/lucene/pull/392#issuecomment-945885397 I've corrected the setup to configure the signing plugin across all projects - this should fix the problem. It works for me locally (with gpg), although on Windows it displays concu

[GitHub] [lucene] mocobeta commented on a change in pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
mocobeta commented on a change in pull request #391: URL: https://github.com/apache/lucene/pull/391#discussion_r731075943 ## File path: dev-tools/scripts/smokeTestRelease.py ## @@ -539,58 +529,72 @@ def unpackAndVerify(java, tmpDir, artifact, gitRevision, version, testArgs):

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430077#comment-17430077 ] Uwe Schindler commented on LUCENE-10182: +1. I approved the pull request. > Te

[GitHub] [lucene] dnhatn commented on pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
dnhatn commented on pull request #389: URL: https://github.com/apache/lucene/pull/389#issuecomment-945953376 > Seems a lot easier to just test this via SORTED_NUMERIC. There isn't any point in testing SORTED_SET as it only makes things more difficult. @rmuir @jpountz Thanks for your

[jira] [Assigned] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uwe Schindler reassigned LUCENE-10182: -- Assignee: Uwe Schindler > TestRamUsageEstimator asserts trivial equality > -

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430123#comment-17430123 ] Uwe Schindler commented on LUCENE-10182: I will merge this later in the evening

[jira] [Commented] (LUCENE-9997) Revisit smoketester for 9.0 build

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430157#comment-17430157 ] Dawid Weiss commented on LUCENE-9997: - The smoke tester writes a repository-local fi

[jira] [Commented] (LUCENE-9997) Revisit smoketester for 9.0 build

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430165#comment-17430165 ] Dawid Weiss commented on LUCENE-9997: - Also, the beauty of gradle is that we (in the

[GitHub] [lucene] dweiss merged pull request #392: LUCENE-9997: Collect signed maven artifacts if -Psign is passed.

2021-10-18 Thread GitBox
dweiss merged pull request #392: URL: https://github.com/apache/lucene/pull/392 -- 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...

[GitHub] [lucene] dweiss commented on pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
dweiss commented on pull request #391: URL: https://github.com/apache/lucene/pull/391#issuecomment-946072163 I merged with main where the artifact-signing changes landed. The buildAndPushRelease.py works for me (although it does create that stray file so I suspect it won't run properly unl

[GitHub] [lucene] mikemccand commented on pull request #375: LUCENE-10093: first cut at fixing conflicting test assert and improving TMP javadocs

2021-10-18 Thread GitBox
mikemccand commented on pull request #375: URL: https://github.com/apache/lucene/pull/375#issuecomment-946076421 OK I think this one is ready -- I downgraded all `// nocommit` to `TODO`s and merged in `main` again. For some reason `./gradlew check` in my local env fails with this:

[GitHub] [lucene] dweiss commented on pull request #375: LUCENE-10093: first cut at fixing conflicting test assert and improving TMP javadocs

2021-10-18 Thread GitBox
dweiss commented on pull request #375: URL: https://github.com/apache/lucene/pull/375#issuecomment-946079849 gradlew clean only cleans active modules. packaging has been renamed and no longer exists. Wipe all old cruft with: git clean -xfd lucene -- This is an automated message from t

[jira] [Created] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
Robert Muir created LUCENE-10185: Summary: gradle check fails on java 17 (security manager deprecation) Key: LUCENE-10185 URL: https://issues.apache.org/jira/browse/LUCENE-10185 Project: Lucene - Core

[jira] [Commented] (LUCENE-9997) Revisit smoketester for 9.0 build

2021-10-18 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430176#comment-17430176 ] ASF subversion and git services commented on LUCENE-9997: - Commi

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Stefan Vodita (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430179#comment-17430179 ] Stefan Vodita commented on LUCENE-10182: I’d be happy to help with this. I saw

[GitHub] [lucene] mikemccand commented on pull request #375: LUCENE-10093: first cut at fixing conflicting test assert and improving TMP javadocs

2021-10-18 Thread GitBox
mikemccand commented on pull request #375: URL: https://github.com/apache/lucene/pull/375#issuecomment-946100745 > gradlew clean only cleans active modules. packaging has been renamed and no longer exists. Wipe all old cruft with: > git clean -xfd lucene Aha! Thermonuclear clean

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430189#comment-17430189 ] Robert Muir commented on LUCENE-10185: -- The following patch seems to fix the issue

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430190#comment-17430190 ] Robert Muir commented on LUCENE-10185: -- I also tried upgrading ecj from 3.25.0 to

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430193#comment-17430193 ] Dawid Weiss commented on LUCENE-10185: -- Weird. Can't see why it'd generate a warni

[GitHub] [lucene] dweiss commented on pull request #375: LUCENE-10093: first cut at fixing conflicting test assert and improving TMP javadocs

2021-10-18 Thread GitBox
dweiss commented on pull request #375: URL: https://github.com/apache/lucene/pull/375#issuecomment-946114742 Yeah, sorry about it. We could add an exclusion to rat-checks but these proliferate over time... When you're switching branches, git will leave temporary files behind, it sometimes

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430196#comment-17430196 ] Robert Muir commented on LUCENE-10185: -- [~dweiss] in the ECJ at least these option

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430197#comment-17430197 ] Dawid Weiss commented on LUCENE-10185: -- I can actually see this warning in intelli

[jira] [Commented] (LUCENE-10054) Handle hierarchy in HNSW graph

2021-10-18 Thread Mayya Sharipova (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430199#comment-17430199 ] Mayya Sharipova commented on LUCENE-10054: -- [~mikemccand] Thanks for checking.

[GitHub] [lucene] rmuir opened a new pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
rmuir opened a new pull request #393: URL: https://github.com/apache/lucene/pull/393 Otherwise, new java releases such as JDK 18, JDK 19, ... may have even more new deprecations, the build shouldn't fail in such cases. Note: I did a very simple suppression-fix for the AttributeFact

[GitHub] [lucene] dweiss commented on a change in pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
dweiss commented on a change in pull request #393: URL: https://github.com/apache/lucene/pull/393#discussion_r731285380 ## File path: lucene/core/src/java/org/apache/lucene/util/AttributeFactory.java ## @@ -148,7 +148,10 @@ public int hashCode() { AttributeFactory delega

[GitHub] [lucene] rmuir commented on pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
rmuir commented on pull request #393: URL: https://github.com/apache/lucene/pull/393#issuecomment-946137876 thanks, i nuked `-source/-release`, just wanted to test that it worked first... -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [lucene] rmuir edited a comment on pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
rmuir edited a comment on pull request #393: URL: https://github.com/apache/lucene/pull/393#issuecomment-946137876 thanks, i nuked `-source/-target`, just wanted to test that it worked first... -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [lucene] rmuir commented on pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
rmuir commented on pull request #393: URL: https://github.com/apache/lucene/pull/393#issuecomment-946139296 I am running `gradle check` with the latest commits on JDK-17 before pushing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [lucene] uschindler commented on a change in pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
uschindler commented on a change in pull request #393: URL: https://github.com/apache/lucene/pull/393#discussion_r731292709 ## File path: lucene/core/src/java/org/apache/lucene/util/AttributeFactory.java ## @@ -148,7 +148,10 @@ public int hashCode() { AttributeFactory de

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430213#comment-17430213 ] Uwe Schindler commented on LUCENE-10182: That's just a cherry-pick across diffe

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Stefan Vodita (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430216#comment-17430216 ] Stefan Vodita commented on LUCENE-10182: Oh, I get it now. Thanks for the expla

[GitHub] [lucene] rmuir merged pull request #393: LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

2021-10-18 Thread GitBox
rmuir merged pull request #393: URL: https://github.com/apache/lucene/pull/393 -- 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...@

[jira] [Resolved] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Muir resolved LUCENE-10185. -- Fix Version/s: main (9.0) Resolution: Fixed > gradle check fails on java 17 (security

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430219#comment-17430219 ] ASF subversion and git services commented on LUCENE-10185: -- Co

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430221#comment-17430221 ] Robert Muir commented on LUCENE-10185: -- {quote} I can actually see this warning in

[jira] [Commented] (LUCENE-10180) Remove usage of lambdas in SegmentMerger?

2021-10-18 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430229#comment-17430229 ] Vigya Sharma commented on LUCENE-10180: --- I would like to work on this issue and g

[jira] [Comment Edited] (LUCENE-10180) Remove usage of lambdas in SegmentMerger?

2021-10-18 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430229#comment-17430229 ] Vigya Sharma edited comment on LUCENE-10180 at 10/18/21, 9:28 PM: ---

[GitHub] [lucene-solr] vigyasharma commented on a change in pull request #2573: LUCENE-10008: Respect ignoreCase flag in CommonGramsFilterFactory

2021-10-18 Thread GitBox
vigyasharma commented on a change in pull request #2573: URL: https://github.com/apache/lucene-solr/pull/2573#discussion_r731346908 ## File path: lucene/CHANGES.txt ## @@ -648,6 +648,8 @@ Optimizations Bug Fixes - +* LUCENE-10008: Respect ignoreCase in C

[GitHub] [lucene] uschindler merged pull request #386: LUCENE-10182: Be specific about which sizeOf() is called

2021-10-18 Thread GitBox
uschindler merged pull request #386: URL: https://github.com/apache/lucene/pull/386 -- 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-unsubsc

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430241#comment-17430241 ] ASF subversion and git services commented on LUCENE-10182: -- Co

[jira] [Commented] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430243#comment-17430243 ] Uwe Schindler commented on LUCENE-10182: I merged the PR. Will check backports

[jira] [Updated] (LUCENE-10182) TestRamUsageEstimator asserts trivial equality

2021-10-18 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10182?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uwe Schindler updated LUCENE-10182: --- Fix Version/s: main (9.0) > TestRamUsageEstimator asserts trivial equality > --

[GitHub] [lucene] janhoy commented on pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
janhoy commented on pull request #391: URL: https://github.com/apache/lucene/pull/391#issuecomment-946219082 > I merged with main where the artifact-signing changes landed. The buildAndPushRelease.py works for me (although it does create that stray file so I suspect it won't run properly u

[GitHub] [lucene] rmuir commented on pull request #389: LUCENE-10159: Fix invalid access in sorted set dv

2021-10-18 Thread GitBox
rmuir commented on pull request #389: URL: https://github.com/apache/lucene/pull/389#issuecomment-946248949 > Sorted set doc values don't have a docValueCount API, they're just expected to return NO_MORE_ORDS when all ords have been exhausted. Thanks, sorry I had completely forgotten

[jira] [Commented] (LUCENE-10180) Remove usage of lambdas in SegmentMerger?

2021-10-18 Thread Michael Sokolov (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430276#comment-17430276 ] Michael Sokolov commented on LUCENE-10180: -- It looks as if [~jpountz] posted a

[GitHub] [lucene] mocobeta opened a new pull request #394: LUCENE-9997: write release revision to system temp dir

2021-10-18 Thread GitBox
mocobeta opened a new pull request #394: URL: https://github.com/apache/lucene/pull/394 Suggested at: - https://github.com/apache/lucene/pull/382#discussion_r729615274 - https://issues.apache.org/jira/browse/LUCENE-9997?focusedCommentId=17430157&page=com.atlassian.jira.plugin.syst

[jira] [Commented] (LUCENE-9997) Revisit smoketester for 9.0 build

2021-10-18 Thread Tomoko Uchida (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430278#comment-17430278 ] Tomoko Uchida commented on LUCENE-9997: --- bq. The smoke tester writes a repository-

[GitHub] [lucene] mocobeta commented on pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
mocobeta commented on pull request #391: URL: https://github.com/apache/lucene/pull/391#issuecomment-946304048 With skipping sthe ign verification (`--not-signed`), smoketester stopped here for me. ``` unpack lucene-9.0.0.tgz... verify that Maven artifacts are same as i

[GitHub] [lucene] mocobeta edited a comment on pull request #391: LUCENE-9997 Second pass smoketester fixes for 9.0

2021-10-18 Thread GitBox
mocobeta edited a comment on pull request #391: URL: https://github.com/apache/lucene/pull/391#issuecomment-946304048 With skipping the sign verification (`--not-signed`), smoketester stopped here for me. ``` unpack lucene-9.0.0.tgz... verify that Maven artifacts are sa

[jira] [Commented] (LUCENE-10185) gradle check fails on java 17 (security manager deprecation)

2021-10-18 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430349#comment-17430349 ] Dawid Weiss commented on LUCENE-10185: -- It's fine, really. This is what struck me