-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GEOT-7564 Allow to skip check on staleness of index files on IndexManager #4738
base: main
Are you sure you want to change the base?
GEOT-7564 Allow to skip check on staleness of index files on IndexManager #4738
Conversation
I'm familiar with the issue and believe the new flag is fine. One of the Linux build failed on the new test:
I did not check the test sources in detail, but the Github action VMs tend to be pretty slow and have erratic behavior time wise, please double check you're allowing plenty of time for tests to run. In GeoServer we use awaitility for this kind of test, it can check a condition periodically, exit as soon as it passes, but also allow a max wait and then fail.... It's also possible to implement something like that manually using short sleeps in a loop. Also, the QA build is failing due to formatting errors:
Building with Maven from the command line automatically fixes the sources, while you're at it I'd recommend doing on build with "-Dqa" to enable all QA tools locally. See the QA guide for more information. |
The test keeps on failing, please have a look at it:
|
When loading a shapefile, a check is made to ensure that the index files (.qif/.fix) are not "stale". This is done by comparing lastModified times of .shp and indexes files, if the latter are <= that .shp's lastModified the indexes are recreated.
The problem is that when files are copied, they will be considered stale and geotools will waste time recreating them.
I propose a new property to skip this check.
The changes are the following:
Checklist
main
branch (backports managed later; ignore for branch specific issues).For core and extension modules:
[GEOT-XYZW] Title of the Jira ticket
.