7u40: Request for approval for CR 8013434 - Xalan and Xerces internal ObjectFactory need rework

[Seán Coffey]

Daniel,

with the b30 code freeze already here, I'd suggest logging a phase 2 
approval request.

regards,
Sean.

On 14/06/2013 09:16, Daniel Fuchs wrote:
> On 6/11/13 11:40 AM, Seán Coffey wrote:
>> Daniel,
>>
>> I just see that now. My understanding is that the "release" field in CCC
>> should specify the earliest fix release which the change is planned for.
>> I'm cc'ing Joe Darcy for advice. Joe - is this change ok for JDK 7u
>> integration ?
>
> Hi Seán,
>
> If Joe is OK - when would be the deadline for this to get integrated?
>
> -- daniel
>
>>
>> regards,
>> Sean.
>>
>> On 11/06/2013 10:10, Daniel Fuchs wrote:
>>> Hi Seán,
>>>
>>> On 6/11/13 10:31 AM, Seán Coffey wrote:
>>>> Thanks for the background information Joe. Can you run this by the CCC
>>>> team then ? (log a request for jdk7u)
>>>
>>> In the CCC I had mentioned our intent to backport the fix to
>>> jdk7u-dev - but maybe that went unnoticed...
>>>
>>> -- daniel
>>>
>>>>
>>>> regards,
>>>> Sean.
>>>>
>>>> On 10/06/2013 23:13, huizhe wang wrote:
>>>>> Hi Sean,
>>>>>
>>>>> It was on my request that Daniel took the trouble to make the
>>>>> backports on this and several of his previous JDK8 changes. Past
>>>>> experience with 6u during jdk7 development showed that it was very
>>>>> beneficial keeping the two in sync.
>>>>>
>>>>> For this particular one, the behavioral change exists only
>>>>> theoretically.  The original implementation allowed plugging-in a
>>>>> third party impl of DTMManager and/or XSLTCDTMManager, but it has
>>>>> become an obsolete design. The only known impl was in an old version
>>>>> of IBM SDK that was for IBM only.  In reality, the plugin layer is
>>>>> most often a performance drag and people actually set a System
>>>>> Property to point to the default JDK implementation to avoid reading
>>>>> the service file.
>>>>>
>>>>> So I think this change is beneficial to JDK7.  It's a performance
>>>>> improvement. It helps keeping it in sync with JDK8 source. It also
>>>>> removes lot of obsolete, unused code.
>>>>>
>>>>> Thanks,
>>>>> Joe
>>>>>
>>>>> On 6/10/2013 9:44 AM, Seán Coffey wrote:
>>>>>> Daniel,
>>>>>>
>>>>>> this looks like a behavioural change in an update release. I see you
>>>>>> logged a CCC for JDK 8 only.
>>>>>>
>>>>>> I don't believe it's suitable for 7u integration. Alan, Joe - any
>>>>>> thoughts ?
>>>>>>
>>>>>> regards,
>>>>>> Sean.
>>>>>>
>>>>>> On 10/06/13 10:56, Daniel Fuchs wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> This is a request for approval to backport JDK-8013434 to
>>>>>>> jdk7u-dev.
>>>>>>>
>>>>>>> With this changeset, DTMManager and XSLTCDTMManager will always use
>>>>>>> their own default implementation.
>>>>>>>
>>>>>>> JDK 8 Review thread:
>>>>>>> <http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-June/017606.html> 
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Bug URL:
>>>>>>> <https://jbs.oracle.com/bugs/browse/JDK-8013434>
>>>>>>>
>>>>>>> JDK 8 Changeset:
>>>>>>> <http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/e93beba07830>
>>>>>>>
>>>>>>> The backport to jdk7u-dev is a simple hg import of the jdk8
>>>>>>> changeset.
>>>>>>>
>>>>>>> -- daniel
>>>>>>
>>>>>
>>>>
>>>
>>
>