TimeUnits (Re: svn commit: r950396)

[David Blevins <da...@visi.com>]

On Jun 10, 2010, at 2:39 AM, Jean-Louis MONTEIRO wrote:

> Ivan Xu wrote:
>> 
>> Do OpenEJB 3.1.* has the minium JRE version of 1.6 ? If not, I guess we
>> could not use TimeUnit.MINUTES, it only exists in JDK 1.6.
>> Thanks !
>> 
> 
> Absolutely!
> 3.1.x --> Java EE 5
> 3.2.x --> Java EE 6
> 
> Can you fill a JIRA and push a patch?

Good catch.  This was my mistake so just went ahead and fixed it.  I converted the 3.1.x version to use Duration.Unit, so we should be good.

On that note, the Duration.Unit enum was only there to work around the lack of MINUTES, HOURS, and DAYS in the Java 5 TimeUnit enum, so now that we are in Java 6 in trunk, I went ahead and finally cleaned up that class.


Thanks for the heads up, Ivan!


-David