Issue Details (XML | Word | Printable)

Key: AMQ-1184
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Jason Dillon
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
ActiveMQ

Allow activeio 3.1-SNAPSHOT to be compiled with JDK 1.4

Created: 28/Feb/07 02:58 PM   Updated: 02/Mar/07 01:09 AM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 4.1.1

Time Tracking:
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works usejdk14.diff 2007-02-28 02:59 PM Jason Dillon 2 kB
Issue Links:
Related
 


 Description  « Hide
Use of one new-style for loop in activeio (and pom.xml config) is preventing activeio 3.1-SNAPSHOT from being used on JDK 1.4...

 All   Comments   Work Log   Change History   Subversion Commits   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
James Strachan added a comment - 01/Mar/07 10:40 AM
Patch applied with thanks - fingers crossed the nightly build will kick in soon...

Jason Dillon added a comment - 01/Mar/07 02:11 PM
Well... AIO is happy now... but looks like more Java 5 stuff is in the AMQ 4.1 snapshot:
java.lang.NoClassDefFoundError: java/util/concurrent/ExecutionException
        at org.apache.activemq.store.DefaultPersistenceAdapterFactory.createJournal(DefaultPersistenceAdapterFactory.java:224)
        at org.apache.activemq.store.DefaultPersistenceAdapterFactory.getJournal(DefaultPersistenceAdapterFactory.java:132)
        at org.apache.activemq.store.DefaultPersistenceAdapterFactory.createPersistenceAdapter(DefaultPersistenceAdapterFactory.java:70)
        at org.apache.activemq.broker.BrokerService.createPersistenceAdapter(BrokerService.java:1320)
        at org.apache.activemq.broker.BrokerService.getPersistenceAdapter(BrokerService.java:631)
        at org.apache.activemq.broker.BrokerService.createRegionBroker(BrokerService.java:1250)
        at org.apache.activemq.broker.BrokerService.createBroker(BrokerService.java:1209)
        at org.apache.activemq.broker.BrokerService.getBroker(BrokerService.java:508)
        at org.apache.activemq.broker.BrokerService.start(BrokerService.java:394)
        at org.apache.activemq.gbean.BrokerServiceGBeanImpl.doStart(BrokerServiceGBeanImpl.java:94)
        at org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java:984)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:267)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:102)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:124)
        at org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:543)
        at org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:379)
        at org.apache.geronimo.kernel.config.ConfigurationUtil.startConfigurationGBeans(ConfigurationUtil.java:378)
        at org.apache.geronimo.kernel.config.KernelConfigurationManager.start(KernelConfigurationManager.java:187)
        at org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:527)
        at org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:508)
        at org.apache.geronimo.kernel.config.SimpleConfigurationManager$$FastClassByCGLIB$$ce77a924.invoke(<generated>)
        at net.sf.cglib.reflect.FastMethod.invoke(FastMethod.java:53)
        ...

Jason Dillon added a comment - 01/Mar/07 08:07 PM
So... even though the build is fine, and generating JDK 1.4 compatible classes... this codebase is still importing JDK 1.5 stuff, so its not buildable with JDK 1.4... nor is it usable by JDK 1.4.

May need to retrotranslate activeio so that activemq 4.1 can use it... see AMQ-1185


James Strachan added a comment - 02/Mar/07 01:01 AM
We'll leave 3.1 trunk of activeio against Java 5 in case we need it for ActiveMQ 5.x (which is based on Java 5), so we'll move ActiveMQ 4.1 branch back to 3.0.0 of activeio to keep it on Java 1.4

Jason Dillon added a comment - 02/Mar/07 01:09 AM
If that works... great!