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

Miroslav Kos miroslav.kos at oracle.com
Fri Sep 11 15:58:50 UTC 2015


On 27/08/15 20:26, Miroslav Kos wrote:
> 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
Hi Alan,
would you have some time to check the changes?

Thanks
Miran





More information about the core-libs-dev mailing list