On Jun 1, 2010, at 4:48 PM, David Blevins wrote: > > On Jun 1, 2010, at 7:06 AM, Kevan Miller wrote: > >> >> On May 27, 2010, at 5:03 PM, [email protected] wrote: >> >>> Author: dblevins >>> Date: Thu May 27 21:03:52 2010 >>> New Revision: 948999 >>> >>> URL: http://svn.apache.org/viewvc?rev=948999&view=rev >>> Log: >>> svn merge -r 948242:948243 >>> https://svn.apache.org/repos/asf/openejb/trunk/openejb3 >>> >>> http://svn.apache.org/viewvc?rev=948243&view=rev >>> ------------------------------------------------------------------------ >>> r948243 | djencks | 2010-05-25 16:03:16 -0700 (Tue, 25 May 2010) | 1 line >>> >>> OPENEJB-1014 remove a lot of incorrect/misleading java: prefixes when >>> accessing local jndi rather than an initial context >> >> David, >> This is going to break Geronimo 2.2.x. The associated jndi changes that make >> this possible are only in Geronimo 3.0. So, we either need to revert to >> previous (admittedly faulty) functionality or do a fair amount of work to >> have Geronimo 2.2.1-SNAPSHOT behave properly. > > Side note, IMO, not supporting it is a failure. > >> >> Was there a strong motivation to merge this back into 3.1.x? > > No, was just trying to keep them in sync where possible. > > Are we seeing failures due to this?
Yes. Running a simple JMS test fails with the following: javax.naming.NameNotFoundException: comp at org.apache.xbean.naming.context.AbstractContext.lookup(AbstractContext.java:167) at org.apache.xbean.naming.context.AbstractContext.lookup(AbstractContext.java:619) at org.apache.xbean.naming.context.AbstractContext.lookupFinalContext(AbstractContext.java:196) at org.apache.xbean.naming.context.AbstractContext.addDeepBinding(AbstractContext.java:241) at org.apache.xbean.naming.context.AbstractContext.bind(AbstractContext.java:660) at org.apache.xbean.naming.context.AbstractContext.bind(AbstractContext.java:651) at org.apache.openejb.core.mdb.MdbInstanceFactory.constructBean(MdbInstanceFactory.java:188) at org.apache.openejb.core.mdb.MdbInstanceFactory.createInstance(MdbInstanceFactory.java:112) at org.apache.openejb.core.mdb.EndpointHandler.<init>(EndpointHandler.java:79) at org.apache.openejb.core.mdb.EndpointFactory.createEndpoint(EndpointFactory.java:67) at org.apache.activemq.ra.ServerSessionPoolImpl.createEndpoint(ServerSessionPoolImpl.java:92) at org.apache.activemq.ra.ServerSessionPoolImpl.createServerSessionImpl(ServerSessionPoolImpl.java:76) at org.apache.activemq.ra.ServerSessionPoolImpl.getServerSession(ServerSessionPoolImpl.java:116) at org.apache.activemq.ActiveMQConnectionConsumer.dispatch(ActiveMQConnectionConsumer.java:129) at org.apache.activemq.ActiveMQConnection$2.processMessageDispatch(ActiveMQConnection.java:1707) at org.apache.activemq.command.MessageDispatch.visit(MessageDispatch.java:108) at org.apache.activemq.ActiveMQConnection.onCommand(ActiveMQConnection.java:1688) at org.apache.activemq.transport.ResponseCorrelator.onCommand(ResponseCorrelator.java:116) at org.apache.activemq.transport.TransportFilter.onCommand(TransportFilter.java:69) at org.apache.activemq.transport.TransportFilter.onCommand(TransportFilter.java:69) at org.apache.activemq.transport.fanout.FanoutTransport.transportListenerOnCommand(FanoutTransport.java:581) at org.apache.activemq.transport.fanout.FanoutTransport$FanoutTransportHandler.onCommand(FanoutTransport.java:127) at org.apache.activemq.transport.WireFormatNegotiator.onCommand(WireFormatNegotiator.java:113) at org.apache.activemq.transport.InactivityMonitor.onCommand(InactivityMonitor.java:217) at org.apache.activemq.transport.TransportSupport.doConsume(TransportSupport.java:83) at org.apache.activemq.transport.tcp.TcpTransport.doRun(TcpTransport.java:219) at org.apache.activemq.transport.tcp.TcpTransport.run(TcpTransport.java:201) at java.lang.Thread.run(Thread.java:637) Running the Geronimo Integration Tests will show a fair number of failures, also... --kevan
