RFR JDK-8197533 move javax.transaction.xa into its own module

Lance Andersen lance.andersen at oracle.com
Thu Mar 1 16:59:08 UTC 2018


> On Feb 28, 2018, at 8:29 PM, Paul Sandoz <paul.sandoz at oracle.com> wrote:
> 
> 
> 
>> On Feb 28, 2018, at 1:22 PM, Lance Andersen <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>> 
>>> 
>>> On Feb 28, 2018, at 2:20 PM, Lance Andersen <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>>> 
>>> Hi Paul,
>>> 
>>> Thank you for the review.
>>>> On Feb 28, 2018, at 1:40 PM, Paul Sandoz <paul.sandoz at oracle.com <mailto:paul.sandoz at oracle.com>> wrote:
>>>> 
>>>> Compatible module refactoring in action!
>>>> 
>>>> Looks good, one comment:
>>>> 
>>>> test/jdk/javax/transaction/xa/testng/JavaSqlModuleDriver.java
>>>> 
>>>> This is not a valid Java source file can you merge the jtreg meta data into XAExceptionTests instead?
>>> 
>>> As we discussed offline, I will change the above file and Driver.java.  I naively assumed this was OK as the change  to add Driver.java was made after I had originally added these tests in 2015.
>> 
>> http://cr.openjdk.java.net/~lancea/8197533/webrev.01/ <http://cr.openjdk.java.net/~lancea/8197533/webrev.01/> has the updated tests
> 
> +1, i second Joe’s request to update package-info.java while we are opportunistically cleaning this area up.

Per your and Joe’s request, please see cr.openjdk.java.net/~lancea/8197533/webrev.02 which has your suggested update to use package-info.java

Thank you both

Best
Lance
> 
> Paul.

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>






More information about the build-dev mailing list