scholzb-hb commented on issue #463:
URL:
https://github.com/apache/maven-build-cache-extension/issues/463#issuecomment-4217328457
In my opinion the decision to use a certain cache item is based on the wrong
logic.
The current logic seems to be implemented like this (pseudocode, I have
omitted the local cache for simplicity):
```
if (Downloaded item matching checksum) {
if (Downloaded item matching plugin parameters) {
Use it
} else {
Rebuild module <-- wrong behavior in our scenario
}
} else if (Remote item matching checksum) {
Download it
if (Downloaded item matching plugin parameters) {
Use it
} else {
Rebuild module
}
} else {
Rebuild module
}
```
The logic should be instead:
```
if (Downloaded item matching checksum AND plugin parameters) {
Use it
} else if (Remote item matching checksum AND plugin parameters) {
Download and use it <-- expected behavior in our scenario
} else {
Rebuild module
}
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
