RE: [DISCUSS] try-with-resouce use

2014-06-06 Thread Rajani Karuturi
ogl...@gmail.com> Sent: ‎07-‎06-‎2014 01:53 To: dev<mailto:dev@cloudstack.apache.org> Subject: [DISCUSS] try-with-resouce use H, I have commited a patch to Upgrade430to440.java that removes all close() calls on PreparedStament en ResultSet objects, replacing it with the try() construct in j

Re: [DISCUSS] try-with-resouce use

2014-06-06 Thread Daan Hoogland
; > I don't see why older upgrade classes in master branch be run in older jvms. > > --Alex > >> -Original Message- >> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] >> Sent: Friday, June 6, 2014 1:23 PM >> To: dev >> Subject: [DISCUSS

RE: [DISCUSS] try-with-resouce use

2014-06-06 Thread Alex Huang
n all upgrade classes could be run in the new jvm. I don't see why older upgrade classes in master branch be run in older jvms. --Alex > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Friday, June 6, 2014 1:23 PM > To: dev > Subject: [

[DISCUSS] try-with-resouce use

2014-06-06 Thread Daan Hoogland
H, I have commited a patch to Upgrade430to440.java that removes all close() calls on PreparedStament en ResultSet objects, replacing it with the try() construct in java 7. I was tempted for a short moment to back-port it to older Upgrade classes. As those upgrade paths are to be supported by lower