In general, as long as it works in the new jvm, I don't see us changing the code. However, if you have the time to do it. The upgrade classes are meant to be immutable as far as data goes. So the idea is we can do upgrades from any version to the current version. Because of that, then 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 > > H, > > I have commited a patch to Upgrade430to440.java that removes all > close() calls on PreparedStament en ResultSet objects, replacing it with the > try(<resouce>) 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 versions of java. In master (and branch 4.4) however > these are only to be run with java 7. So should we fix the upgrades by try- > with-resources or not? > > -- > Daan
