[PR] Take an incremental update only when all needed chunks are listed [maven-indexer]

["slachiewicz (via GitHub)" <gi...@apache.org>]

slachiewicz opened a new pull request, #790:
URL: https://github.com/apache/maven-indexer/pull/790

   `canRetrieveAllChunks` in `DefaultIncrementalHandler` and in indexer-reader's `IndexReader` is documented as checking that all the chunks an update needs are still listed in the remote properties, but it returned `true` as soon as the chunk after the local counter was listed, and the chunk names were then generated up to `nexus.index.last-incremental`. Both now also require that every chunk up to the remote counter is listed and that the remote counter is not behind the local one; otherwise the update falls back to a full download. A remote counter that is not a number no longer throws `NumberFormatException` in indexer-core.


-- 
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-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Take an incremental update only when all needed chunks are listed [maven-indexer]

["slachiewicz (via GitHub)" <gi...@apache.org>]

slachiewicz merged PR #790:
URL: https://github.com/apache/maven-indexer/pull/790


-- 
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-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org