[9] Review request JDK-8131667: JAX-WS Plugability Layer: using java.util.ServiceLoader

[Miroslav Kos]

On 20/08/15 19:18, Alan Bateman wrote:
> On 19/08/2015 16:37, Miroslav Kos wrote:
>> Hi everybody,
>>
>> I am sending changes for review.
>>
>> CCC: http://ccc.us.oracle.com/8131667
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8131667
>> webrev: http://cr.openjdk.java.net/~mkos/8131667/jaxws.01/
>>
>> It's about migrating to standard java.util.ServiceLoader. This part 
>> of service discovery was implemented previously "own" way. There are 
>> some changes in javadoc and implementation has been refactored in 
>> order to use same code as is in JAX-B and SAAJ.
> I skimmed through this and it mostly looks okay. One thing that 
> probably should be re-examined is the catch Exception in 
> fromJDKProperties as an exception accessing or parsing 
> jaxws.properties will be difficult to diagnose with the code as it is.
>
> -Alan
Hi Alan,
I added logging (at least for now) - would it work for you?
updated webrev: http://cr.openjdk.java.net/~mkos/8131667/jaxws.02/

If you think the exception shouldn't be ignored I suggest to solve this 
in separate issue (and maybe revisit all similar cases in SAAJ/JAX-*)

Thanks
Miran