AlexanderAshitkin commented on pull request #617: URL: https://github.com/apache/maven/pull/617#issuecomment-979729650
> I'm going to merge. What's the reasoning for using an `Optional ` instead of throwing an exception ? Hi. The rationale is that cache misses are natural. Optionals allow to isolate normal flow from real errors (like transport and format issues) and to avoid flood of not so useful "not found" exceptions. If it is necessary to assert a resource presence it should be done in cache logic by a caller -- 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]
