RFR: JDK-8199352: The Jib artifact resolver in test lib needs to print better error messages
George Triantafillou
george.triantafillou at oracle.com
Thu Mar 8 23:15:30 UTC 2018
Erik,
Looks good.
-George
On 3/8/2018 5:08 PM, Erik Joelsson wrote:
> The Jib artifact resolver is not very good at telling us why things go
> wrong. The reason is that it swallows exceptions. This patch changes
> the API from throwing a FileNotFoundException, which I don't really
> think fits correctly in all cases, to a new API specific exception.
>
> I have greped for all uses of this API in the tests and changed the
> exception type caught at the caller location. I verified that I didn't
> break anything by compiling all the affected test classes and by
> running some of them for a bit.
>
> With these changes it should be easier to diagnose problems with
> resolving artifacts in the future.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8199352
>
> Webrev: http://cr.openjdk.java.net/~erikj/8199352/webrev.01/index.html
>
> /Erik
>
More information about the hotspot-dev
mailing list