[
https://issues.apache.org/jira/browse/LUCENE-10330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463197#comment-17463197
]
ASF subversion and git services commented on LUCENE-10330:
----------------------------------------------------------
Commit 0f41b5511e8ababe5a8a76d168165aa81f0a59e8 in lucene's branch
refs/heads/branch_9x from Uwe Schindler
[ https://gitbox.apache.org/repos/asf?p=lucene.git;h=0f41b55 ]
LUCENE-10330: Make MMapDirectory tests fail by default, if unmapping does not
work (add changes)
> Make the mmap directory tests fail by default, if unmapping does not work
> -------------------------------------------------------------------------
>
> Key: LUCENE-10330
> URL: https://issues.apache.org/jira/browse/LUCENE-10330
> Project: Lucene - Core
> Issue Type: Improvement
> Components: core/store
> Affects Versions: 9.0, 9.1, 10.0 (main)
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Priority: Major
> Time Spent: 2h
> Remaining Estimate: 0h
>
> Since the recent refactoring with Java modules, we figured out that we have
> no tests, that MMapDirectory always works correctly and is choosen as
> default, e.g. because you have a module setup where {{jdk.unsupported}} is
> missing (last seen on Luke) or since the refactoring of tests after
> LUCENE-10255 (see also discussion about how tests should be ran with module
> system in LUCENE-10328).
> Currently TestMMapDirectory and similar ones silently disable themselves if
> the "Unmap Hack" is not detected. The warning is hidden by gradle, so that's
> a bad default.
> We should simply fail the test when unmapping does not work and add an option
> to "still run tests if you know what you're doing". All current Java releases
> are OpenJDK based (at least those in main/9.x as those use JDK 11), so the
> unmap hack always works.
> Of course MMapDirectory still disables itsself at runtime (e.g. when security
> policy is preventing it or if user has module system and does not add it --
> we should add this to the warning message), but we should make sure that we
> test it on known platforms.
> Another option would be to check the assume and modify it to also check the
> version tring for "OpenJDK", "Oracle" and enforce that test is enabled. But
> this is too risky, so I would simply fail test if unmapping does not work
> with a system property to disable checks.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
