RE: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
I'm so thrilled this is happening! Thanks! iris -Original Message- From: huizhe wang Sent: Monday, July 15, 2013 1:08 PM To: Iris Clark Cc: Lance Andersen; Core-Libs-Dev Subject: Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12 Yes, it just happened! SQE team has agreed and we'll start to plan on the migration! -Joe On 7/12/2013 4:57 PM, Iris Clark wrote: FWIW, I really like the idea of migrating the jaxp tests to the jaxp repo. I've always thought it odd that the code change and the test for it aren't in the same changeset. Thanks, iris -Original Message- From: huizhe wang Sent: Friday, July 12, 2013 10:10 AM To: Lance Andersen - Oracle Cc: Core-Libs-Dev Subject: Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12 On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
Yes, it just happened! SQE team has agreed and we'll start to plan on the migration! -Joe On 7/12/2013 4:57 PM, Iris Clark wrote: FWIW, I really like the idea of migrating the jaxp tests to the jaxp repo. I've always thought it odd that the code change and the test for it aren't in the same changeset. Thanks, iris -Original Message- From: huizhe wang Sent: Friday, July 12, 2013 10:10 AM To: Lance Andersen - Oracle Cc: Core-Libs-Dev Subject: Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12 On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
On 7/12/2013 2:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? Ah, I accidentally placed them together with those original tests. There's a place holder for jaxp: jdk/test/javax/xml/jaxp. A longer term goal may be to migrate tests to the jaxp repo. -Joe As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
On 12 Jul 2013, at 18:09, huizhe wang huizhe.w...@oracle.com wrote: On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Sounds good to me. -Chris Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
Thanks Chris! On 7/12/2013 10:59 AM, Chris Hegarty wrote: On 12 Jul 2013, at 18:09, huizhe wang huizhe.w...@oracle.com wrote: On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Sounds good to me. -Chris Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
RE: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12
FWIW, I really like the idea of migrating the jaxp tests to the jaxp repo. I've always thought it odd that the code change and the test for it aren't in the same changeset. Thanks, iris -Original Message- From: huizhe wang Sent: Friday, July 12, 2013 10:10 AM To: Lance Andersen - Oracle Cc: Core-Libs-Dev Subject: Re: RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12 On 7/12/2013 3:30 AM, Lance Andersen - Oracle wrote: The fix looks fine to me Joe as does the test. I assume you will go with Chris's plan below for the test locations? It will be jdk/test/javax/xml/jaxp, that is the location for all jaxp tests for now. We will look into migrating jaxp tests to the jaxp repo. Thanks, Joe Best Lance On Jul 12, 2013, at 5:44 AM, Chris Hegarty wrote: The source changes look fine to me. The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp? As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure. -Chris On 12 Jul 2013, at 09:59, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix on a regression caused by 8016648. The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check. I've run the tests used in the nightly build with a local build, all passed with this patch. webrev: http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/ Thanks, Joe Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com