[jira] Assigned: (GERONIMO-2286) app client plan still uses Strings for dependency Module IDs

2007-07-30 Thread David Jencks (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-2286?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Jencks reassigned GERONIMO-2286:
--

Assignee: David Jencks

> app client plan still uses Strings for dependency Module IDs
> 
>
> Key: GERONIMO-2286
> URL: https://issues.apache.org/jira/browse/GERONIMO-2286
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: application client, deployment
>Affects Versions: 1.1
>Reporter: Aaron Mulder
>Assignee: David Jencks
> Fix For: 1.x, 2.0
>
>
> The geronimo-application-client schema has:
> 
> 
> 
> 
> 
> 
> 
> 
> 
> That would typically be used like this:
>   
> tranql/tranql-connector/1.2/rar
>
> Everywhere else we've changed elements holding module IDs to be of 
> patternType, using separate groupId, artifactId, version, and type elements.  
> There's no reason external-rar should still be a single slash-delimited 
> String here (though internal-rar should be since it's presumably a path 
> within the JAR?).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (GERONIMO-3362) Enable Tomcat context level cluster

2007-07-30 Thread YunFeng Ma (JIRA)
Enable Tomcat context level cluster
---

 Key: GERONIMO-3362
 URL: https://issues.apache.org/jira/browse/GERONIMO-3362
 Project: Geronimo
  Issue Type: Bug
  Security Level: public (Regular issues)
  Components: Clustering
Affects Versions: 2.0.x, 2.1
Reporter: YunFeng Ma
 Fix For: 2.0.x


Geronimo v1.x supports tomcat context level cluster, but Geronimo v2.0.x only 
support tomcat host level cluster. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-3362) Enable Tomcat context level cluster

2007-07-30 Thread YunFeng Ma (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3362?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

YunFeng Ma updated GERONIMO-3362:
-

Attachment: GERONIMO-3362.patch

> Enable Tomcat context level cluster
> ---
>
> Key: GERONIMO-3362
> URL: https://issues.apache.org/jira/browse/GERONIMO-3362
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: Clustering
>Affects Versions: 2.0.x, 2.1
>Reporter: YunFeng Ma
> Fix For: 2.0.x
>
> Attachments: GERONIMO-3362.patch
>
>
> Geronimo v1.x supports tomcat context level cluster, but Geronimo v2.0.x only 
> support tomcat host level cluster. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: [jira] Updated: (GERONIMO-3326) ClassLoader memory leak caused by OpenJPA

2007-07-30 Thread Donald Woods

Kevan, is there an associated OpenJPA JIRA for this, so we can track its 
progress?


-Donald

Kevan Miller (JIRA) wrote:

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3326?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kevan Miller updated GERONIMO-3326:
---

Attachment: OpenJPAMemLeak-G.patch
OpenJPAMemLeak-OJPA.patch

I'm attaching patches for both OpenJPA and Geronimo that fixes the problem. 


Will need to work with the OpenJPA community to get the OpenJPA patch applied.

With this patch basic deploy/undeploy of DayTrader is much better. I've run 
over 350 deploy/undeploys. There may be a bit of memory growth. Without 
profiling, it's hard to tell. For infrequent operations like deploy/undeploy, 
I'd say things look very good...

I've only deployed/undeployed DT. Would be good to try DT operations in between 
deploy/undeploy...


ClassLoader memory leak caused by OpenJPA
-

Key: GERONIMO-3326
URL: https://issues.apache.org/jira/browse/GERONIMO-3326
Project: Geronimo
 Issue Type: Bug
 Security Level: public(Regular issues) 
 Components: persistence

   Affects Versions: 2.0-M7
   Reporter: Kevan Miller
   Assignee: Kevan Miller
   Priority: Critical
Fix For: 2.0

Attachments: OpenJPAMemLeak-G.patch, OpenJPAMemLeak-OJPA.patch


As David Jencks mentioned in GERONIMO-3305, there's a ClassLoader memory leak in deploy/undeploy. 
Geronimo is running out of PermGen space in some simple deploy/undeploy scenarios involving OpenJPA. The cause of the problem seems to be the _metas table in PCRegistry. _metas is a ConcurrentReferenceHashMap with WEAK reference keys and HARD reference values. The keys are the PersistenceCapable classes. While the values are the metadata for these classes which are maintained by the internal Meta class.

The cause of the ClassLoader memory leak is simple -- if any of the 
objects/classes held by the Meta class (e.g. fieldTypes) have also been loaded 
by the same ClassLoader used to load the PersistenceCapable class, the 
PersistenceCapable class (the weak key) will never be GCed. The value of the 
HashMap entry will always maintain a hard reference to the ClassLoader. Since 
the ClassLoader will never be GC'ed, the the the pcClass Class object will 
never be GC'able...
The problem can be easily recreated using current Geronimo trunk and the 
Geronimo Daytrader application.
FYI, here are the GC Roots for one of our ClassLoaders being leaked -- 
http://people.apache.org/~kevan/PCRegistryLeak.html
I've contacted the OpenJPA dev list. Hopefully, can get this resolved, quickly.




smime.p7s
Description: S/MIME Cryptographic Signature


[jira] Commented: (SM-1005) File BC doesnt set correlation Id and sender property

2007-07-30 Thread Gert Vanthienen (JIRA)

[ 
https://issues.apache.org/activemq/browse/SM-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39794
 ] 

Gert Vanthienen commented on SM-1005:
-

On my machine, the message exchanges sent by the file binding component already 
contain these properties.  Perhaps you're still using the 3.1 version, SM-861 
fixes this in 3.1.1 for exchanges that are sent using sendSync() (such as those 
sent by the FilePoller)

> File BC doesnt set correlation Id and sender property
> -
>
> Key: SM-1005
> URL: https://issues.apache.org/activemq/browse/SM-1005
> Project: ServiceMix
>  Issue Type: Bug
>  Components: servicemix-file
>Affects Versions: 3.1, 3.1.1
>Reporter: Gianfranco Boccalon
>
> The File binding component doesn't set the properties correlation id and 
> sender.
> We fixed the problem changing the method processFile in class 
> org.apache.servicemix.file.FilePollerEndpoint, adding these rows before 
> sending the message (sendSync):
>exchange.setProperty(JbiConstants.SENDER_ENDPOINT, service.toString());
>exchange.setProperty(JbiConstants.CORRELATION_ID, 
> exchange.getExchangeId());

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: [jira] Updated: (GERONIMO-3326) ClassLoader memory leak caused by OpenJPA

2007-07-30 Thread Kevan Miller


On Jul 30, 2007, at 8:07 AM, Donald Woods wrote:

Kevan, is there an associated OpenJPA JIRA for this, so we can  
track its progress?




https://issues.apache.org/jira/browse/OPENJPA-285


[jira] Created: (GERONIMO-3363) ArrayList thread safe problem in OpenJPA

2007-07-30 Thread YunFeng Ma (JIRA)
ArrayList thread safe problem in OpenJPA


 Key: GERONIMO-3363
 URL: https://issues.apache.org/jira/browse/GERONIMO-3363
 Project: Geronimo
  Issue Type: Bug
  Security Level: public (Regular issues)
  Components: persistence
Affects Versions: 2.0-M7
Reporter: YunFeng Ma
 Fix For: 2.0-M7


When running stress testing using DayTrader with JPA mode, got a lot of 
ArrayList thread safe problem.

The thread safe problem happened in  
org.apache.openjpa.kernel.AbstractBrokerFactory.syncWithManagedTransaction, but 
this method has the following comments:
// we don't need to synchronize on brokers or guard against multiple
// threads using the same trans since one JTA transaction can never
// be active on multiple concurrent threads.
Collection brokers = (Collection) _transactional.get(trans);
if (brokers == null) {
brokers = new ArrayList(2);
_transactional.put(trans, brokers);
trans.registerSynchronization(new RemoveTransactionSync(trans));
}
brokers.add(broker);

Does this mean that it's Geronimo which causes the thread safe problem?

The exception stack:
java.rmi.RemoteException: The bean encountered a non-application exception.; 
nested exception is:
javax.ejb.EJBException: TradeBean.getClosedOrders - error
at 
org.apache.openejb.core.transaction.TransactionPolicy.throwExceptionToServer(TransactionPolicy.java:211)
at 
org.apache.openejb.core.transaction.TxRequired.handleSystemException(TxRequired.java:106)
at 
org.apache.openejb.core.stateless.StatelessContainer._invoke(StatelessContainer.java:210)
at 
org.apache.openejb.core.stateless.StatelessContainer.invoke(StatelessContainer.java:165)
at 
org.apache.openejb.core.ivm.EjbObjectProxyHandler.businessMethod(EjbObjectProxyHandler.java:211)
at 
org.apache.openejb.core.ivm.EjbObjectProxyHandler._invoke(EjbObjectProxyHandler.java:65)
at 
org.apache.openejb.core.ivm.BaseEjbProxyHandler.invoke(BaseEjbProxyHandler.java:232)
at 
org.apache.openejb.util.proxy.Jdk13InvocationHandler.invoke(Jdk13InvocationHandler.java:49)
at $Proxy109.getClosedOrders(Unknown Source)
at 
org.apache.geronimo.samples.daytrader.TradeAction.getClosedOrders(TradeAction.java:276)
at 
org.apache.geronimo.samples.daytrader.web.OrdersAlertFilter.doFilter(OrdersAlertFilter.java:76)
at 
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
at 
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
at 
org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:230)
at 
org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
at 
org.apache.geronimo.tomcat.valve.DefaultSubjectValve.invoke(DefaultSubjectValve.java:56)
at 
org.apache.geronimo.tomcat.GeronimoStandardContext$SystemMethodValve.invoke(GeronimoStandardContext.java:351)

at 
org.apache.geronimo.tomcat.valve.GeronimoBeforeAfterValve.invoke(GeronimoBeforeAfterValve.java:47)
at 
org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:128)
at 
org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:104)
at 
org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
at 
org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:563)
at 
org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:261)
at 
org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:844)
at 
org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:581)
at 
org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:447)
at java.lang.Thread.run(Thread.java:801)
Caused by: javax.ejb.EJBException: TradeBean.getClosedOrders - error
at 
org.apache.geronimo.samples.daytrader.TradeJPA.getClosedOrders(TradeJPA.java:491)
at sun.reflect.GeneratedMethodAccessor324.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:615)
at 
org.apache.openejb.core.interceptor.ReflectionInvocationContext$Invocation.invoke(ReflectionInvocationContext
.java:146)
at 
org.apache.openejb.core.interceptor.ReflectionInvocationContext.proceed(ReflectionInvocationContext.java:129)

at 
org.apache.openejb.core.interceptor.InterceptorStack.invoke(InterceptorStack.java:67)
at 
org.apache.openejb.core.stateless.StatelessContainer._invoke(StatelessContainer.java:203)
... 24 more
Caused by: <1.0.0-SNAP

[jira] Updated: (GERONIMO-3363) ArrayList thread safe problem in OpenJPA

2007-07-30 Thread Donald Woods (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods updated GERONIMO-3363:
---

 Priority: Blocker  (was: Major)
Fix Version/s: (was: 2.0-M7)
   2.0

> ArrayList thread safe problem in OpenJPA
> 
>
> Key: GERONIMO-3363
> URL: https://issues.apache.org/jira/browse/GERONIMO-3363
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: persistence
>Affects Versions: 2.0-M7
>Reporter: YunFeng Ma
>Priority: Blocker
> Fix For: 2.0
>
>
> When running stress testing using DayTrader with JPA mode, got a lot of 
> ArrayList thread safe problem.
> The thread safe problem happened in  
> org.apache.openjpa.kernel.AbstractBrokerFactory.syncWithManagedTransaction, 
> but this method has the following comments:
> // we don't need to synchronize on brokers or guard against 
> multiple
> // threads using the same trans since one JTA transaction can 
> never
> // be active on multiple concurrent threads.
> Collection brokers = (Collection) _transactional.get(trans);
> if (brokers == null) {
> brokers = new ArrayList(2);
> _transactional.put(trans, brokers);
> trans.registerSynchronization(new 
> RemoveTransactionSync(trans));
> }
> brokers.add(broker);
> Does this mean that it's Geronimo which causes the thread safe problem?
> The exception stack:
> java.rmi.RemoteException: The bean encountered a non-application exception.; 
> nested exception is:
> javax.ejb.EJBException: TradeBean.getClosedOrders - error
> at 
> org.apache.openejb.core.transaction.TransactionPolicy.throwExceptionToServer(TransactionPolicy.java:211)
> at 
> org.apache.openejb.core.transaction.TxRequired.handleSystemException(TxRequired.java:106)
> at 
> org.apache.openejb.core.stateless.StatelessContainer._invoke(StatelessContainer.java:210)
> at 
> org.apache.openejb.core.stateless.StatelessContainer.invoke(StatelessContainer.java:165)
> at 
> org.apache.openejb.core.ivm.EjbObjectProxyHandler.businessMethod(EjbObjectProxyHandler.java:211)
> at 
> org.apache.openejb.core.ivm.EjbObjectProxyHandler._invoke(EjbObjectProxyHandler.java:65)
> at 
> org.apache.openejb.core.ivm.BaseEjbProxyHandler.invoke(BaseEjbProxyHandler.java:232)
> at 
> org.apache.openejb.util.proxy.Jdk13InvocationHandler.invoke(Jdk13InvocationHandler.java:49)
> at $Proxy109.getClosedOrders(Unknown Source)
> at 
> org.apache.geronimo.samples.daytrader.TradeAction.getClosedOrders(TradeAction.java:276)
> at 
> org.apache.geronimo.samples.daytrader.web.OrdersAlertFilter.doFilter(OrdersAlertFilter.java:76)
> at 
> org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
> at 
> org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
> at 
> org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:230)
> at 
> org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
> at 
> org.apache.geronimo.tomcat.valve.DefaultSubjectValve.invoke(DefaultSubjectValve.java:56)
> at 
> org.apache.geronimo.tomcat.GeronimoStandardContext$SystemMethodValve.invoke(GeronimoStandardContext.java:351)
> at 
> org.apache.geronimo.tomcat.valve.GeronimoBeforeAfterValve.invoke(GeronimoBeforeAfterValve.java:47)
> at 
> org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:128)
> at 
> org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:104)
> at 
> org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
> at 
> org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:563)
> at 
> org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:261)
> at 
> org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:844)
> at 
> org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:581)
> at 
> org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:447)
> at java.lang.Thread.run(Thread.java:801)
> Caused by: javax.ejb.EJBException: TradeBean.getClosedOrders - error
> at 
> org.apache.geronimo.samples.daytrader.TradeJPA.getClosedOrders(TradeJPA.java:491)
> at sun.reflect.GeneratedMethodAccessor324.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Me

[jira] Updated: (GERONIMO-3326) ClassLoader memory leak caused by OpenJPA

2007-07-30 Thread Donald Woods (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3326?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods updated GERONIMO-3326:
---

Priority: Blocker  (was: Critical)

> ClassLoader memory leak caused by OpenJPA
> -
>
> Key: GERONIMO-3326
> URL: https://issues.apache.org/jira/browse/GERONIMO-3326
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: persistence
>Affects Versions: 2.0-M7
>Reporter: Kevan Miller
>Assignee: Kevan Miller
>Priority: Blocker
> Fix For: 2.0
>
> Attachments: OpenJPAMemLeak-G.patch, OpenJPAMemLeak-OJPA.patch
>
>
> As David Jencks mentioned in GERONIMO-3305, there's a ClassLoader memory leak 
> in deploy/undeploy. 
> Geronimo is running out of PermGen space in some simple deploy/undeploy 
> scenarios involving OpenJPA. The cause of the problem seems to be the _metas 
> table in PCRegistry. _metas is a ConcurrentReferenceHashMap with WEAK 
> reference keys and HARD reference values. The keys are the PersistenceCapable 
> classes. While the values are the metadata for these classes which are 
> maintained by the internal Meta class.
> The cause of the ClassLoader memory leak is simple -- if any of the 
> objects/classes held by the Meta class (e.g. fieldTypes) have also been 
> loaded by the same ClassLoader used to load the PersistenceCapable class, the 
> PersistenceCapable class (the weak key) will never be GCed. The value of the 
> HashMap entry will always maintain a hard reference to the ClassLoader. Since 
> the ClassLoader will never be GC'ed, the the the pcClass Class object will 
> never be GC'able...
> The problem can be easily recreated using current Geronimo trunk and the 
> Geronimo Daytrader application.
> FYI, here are the GC Roots for one of our ClassLoaders being leaked -- 
> http://people.apache.org/~kevan/PCRegistryLeak.html
> I've contacted the OpenJPA dev list. Hopefully, can get this resolved, 
> quickly.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (GERONIMO-3349) broken link in console when on the Apache HTTP page

2007-07-30 Thread Donald Woods (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3349?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods closed GERONIMO-3349.
--

   Resolution: Fixed
Fix Version/s: 2.1

Committed revision 560981 in branches/2.0
Committed revision 560984 in trunk (2.1)
Viet Hung Nguyen, thanks for the patch.

> broken link in console when on the  Apache HTTP page
> 
>
> Key: GERONIMO-3349
> URL: https://issues.apache.org/jira/browse/GERONIMO-3349
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: console
>Affects Versions: 2.0-M6
> Environment: windows xp
>Reporter: Viet Hung Nguyen
>Assignee: Donald Woods
> Fix For: 2.0, 2.1
>
> Attachments: geronimo-3349.patch
>
>
> in the admin console, when running tomcat, if you click on 'Apache HTTP' and 
> click on 'Get Started' you do not see anything render. Instead, it warns this 
> at the commandline:
> WARN  [AJPHandler] Found AJP listener on port 8009

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: Tomcat connectors

2007-07-30 Thread Paul McMahan
FYI -- I'm working on fixing the console's web connector portlet for  
the new WebManager apis added to support these new connectors.



Best wishes,
Paul

On Jul 25, 2007, at 1:43 PM, Jeff Genender wrote:


Ok I added a whole bunch of new connectors in the o.a.g.t.connectors
package.

I am still working on APR - more notes to follow on this as its a  
little
squirly since the Tomcat Connector somewhat "chooses" this  
automatically

based on the existence of a native libraries.  For the console we may
wish to do a check on whether the native libs exist, and if so,  
present

the APR connector.  More on this in another email.

Here are the connectors we care about at the moment...

AJP13ConnectorGBean - Implements AJP
Http11ConnectorGBean - Implements blocking Http connector
Https11ConnectorGBean  - Implements blocking Https connector
Http11NIOConnectorGBean - Implements non-blocking Http connector
Https11NIOConnectorGBean - Implements non-blocking Https connector

I have not wired them into the container and other GBeans yet...I want
to clena them up and get any feedback before making the switch since
this obviously will impact the console upon wiring them in.

As a side note...I am not using any references to the WebManager or
other interfaces we used that hooked into the console.  We can re-add
those if those are deemed necessary.

Jeff

Paul McMahan wrote:
I agree NIO support would be great to have in 2.0, especially  
since its

required for comet.

Best wishes,
Paul

On Jul 23, 2007, at 2:42 PM, Jeff Genender wrote:


Hi,

I was going through some JIRAs and the Geronimo2.0 source and  
noticed it

will be difficult at best to get the NIO connector and setting
attributes on the APR connector for Tomcat due to its current
implementation.  I really think the ability to use these 2  
connectors is
very important for the 2.0 release and I would like to put these  
in.  If

there are no objections, I would like this to be a part of the 2.0
release.

Jeff




Re: [jira] Updated: (GERONIMO-3326) ClassLoader memory leak caused by OpenJPA

2007-07-30 Thread Kevan Miller


On Jul 30, 2007, at 8:07 AM, Donald Woods wrote:

Kevan, is there an associated OpenJPA JIRA for this, so we can  
track its progress?




By the way, you'd probably be interested in https://issues.apache.org/ 
jira/browse/OPENEJB-622


It's another ClassLoader memory leak. You can recreate by deploy DT,  
login/logout in EJB mode, undeploy DT, repeat.


There's a good chance that there are other problems, in a similar  
vein. Would be really cool to set up an automated test that deploys  
DT, logs in/logs out in each DT mode, executes a trade, undeploys DT.


--kevan


explicit artifact name in DefaultArtifactResolver

2007-07-30 Thread Kevan Miller
I ran across the following gem in DefaultArtifactResolver. Anybody  
know why it's there?


if (list.isEmpty()) {
if ("xbean-naming".equals(working.getArtifactId())) {
return new Artifact("org.apache.xbean", "xbean- 
naming", "2.8", "jar");

} else {
return null;
}
}

--kevan


[jira] Commented: (GERONIMO-1786) JMS Listeners for protocols activeio, peer and openwire fail to start

2007-07-30 Thread Donald Woods (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-1786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516393
 ] 

Donald Woods commented on GERONIMO-1786:


The Peer transport is enabled again in 2.0, but doesn't support this operation, 
so it needs to be disabled again -
10:11:57,375 ERROR [GBeanInstanceState] Error while starting; GBean is now in th
e FAILED state: abstractName="org.apache.geronimo.configs/activemq-broker/2.0-SN
APSHOT/car?JMSServer=ActiveMQ,ServiceModule=org.apache.geronimo.configs/activemq
-broker/2.0-SNAPSHOT/car,j2eeType=GBean,name=Peer"
java.io.IOException: This protocol does not support being bound.
at org.apache.activemq.transport.peer.PeerTransportFactory.doBind(PeerTr
ansportFactory.java:114)


> JMS Listeners for protocols activeio, peer and openwire fail to start
> -
>
> Key: GERONIMO-1786
> URL: https://issues.apache.org/jira/browse/GERONIMO-1786
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: ActiveMQ, console
>Affects Versions: 1.0, 1.1, 1.1.1, 1.2
> Environment: Geronimo 1.0.0
>Reporter: Donald Woods
>Assignee: Donald Woods
> Fix For: 2.0
>
>
> Even though addition of JMS Listeners for activeio, peer and openwire 
> protocols is successful, the listeners fail to
> start with the following exceptions.
> activeio --- java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> openwire --- javax.jms.JMSException: Could not load protocol: openwire. 
> Reason: java.lang.ClassNotFoundException:
> org.activemq.transport.openwire.OpenWireTransportServerChannelFactory
> peer --- javax.jms.JMSException: Could not load protocol: peer. Reason: 
> java.lang.ClassNotFoundException:
> org.activemq.transport.peer.PeerTransportServerChannelFactory
> Stack trace of the same.
> 192: 14:17:49,499 ERROR [GBeanInstanceState] Error while starting; GBean is 
> now in the FAILED state:
> objectName="geronimo.server:J2EEApplication=null,J2EEModule=geronimo/activemq-broker/1.0/car,J2EEServer=geronimo,br
> oker=ActiveMQ,j2eeType=JMSConnector,name=ActiveMQ.activeio.0.0.0.0.12340-aio"
> 193: java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> 194:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.createAsynchChannelServer(ActiveIOTranspo
> rtServerChannelFactory.java:60)
> 195:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.create(ActiveIOTransportServerChannelFact
> ory.java:49)
> 196:  at 
> org.activemq.transport.TransportServerChannelProvider.create(TransportServerChannelProvider.java:45)
> 197:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.createTransportServerChannel(BrokerConnectorImpl.java:425)
> 198:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.(BrokerConnectorImpl.java:69)
> 199:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.createBrokerConnector(ActiveMQConnectorGBean.java:161)
> 200:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.doStart(ActiveMQConnectorGBean.java:129)
> 201:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java(Compiled
>  Code))
> 202:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:325)
> 203:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:110)
> 204:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:132)
> 205:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:537)
> 206:  at 
> org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:208)
> 207:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor$StartRecursiveInvoke.invoke(ProxyMethodInterceptor.java
> :365)
> 208:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor.intercept(ProxyMethodInterceptor.java:96)
> 209:  at 
> org.activemq.gbean.ActiveMQConnector$$EnhancerByCGLIB$$2b4c85ae.startRecursive()
> 210:  at 
> org.apache.geronimo.console.jmsmanager.server.JMSConnectorPortlet.processAction(JMSConnectorPortlet.java:79)
> 211:  at 
> org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:229)
> 212:  at org.apache.pluto.core.PortletServlet.doGet(PortletServlet.java:158)
> 213:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:595)
> 214:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:688)
> 215:  at org.apache.pluto.core.PortletServlet.service(PortletServlet.java:153)
> 216:  at 
> org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilte

Re: [DISCUSS] Adding a "Tech Preview" portlet group in Admin Console in G 2.0

2007-07-30 Thread Kevan Miller


On Jul 30, 2007, at 2:04 AM, Vamsavardhana Reddy wrote:




On 7/30/07, Vamsavardhana Reddy <[EMAIL PROTECTED]> wrote:


On 7/30/07, Kevan Miller < [EMAIL PROTECTED]> wrote:

On Jul 28, 2007, at 1:41 AM, Vamsavardhana Reddy wrote:




On 7/28/07, Kevan Miller <[EMAIL PROTECTED]> wrote:

On Jul 26, 2007, at 1:21 AM, Vamsavardhana Reddy wrote:

How about adding a "Tech Preview" portlet group in Admin Console  
in G 2.0 where we can include the portlets (for e.g., the "Create  
Plan" portlet Shiva is working on.  See https://issues.apache.org/ 
jira/browse/GERONIMO-3254 ) for which user feedback will play a  
great role in improving those further.  Once we decide on whether  
to retain those or not, we can either move them out to a  
different group or remove them from Admin Console accordingly.


Some of the new pluggable console support would have helped, here.  
But that's not going into 2.0. Personally, sounds like this should  
go into trunk, if it's not ready... Can still get good feedback...


--kevan


The idea is that if these portlets go in as part of a release with  
binaries readily available for download, we have a better chance  
of getting more users to try the portlets and possibly provide  
some feedback too in the linked wiki pages.  If it goes only into  
trunk, the users will still have to build the binaries from trunk,  
which they may not want to/may not succeed due to some reason or  
the other.


Yes, I understood the idea.

If a committer thinks that this code is ready and wants to commit  
it, then why isn't it in trunk? If it was in trunk, then we could  
evaluate and discuss getting it into branches/2.0. I'm currently  
doubtful that it should go into 2.0...


I am not referring to code that may be delivering 100% of what it  
is expected to deliver.


I thought an example may help here.  The plan creator portlet that  
Shiva is working on, currently handles WAR files.  The fully  
functional one is expected to handle  WAR, EAR, EJB JAR,  etc.


A WAR plan creator sounds like useful capability in its own right. I  
don't see any reason why it couldn't be provided stand alone. If this  
capability was committed in trunk, provided useful capability, and  
had low risk for causing other problems, I would probably be in favor  
of putting it in the 2.0 release. I would not tag it with a "Tech  
Preview" label.


Is a WAR capability the function you'd like to put into 2.0? I'll  
repeat -- you're best starting point is to get the function into  
trunk...


This discussion has gotten me wondering about plan generation  
wizards, the j2g migration tool, and IDE tooling. Perhaps we should  
be thinking more holistically about these functions, rather than as  
disparate capabilities...


--kevan

[jira] Commented: (SM-1005) File BC doesnt set correlation Id and sender property

2007-07-30 Thread Gianfranco Boccalon (JIRA)

[ 
https://issues.apache.org/activemq/browse/SM-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39795
 ] 

Gianfranco Boccalon commented on SM-1005:
-

Sorry, I tried the 3.1.1 and it works properly: the properties are sent.

> File BC doesnt set correlation Id and sender property
> -
>
> Key: SM-1005
> URL: https://issues.apache.org/activemq/browse/SM-1005
> Project: ServiceMix
>  Issue Type: Bug
>  Components: servicemix-file
>Affects Versions: 3.1, 3.1.1
>Reporter: Gianfranco Boccalon
>
> The File binding component doesn't set the properties correlation id and 
> sender.
> We fixed the problem changing the method processFile in class 
> org.apache.servicemix.file.FilePollerEndpoint, adding these rows before 
> sending the message (sendSync):
>exchange.setProperty(JbiConstants.SENDER_ENDPOINT, service.toString());
>exchange.setProperty(JbiConstants.CORRELATION_ID, 
> exchange.getExchangeId());

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (GERONIMO-1786) JMS Listeners for protocols activeio, peer and openwire fail to start

2007-07-30 Thread Donald Woods (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-1786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516397
 ] 

Donald Woods commented on GERONIMO-1786:


Failover protocol does not support Add operation either -
10:22:41,140 ERROR [GBeanInstanceState] Error while starting; GBean is now in th
e FAILED state: abstractName="org.apache.geronimo.configs/activemq-broker/2.0-SN
APSHOT/car?JMSServer=ActiveMQ,ServiceModule=org.apache.geronimo.configs/activemq
-broker/2.0-SNAPSHOT/car,j2eeType=GBean,name=FailOver"
java.io.IOException: Invalid server URI: failover://localhost:61623
at org.apache.activemq.transport.failover.FailoverTransportFactory.doBin
d(FailoverTransportFactory.java:73)
at org.apache.activemq.transport.TransportFactory.bind(TransportFactory.
java:109)


> JMS Listeners for protocols activeio, peer and openwire fail to start
> -
>
> Key: GERONIMO-1786
> URL: https://issues.apache.org/jira/browse/GERONIMO-1786
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: ActiveMQ, console
>Affects Versions: 1.0, 1.1, 1.1.1, 1.2
> Environment: Geronimo 1.0.0
>Reporter: Donald Woods
>Assignee: Donald Woods
> Fix For: 2.0
>
>
> Even though addition of JMS Listeners for activeio, peer and openwire 
> protocols is successful, the listeners fail to
> start with the following exceptions.
> activeio --- java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> openwire --- javax.jms.JMSException: Could not load protocol: openwire. 
> Reason: java.lang.ClassNotFoundException:
> org.activemq.transport.openwire.OpenWireTransportServerChannelFactory
> peer --- javax.jms.JMSException: Could not load protocol: peer. Reason: 
> java.lang.ClassNotFoundException:
> org.activemq.transport.peer.PeerTransportServerChannelFactory
> Stack trace of the same.
> 192: 14:17:49,499 ERROR [GBeanInstanceState] Error while starting; GBean is 
> now in the FAILED state:
> objectName="geronimo.server:J2EEApplication=null,J2EEModule=geronimo/activemq-broker/1.0/car,J2EEServer=geronimo,br
> oker=ActiveMQ,j2eeType=JMSConnector,name=ActiveMQ.activeio.0.0.0.0.12340-aio"
> 193: java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> 194:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.createAsynchChannelServer(ActiveIOTranspo
> rtServerChannelFactory.java:60)
> 195:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.create(ActiveIOTransportServerChannelFact
> ory.java:49)
> 196:  at 
> org.activemq.transport.TransportServerChannelProvider.create(TransportServerChannelProvider.java:45)
> 197:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.createTransportServerChannel(BrokerConnectorImpl.java:425)
> 198:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.(BrokerConnectorImpl.java:69)
> 199:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.createBrokerConnector(ActiveMQConnectorGBean.java:161)
> 200:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.doStart(ActiveMQConnectorGBean.java:129)
> 201:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java(Compiled
>  Code))
> 202:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:325)
> 203:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:110)
> 204:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:132)
> 205:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:537)
> 206:  at 
> org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:208)
> 207:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor$StartRecursiveInvoke.invoke(ProxyMethodInterceptor.java
> :365)
> 208:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor.intercept(ProxyMethodInterceptor.java:96)
> 209:  at 
> org.activemq.gbean.ActiveMQConnector$$EnhancerByCGLIB$$2b4c85ae.startRecursive()
> 210:  at 
> org.apache.geronimo.console.jmsmanager.server.JMSConnectorPortlet.processAction(JMSConnectorPortlet.java:79)
> 211:  at 
> org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:229)
> 212:  at org.apache.pluto.core.PortletServlet.doGet(PortletServlet.java:158)
> 213:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:595)
> 214:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:688)
> 215:  at org.apache.pluto.core.PortletServlet.service(PortletServlet.java:153)
> 216:  at 
> org.apache.catalina.core.Appli

[jira] Closed: (GERONIMO-1786) JMS Listeners for protocols activeio, peer and openwire fail to start

2007-07-30 Thread Donald Woods (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-1786?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods closed GERONIMO-1786.
--

   Resolution: Fixed
Fix Version/s: 2.1
   1.2

Merged in 1.2 fix from GERONIMO-2637 into 2.0 and 2.1.

> JMS Listeners for protocols activeio, peer and openwire fail to start
> -
>
> Key: GERONIMO-1786
> URL: https://issues.apache.org/jira/browse/GERONIMO-1786
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: ActiveMQ, console
>Affects Versions: 1.0, 1.1, 1.1.1, 1.2
> Environment: Geronimo 1.0.0
>Reporter: Donald Woods
>Assignee: Donald Woods
> Fix For: 1.2, 2.0, 2.1
>
>
> Even though addition of JMS Listeners for activeio, peer and openwire 
> protocols is successful, the listeners fail to
> start with the following exceptions.
> activeio --- java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> openwire --- javax.jms.JMSException: Could not load protocol: openwire. 
> Reason: java.lang.ClassNotFoundException:
> org.activemq.transport.openwire.OpenWireTransportServerChannelFactory
> peer --- javax.jms.JMSException: Could not load protocol: peer. Reason: 
> java.lang.ClassNotFoundException:
> org.activemq.transport.peer.PeerTransportServerChannelFactory
> Stack trace of the same.
> 192: 14:17:49,499 ERROR [GBeanInstanceState] Error while starting; GBean is 
> now in the FAILED state:
> objectName="geronimo.server:J2EEApplication=null,J2EEModule=geronimo/activemq-broker/1.0/car,J2EEServer=geronimo,br
> oker=ActiveMQ,j2eeType=JMSConnector,name=ActiveMQ.activeio.0.0.0.0.12340-aio"
> 193: java.lang.NoSuchMethodError: org.activeio.ChannelFactory: method
> bindAsynchChannel(Ljava/net/URI;)Lorg/activeio/AsynchChannelServer; not found
> 194:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.createAsynchChannelServer(ActiveIOTranspo
> rtServerChannelFactory.java:60)
> 195:  at 
> org.activemq.transport.activeio.ActiveIOTransportServerChannelFactory.create(ActiveIOTransportServerChannelFact
> ory.java:49)
> 196:  at 
> org.activemq.transport.TransportServerChannelProvider.create(TransportServerChannelProvider.java:45)
> 197:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.createTransportServerChannel(BrokerConnectorImpl.java:425)
> 198:  at 
> org.activemq.broker.impl.BrokerConnectorImpl.(BrokerConnectorImpl.java:69)
> 199:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.createBrokerConnector(ActiveMQConnectorGBean.java:161)
> 200:  at 
> org.activemq.gbean.ActiveMQConnectorGBean.doStart(ActiveMQConnectorGBean.java:129)
> 201:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java(Compiled
>  Code))
> 202:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:325)
> 203:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:110)
> 204:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:132)
> 205:  at 
> org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:537)
> 206:  at 
> org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:208)
> 207:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor$StartRecursiveInvoke.invoke(ProxyMethodInterceptor.java
> :365)
> 208:  at 
> org.apache.geronimo.kernel.basic.ProxyMethodInterceptor.intercept(ProxyMethodInterceptor.java:96)
> 209:  at 
> org.activemq.gbean.ActiveMQConnector$$EnhancerByCGLIB$$2b4c85ae.startRecursive()
> 210:  at 
> org.apache.geronimo.console.jmsmanager.server.JMSConnectorPortlet.processAction(JMSConnectorPortlet.java:79)
> 211:  at 
> org.apache.pluto.core.PortletServlet.dispatch(PortletServlet.java:229)
> 212:  at org.apache.pluto.core.PortletServlet.doGet(PortletServlet.java:158)
> 213:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:595)
> 214:  at javax.servlet.http.HttpServlet.service(HttpServlet.java:688)
> 215:  at org.apache.pluto.core.PortletServlet.service(PortletServlet.java:153)
> 216:  at 
> org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252)
> 217:  at 
> org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
> 218:  at 
> org.apache.catalina.core.ApplicationDispatcher.invoke(ApplicationDispatcher.java:672)
> 219:  at 
> org.apache.catalina.core.ApplicationDispatcher.doInclude(ApplicationDispatcher.java:574)
> 220:  at 
> org.apache.catalina.core.ApplicationDispatcher.include(ApplicationDispatcher.java:499)
> 221:  at 
> org.apache.pluto.invoker.impl.PortletI

[jira] Commented: (SM-1001) FtpReceiverEndpoint (InternalEndpoint)

2007-07-30 Thread Guillaume Nodet (JIRA)

[ 
https://issues.apache.org/activemq/browse/SM-1001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39796
 ] 

Guillaume Nodet commented on SM-1001:
-

I think there is one big problem: the message contains some  informations used 
to configure the endpoint when the message is received.
This means that the endpoint is not thread safe.  Endpoints have to be, as they 
are given exchanges concurrently.

Another thing is that I' d really like to have a real WSDL to describe the 
interface for this endpoint, the operations and their parameters.

> FtpReceiverEndpoint (InternalEndpoint) 
> ---
>
> Key: SM-1001
> URL: https://issues.apache.org/activemq/browse/SM-1001
> Project: ServiceMix
>  Issue Type: Improvement
>Reporter: Rabi
> Attachments: patch.txt
>
>
> FtpReceiverEndpoint (InternalEndpoint) 
> - Can accpet in-out mep for listing files
> - Supports InOnly mep for get/mget operations...
> Usage:
> xbean configuration:
> --
> http://servicemix.apache.org/ftp/1.0";
>xmlns:b="http://servicemix.apache.org/samples/ftp-receiver";>
> endpoint="endpoint"
>   uri="ftp://rabi:[EMAIL PROTECTED]">   
>   
>   
>   
>   
>
> Input Message structure
> -
> 
> http://servicemix.apache.org/ftp/1.0"; operation="mget" 
> recursive="true" filter="*" relativeUri="/inbox">
> 
> Output for file listing
> 
> STATUS: 200
> 
> http://servicemix.apache.org/ftp/1.0";>
>  
> /http/inbox/data/inner/tx.xml
>  
>  /http/inbox/data/servicemix.xml
> 
> This is a very simple version. Please suggest on the improvements..

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (SM-1005) File BC doesnt set correlation Id and sender property

2007-07-30 Thread Guillaume Nodet (JIRA)

 [ 
https://issues.apache.org/activemq/browse/SM-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Guillaume Nodet closed SM-1005.
---

Resolution: Cannot Reproduce

> File BC doesnt set correlation Id and sender property
> -
>
> Key: SM-1005
> URL: https://issues.apache.org/activemq/browse/SM-1005
> Project: ServiceMix
>  Issue Type: Bug
>  Components: servicemix-file
>Affects Versions: 3.1, 3.1.1
>Reporter: Gianfranco Boccalon
>
> The File binding component doesn't set the properties correlation id and 
> sender.
> We fixed the problem changing the method processFile in class 
> org.apache.servicemix.file.FilePollerEndpoint, adding these rows before 
> sending the message (sendSync):
>exchange.setProperty(JbiConstants.SENDER_ENDPOINT, service.toString());
>exchange.setProperty(JbiConstants.CORRELATION_ID, 
> exchange.getExchangeId());

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: What to do with legacy EJB 2.1 code in DayTrader?

2007-07-30 Thread Christopher Blythe
I like having the 2.1 code around as well for the same reasons you stated. I
just don't like the idea of having the two implementations inter-mingled in
the same ear/jar. It makes DayTrader's usefulness as a code sample for
developers a lot harder to swallow. I also have a hard time believing that
there are going to be a lot of production applications out there that use
both EJB 2.1 and 3.0 components in the same package.

So, my original thinking was to, use DT 1.2 as the J2EE 1.4 based sample and
DT 2.0 as the EE 5 based sample.

Another option I just thought of is to refactor the packaging such that two
ejb jar files can be created, one for the EJB 2.1 legacy code and another
for the new EJB 3.0 components. The pom files could then be modified to
create two ear files.

Thoughts?

On 7/29/07, Kevan Miller <[EMAIL PROTECTED]> wrote:
>
>
> On Jul 27, 2007, at 3:31 PM, Dain Sundstrom wrote:
>
> > I'd like to see the 2.1 code kept around so we can compare base EJB
> > performance against other servers.  There is going to be legacy
> > code for a long time and this tool is our only way to see how
> > legacy code performs on our server.
> >
> > -dain
> >
> > On Jul 25, 2007, at 9:56 AM, Christopher Blythe wrote:
> >
> >> All,
> >>
> >> Given Geronimo 2.0 and DayTrader 2.0's focus on Java EE 5, I was
> >> wondering if it made sense to remove the old EJB 2.1 code? To be
> >> quite honest, I am torn. One one side, it would be nice to have
> >> both the EJB 2.1 and 3.0 impls at the same time for comparison
> >> purposes. However, keeping the old stuff around seems to hide the
> >> fact that 3.0 is supposed to be easier to work with and develop.
> >>
> >> Here are some options along with my own arguments for each...
> >>
> >> 1) Remove the old EJB 2.1 modes and make DayTrader 2.0 EJB 3 only
> >> - highlights the advantages of EJB 3.0 (less DDs, etc.)
> >> - makes the packaging and various runtime modes less confusing
> >> - can use the DayTrader 1.2 code for comparisons between EJB
> >> 2.1 and 3.0
> >> - EJB 2.1 mode never worked under load to begin with due to
> >> consistency issues
> >>
> >> 2) Leave 2.1 code in there for now and phase out in a DayTrader 2.X
> >> - comparisons can be done using a single ear
> >> - DT 2.x could be spun up immediately
> >>
> >> Now that I think about it, I think I'm swaying more towards option
> >> 1. However, given the time constraints to get 2.0 out  the door,
> >> I'm not sure if 1 is realistic.
>
> I like having 2.1 code around, also. I was just using it to identify
> JPA/CMP/Entity problems, yesterday...
>
> One of the strengths of DayTrader is the breadth of technologies that
> it can drive. IMO, it's not intended to be an exemplar of how simple
> it is to write a Java EE 5 applications... I'd be hesitant to lose
> the flexibility that Daytrader gives us. If this flexibility is
> hurting our ability to gather valid performance results or cannot be
> reasonably maintained, then I'm all ears.
>
> --kevan
>



-- 
"I say never be complete, I say stop being perfect, I say let... lets
evolve, let the chips fall where they may." - Tyler Durden


Re: [DISCUSS] Geronimo-Tuscany integration(Sending to both lists)

2007-07-30 Thread Vamsavardhana Reddy
The stuff in sandbox covers points 1 and 2 below to some extent.  Apart from
this, we have a GeronimoServletHost implementation so that Geronimo's tomcat
server is used when a service is exposed as webservice.  We have got the
calculator, simple-bigbank and helloworld-ws-service samples  from tuscany
samples working in Geronimo.  More details inline.


On 7/27/07, David Jencks <[EMAIL PROTECTED]> wrote:
>
> I had a long discussion with js delphino at oscon today and I'd like
> to write down some of what we talked about before I forget it all :-)
> I haven't looked at the sandbox work so I don't really know how it
> relates to what we were discussing.
>
> Here's what I would recommend to start integrating tuscany and
> provide sca "wrappers" around ejbs:
>
> 1. write a gbean that wraps and starts the tuscany runtime "server"
> that registers the components and composites and hooks up the
> wiring.  Deploy this in a service configuration together with the
> tuscany jar(s).

We have an EmbeddedSCADomainGBean that wraps an EmbeddedSCADomain tuscany
class.  We have a TuscanyBuilder to deploy standalone tuscany jars into
Geronimo.  Both these GBeans are part of a configuration that runs as a
"tuscany-plugin" for Geronimo.


2. write a gbean that represents a bunch of pojo components in a jar
> together with a sca "plan".  The code should be in the previous
> config.  The gbean has a reference to the tuscany runtime service and
> knows where the jar is, maybe where the sca plan(s) are etc etc.
> When it starts, it tells the sca runtime gbean to activate and start
> the sca stuff described in the jar (+ plans)


The TuscanyBuilder GBean, as part of deploying a tuscany jar, creates a
configuration and adds an "EmbeddedRuntimeGBean" to the configuration and
copies the jar to repository.  When the configuration is started, the
EmbeddedRuntimeGBean computes the contribution from the tuscany jar, adds
the contribution to EmbeddedSCADomain wrapped by the EmbeddedSCADomainGBean
and starts the components.  When the configuration is stopped, it stops the
components and removes the contribution from EmbeddedSCADomain.  Apart from
this the EmbeddedRuntimeGBean is starting all components (components from
other tuscany jars as well) as a work around for some problems we have come
across in EmbeddedSCADomain class.


3. write a gbean that exposes ejbs to the tuscany wiring mechanism.
> The idea here is that you can add this gbean to a geronimo plan for
> an ejb module, and when it starts it will call the tuscany runtime
> gbean with "tuscany wire ends" that hook up to the ejbs in the ejb
> module.  The tuscany wiring framework can then hook up the
> appropriate wires to the ejbs.
>
> This more or less handles the "in" end of exposing an ejb as a sca
> component.
>
> There's also the "out" end of a sca component.  This is relevant to
> servlets and ejbs that may want to call an sca component.  It seems
> like the easiest way to model this in javaee is probably to model
> this end of the sca wire as an injected ejb3 stateless session bean
> instance.  This may not handle "conversational" scope, but perhaps
> this can be hidden inside the sca wire.
>
>
> I think this should let us demonstrate some tuscany/sca functionality
> pretty quickly without spending a lot of time writing deployers, by
> letting humans be the deployers (adding the gbeans by hand to
> geronimo plans).  Later on we may want ModuleBuilderExtensions that
> can add these gbeans to for instance ejb apps automatically when they
> recognize sca plans.  Also I think we will need some kind of
> TuscanyConfigurationBuilder that can deploy an entire contribution at
> once.  I think getting stuff to run with "manual deployment" first
> would be a good idea.



Hopefully I will be able to find out how this relates to what is in
> the sandbox soon :-)



thanks
> david jencks
>
> On Jul 26, 2007, at 5:46 AM, Vamsavardhana Reddy wrote:
>
> > Hi,
> >
> > We have made some progress.  See the details in the wiki page
> > http://cwiki.apache.org/confluence/display/TUSCANYWIKI/Tuscany
> > +Geronimo+Integrationunder
> > Current status.  Code is in Geronimo Sandbox.
> >
> > Thanks and regards,
> > Vamsi
> >
> > On 7/7/07, Raymond Feng <[EMAIL PROTECTED]> wrote:
> >>
> >> Hi,
> >>
> >> I created an empty WIKI page @
> >>
> >> http://cwiki.apache.org/confluence/display/TUSCANYWIKI/Tuscany
> >> +Geronimo+Integration
> >> .
> >> We should try to capture the key points for the discussions.
> >>
> >> Thanks,
> >> Raymond
> >>
> >> - Original Message -
> >> From: "Manu George" <[EMAIL PROTECTED]>
> >> To: <[EMAIL PROTECTED]>
> >> Sent: Friday, July 06, 2007 2:42 PM
> >> Subject: Re: [DISCUSS] Geronimo-Tuscany integration(Sending to
> >> both lists)
> >>
> >>
> >>> Hi Simon,
> >>>Comments inline.
> >>>
> >>> On 7/5/07, Simon Laws <[EMAIL PROTECTED]> wrote:
>  Hi Manu
> 
>  more comments in line
> 
>  On 7/4/07, Manu George <[EMAIL PROTEC

[jira] Commented: (GERONIMO-2878) JVM stats exposed through JMX are incorrect and missing init/max heap size

2007-07-30 Thread Donald Woods (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-2878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516410
 ] 

Donald Woods commented on GERONIMO-2878:


>From http://java.sun.com/j2se/1.5.0/docs/guide/management/jconsole.html - 
>Uptime: Total amount of time since the JVM was started.
A "Time" attribute would be the current OS time since 1970...
Since we now require Java 5 or later, I'm going to use the new in VM 
ManagementFactory to get the actual JVM provided mxbean for the correct results.

> JVM stats exposed through JMX are incorrect and missing init/max heap size
> --
>
> Key: GERONIMO-2878
> URL: https://issues.apache.org/jira/browse/GERONIMO-2878
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: management
>Affects Versions: 2.0-M5
> Environment: WinXP, Java 1.5.0_11, JMX Viewer
>Reporter: Donald Woods
>Assignee: Donald Woods
> Fix For: 2.0
>
>
> The statistics for geronimo:J2EEServer=geronimo,name=JVM,j2eeType=JVM are:
> 1) incorrect for JVM Up Time, which shows 1171999222859 msecs. after only 
> about 10 mins.
> 2) does not provide init or max Heap size values for health monitoring - 
> http://java.sun.com/j2se/1.5.0/docs/api/java/lang/management/MemoryUsage.html
> 3) does not provide all of the JVM attributes, as the JVM portlet does

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Resolved: (SM-1012) Possible resource leak in FilePoller

2007-07-30 Thread Guillaume Nodet (JIRA)

 [ 
https://issues.apache.org/activemq/browse/SM-1012?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Guillaume Nodet resolved SM-1012.
-

   Resolution: Fixed
Fix Version/s: 3.2
   3.1.2
 Assignee: Guillaume Nodet

URL: http://svn.apache.org/viewvc?view=rev&rev=561009

> Possible resource leak in FilePoller
> 
>
> Key: SM-1012
> URL: https://issues.apache.org/activemq/browse/SM-1012
> Project: ServiceMix
>  Issue Type: Bug
>  Components: servicemix-file
>Affects Versions: 3.1.1
> Environment: Windows XP, JDK 5.0
>Reporter: Artur Karazniewicz
>Assignee: Guillaume Nodet
>Priority: Minor
> Fix For: 3.1.2, 3.2
>
>
> In the org.apache.servicemix.components.file.FilePoller#processFile(File) it 
> is possible that allocated file would leak. Now, it goes like:
>   protected void processFile(File aFile) throws Exception {
> String name = aFile.getCanonicalPath();
> InputStream in = new BufferedInputStream(new FileInputStream(aFile));
> InOnly exchange = getExchangeFactory().createInOnlyExchange();
> NormalizedMessage message = exchange.createMessage();
> exchange.setInMessage(message);
> marshaler.readMessage(exchange, message, in, name);
> getDeliveryChannel().sendSync(exchange);
> in.close();
> }
> But, we should properly clean-up in the case of exception thrown before 
> in.close(). Thus, should be:
>   protected void processFile(File aFile) throws Exception {
>   InputStream in = null;
>   try {
>   String name = aFile.getCanonicalPath();
>   in = new BufferedInputStream(new 
> FileInputStream(aFile));
>   InOnly exchange = 
> getExchangeFactory().createInOnlyExchange();
>   NormalizedMessage message = exchange.createMessage();
>   exchange.setInMessage(message);
>   marshaler.readMessage(exchange, message, in, name);
>   getDeliveryChannel().sendSync(exchange);
>   } 
>   finally {
>   if (in != null) {
>   in.close();
>   }
>   }
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: [DISCUSS] Adding a "Tech Preview" portlet group in Admin Console in G 2.0

2007-07-30 Thread Shiva Kumar H R
On 7/30/07, Kevan Miller <[EMAIL PROTECTED]> wrote:
>
>
> On Jul 30, 2007, at 2:04 AM, Vamsavardhana Reddy wrote:
>
>
>
> On 7/30/07, Vamsavardhana Reddy <[EMAIL PROTECTED]> wrote:
> >
> >
> >
> > On 7/30/07, Kevan Miller < [EMAIL PROTECTED]> wrote:
> > >
> > >
> > > On Jul 28, 2007, at 1:41 AM, Vamsavardhana Reddy wrote:
> > >
> > >
> > >
> > > On 7/28/07, Kevan Miller <[EMAIL PROTECTED]> wrote:
> > > >
> > > >
> > > > On Jul 26, 2007, at 1:21 AM, Vamsavardhana Reddy wrote:
> > > >
> > > > How about adding a "Tech Preview" portlet group in Admin Console in
> > > > G 2.0 where we can include the portlets (for e.g., the "Create Plan"
> > > > portlet Shiva is working on.  See 
> > > > https://issues.apache.org/jira/browse/GERONIMO-3254
> > > > ) for which user feedback will play a great role in improving those
> > > > further.  Once we decide on whether to retain those or not, we can 
> > > > either
> > > > move them out to a different group or remove them from Admin Console
> > > > accordingly.
> > > >
> > > >
> > > > Some of the new pluggable console support would have helped, here.
> > > > But that's not going into 2.0. Personally, sounds like this should
> > > > go into trunk, if it's not ready... Can still get good feedback...
> > > >
> > > > --kevan
> > > >
> > >
> > >
> > > The idea is that if these portlets go in as part of a release with
> > > binaries readily available for download, we have a better chance of 
> > > getting
> > > more users to try the portlets and possibly provide some feedback too in 
> > > the
> > > linked wiki pages.  If it goes only into trunk, the users will still have 
> > > to
> > > build the binaries from trunk, which they may not want to/may not succeed
> > > due to some reason or the other.
> > >
> > >
> > > Yes, I understood the idea.
> > >
> > > If a committer thinks that this code is ready and wants to commit it,
> > > then why isn't it in trunk? If it was in trunk, then we could evaluate and
> > > discuss getting it into branches/2.0. I'm currently doubtful that it 
> > > should
> > > go into 2.0...
> > >
> >
> > I am not referring to code that may be delivering 100% of what it is
> > expected to deliver.
> >
>
> I thought an example may help here.  The plan creator portlet that Shiva
> is working on, currently handles WAR files.  The fully functional one is
> expected to handle  WAR, EAR, EJB JAR,  etc.
>
>
> A WAR plan creator sounds like useful capability in its own right. I don't
> see any reason why it couldn't be provided stand alone. If this capability
> was committed in trunk, provided useful capability, and had low risk for
> causing other problems, I would probably be in favor of putting it in the
> 2.0 release. I would not tag it with a "Tech Preview" label.
>
> Is a WAR capability the function you'd like to put into 2.0? I'll repeat
> -- you're best starting point is to get the function into trunk...
>
> This discussion has gotten me wondering about plan generation wizards, the
> j2g migration tool, and IDE tooling. Perhaps we should be thinking more
> holistically about these functions, rather than as disparate capabilities...
>

Thanks for raising this Kevan. The idea of "Plan Generation Wizard"
basically started off as a new feature proposal to Geronimo Eclipse Plug-in.
Later feedback from user mailing list suggested that such a tool would fit
best within "Admin Console" in close proximity with "Deploy New" tool. Some
of this discussion can be found here:
http://www.mail-archive.com/dev@geronimo.apache.org/msg46831.html

Hence the current focus is on getting this feature implemented as a new
Portlet in Admin Console. This work will later be incorporated into Geronimo
Eclipse Plug-in also to simplify developmental activities. Admin Console
portlet is targeted to simplify the job of server administrators.

Looks like there will be a overlap of the functionality of J2G migration
tool and Plan Generation Wizards. Haven't looked into the details of J2G
migration tool, but my initial understanding is that in addition to
auto-transforming J-specific deployment plans into Geronimo deployment
plans, it also auto-maps some J-specific security classes into the ones
supported by Geronimo. So if a user already has a J-specific plan then "J2G
migration tool" is the tool for him, else he can start looking into Plan
Generation Wizards.

And yes, sharing of code for the overlapping functionality between J2G
migration tool and plan generation wizards is something to be looked into.

Comments welcome.

- Shiva

--kevan
>


[jira] Commented: (GERONIMO-3359) naming builders are invoked twice

2007-07-30 Thread David Jencks (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516425
 ] 

David Jencks commented on GERONIMO-3359:


It's more or less by design although we might be able to find a way to change 
the architecture to eliminate duplication of actually constructing the 
references.   Each builder (web module builder, MyFacesMBE, JspMBE) are looking 
at different annotations and adding xml to the 
gradually-getting-more-metadata-complete spec dd so they call the 
NamingBuilders to add the xml and turn it into references.

I think to fix this we'd have to add another phase to the NamingBuilder 
interface to process annotations, and call that one from each MBE and call the 
buildNaming method only from the ModuleBuilder.

> naming builders are invoked twice
> -
>
> Key: GERONIMO-3359
> URL: https://issues.apache.org/jira/browse/GERONIMO-3359
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>Affects Versions: 2.0
>Reporter: Jarek Gawor
>
> I have a simple web.xml with one service-ref entry. However, I'm seeing that 
> the code that creates the service ref is called twice with the same 
> information. 
> From the configuration and the code it looks like 
> MyFacesModuleBuilderExtension.addGBeans() and 
> JspModuleBuilderExtension.addGBeans() methods call 
> namingBuilders.buildNaming(webApp, jettyWebApp, webModule, buildingContext);. 
> But for web applications, 
> AbstractWebModuleBuilder.configureBasicWebModuleAttributes() already does 
> that. 
> Not sure if this is by design or a mistake.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: KEYS ?

2007-07-30 Thread Dain Sundstrom
I'm not sure, but it may be mentioned in old binaries as the location  
for verifying the signatures.  I could be completely wrong... I just  
have a memory of some external references.  If you do move the file,  
I'd definitely search the website for references.


-dain

On Jul 27, 2007, at 12:35 PM, Jason Dillon wrote:


What external references are you referring to?

I believe that it does have symlink support, though I hardly ever  
use it.


--jason


On Jul 27, 2007, at 12:29 PM, Dain Sundstrom wrote:

I think there are external references to this file, but I'm not  
sure how we should deal with them.  Does svn have symlink support?


-dain

On Jul 26, 2007, at 3:52 PM, Jason Dillon wrote:

Should keys move up to a peer to the STATUS file too?  Seems  
these are project global, not specific to the server bits...


--jason








Re: [DISCUSS] Adding a "Tech Preview" portlet group in Admin Console in G 2.0

2007-07-30 Thread Paul McMahan

On Jul 30, 2007, at 10:18 AM, Kevan Miller wrote:

A WAR plan creator sounds like useful capability in its own right.  
I don't see any reason why it couldn't be provided stand alone. If  
this capability was committed in trunk, provided useful capability,  
and had low risk for causing other problems, I would probably be in  
favor of putting it in the 2.0 release. I would not tag it with a  
"Tech Preview" label.


Is a WAR capability the function you'd like to put into 2.0? I'll  
repeat -- you're best starting point is to get the function into  
trunk...


The new plan generator portlet sounds really interesting!  I tend to  
agree with Kevan that instead of thinking about 2.0 right now it  
should go into trunk first and without a "Tech Preview" label.  To me  
trunk is by definition the place for incomplete works in progress  
that we want to get into users' hands for feedback.  Or sandbox.


Best wishes,
Paul


[jira] Commented: (GERONIMO-3348) java.lang.NoSuchMethodError in org.springframework.context.i18n.LocaleContextHolder

2007-07-30 Thread Aleksandr Tarutin (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516450
 ] 

Aleksandr Tarutin commented on GERONIMO-3348:
-

Shouldn't a container like Geronimo filter out everything except for the 
classes that implement the specifications?

> java.lang.NoSuchMethodError in 
> org.springframework.context.i18n.LocaleContextHolder
> ---
>
> Key: GERONIMO-3348
> URL: https://issues.apache.org/jira/browse/GERONIMO-3348
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>Affects Versions: 2.0-M6
> Environment: 2.6.18-gentoo-r2 #1 Sat Nov 11 03:36:37 EST 2006 i686 
> Pentium III (Katmai) GenuineIntel GNU/Linux
> JDK-1.5.0.12
>Reporter: Aleksandr Tarutin
>
> When deploying and running [Proximity|http://proximity.abstracthorizon.org/] 
> it works without any problem in geronimo-1.1.1. But when the same application 
> is deployed to 2.0-M6 following exception is thrown:
> {noformat}
> 15:57:53,267 INFO  [DirectoryHotDeployer] Deploying proximity
> 15:57:56,690 WARN  [JettyModuleBuilder] Web application . does not contain a 
> WEB-INF/geronimo-web.xml deployment plan.  This may or may not be a problem, 
> depending on whether you have things like resource references that need to be 
> resolved.  You can also give the deployer a separate deployment plan file on 
> the command line.
> 15:58:04,709 WARN  [JspModuleBuilderExtension] Invalid transformed taglib
> org.apache.xmlbeans.XmlException: Invalid deployment descriptor: errors:
> jar:file:/opt/geronimo-jetty6-jee5-2.0-M6/repository/default/proximity/1185307073730/proximity-1185307073730.war/WEB-INF/lib/bsf-2.3.0.jar!/META-INF/taglib.tld:15:3:
>  error: cvc-datatype-valid.1.1: string value 'BSF JSP Support' does not match 
> pattern for tld-canonical-nameType in namespace 
> http://java.sun.com/xml/ns/javaee
> Descriptor:
> 
> http://java.sun.com/xml/ns/javaee  
> http://java.sun.com/xml/ns/javaee/web-jsptaglibrary_2_1.xsd"; version="2.1" 
> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; 
> xmlns="http://java.sun.com/xml/ns/javaee";>
>   

[jira] Commented: (SM-939) CXF based Service Engine and Bnding Component

2007-07-30 Thread Guillaume Nodet (JIRA)

[ 
https://issues.apache.org/activemq/browse/SM-939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39811
 ] 

Guillaume Nodet commented on SM-939:


I've modified and applied the patch :-)
Thanks !

> CXF based Service Engine and Bnding Component
> -
>
> Key: SM-939
> URL: https://issues.apache.org/activemq/browse/SM-939
> Project: ServiceMix
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Freeman Fang
>Priority: Critical
> Fix For: 3.2
>
> Attachments: patch.txt, patch0627.txt, patch0702.txt, patch0720.txt
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (GERONIMO-3347) Improve output formatting in J2G Conversion tool

2007-07-30 Thread Jason Warner (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3347?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jason Warner closed GERONIMO-3347.
--

Resolution: Fixed

> Improve output formatting in J2G Conversion tool
> 
>
> Key: GERONIMO-3347
> URL: https://issues.apache.org/jira/browse/GERONIMO-3347
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>  Components: J2G
>Reporter: Jason Warner
>Assignee: Jason Warner
> Attachments: Geronimo-3347.patch
>
>
> The current output for any of the subtools for the J2G Conversion Tool is 
> sometimes vague and confusing.  The formatting could also be improved to make 
> it easier to read.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-3363) ArrayList thread safe problem in OpenJPA

2007-07-30 Thread Prasad Kashyap (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Prasad Kashyap updated GERONIMO-3363:
-

Fix Version/s: (was: 2.0)
   2.0.x

> ArrayList thread safe problem in OpenJPA
> 
>
> Key: GERONIMO-3363
> URL: https://issues.apache.org/jira/browse/GERONIMO-3363
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: persistence
>Affects Versions: 2.0-M7
>Reporter: YunFeng Ma
>Priority: Blocker
> Fix For: 2.0.x
>
>
> When running stress testing using DayTrader with JPA mode, got a lot of 
> ArrayList thread safe problem.
> The thread safe problem happened in  
> org.apache.openjpa.kernel.AbstractBrokerFactory.syncWithManagedTransaction, 
> but this method has the following comments:
> // we don't need to synchronize on brokers or guard against 
> multiple
> // threads using the same trans since one JTA transaction can 
> never
> // be active on multiple concurrent threads.
> Collection brokers = (Collection) _transactional.get(trans);
> if (brokers == null) {
> brokers = new ArrayList(2);
> _transactional.put(trans, brokers);
> trans.registerSynchronization(new 
> RemoveTransactionSync(trans));
> }
> brokers.add(broker);
> Does this mean that it's Geronimo which causes the thread safe problem?
> The exception stack:
> java.rmi.RemoteException: The bean encountered a non-application exception.; 
> nested exception is:
> javax.ejb.EJBException: TradeBean.getClosedOrders - error
> at 
> org.apache.openejb.core.transaction.TransactionPolicy.throwExceptionToServer(TransactionPolicy.java:211)
> at 
> org.apache.openejb.core.transaction.TxRequired.handleSystemException(TxRequired.java:106)
> at 
> org.apache.openejb.core.stateless.StatelessContainer._invoke(StatelessContainer.java:210)
> at 
> org.apache.openejb.core.stateless.StatelessContainer.invoke(StatelessContainer.java:165)
> at 
> org.apache.openejb.core.ivm.EjbObjectProxyHandler.businessMethod(EjbObjectProxyHandler.java:211)
> at 
> org.apache.openejb.core.ivm.EjbObjectProxyHandler._invoke(EjbObjectProxyHandler.java:65)
> at 
> org.apache.openejb.core.ivm.BaseEjbProxyHandler.invoke(BaseEjbProxyHandler.java:232)
> at 
> org.apache.openejb.util.proxy.Jdk13InvocationHandler.invoke(Jdk13InvocationHandler.java:49)
> at $Proxy109.getClosedOrders(Unknown Source)
> at 
> org.apache.geronimo.samples.daytrader.TradeAction.getClosedOrders(TradeAction.java:276)
> at 
> org.apache.geronimo.samples.daytrader.web.OrdersAlertFilter.doFilter(OrdersAlertFilter.java:76)
> at 
> org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
> at 
> org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
> at 
> org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:230)
> at 
> org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
> at 
> org.apache.geronimo.tomcat.valve.DefaultSubjectValve.invoke(DefaultSubjectValve.java:56)
> at 
> org.apache.geronimo.tomcat.GeronimoStandardContext$SystemMethodValve.invoke(GeronimoStandardContext.java:351)
> at 
> org.apache.geronimo.tomcat.valve.GeronimoBeforeAfterValve.invoke(GeronimoBeforeAfterValve.java:47)
> at 
> org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:128)
> at 
> org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:104)
> at 
> org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
> at 
> org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:563)
> at 
> org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:261)
> at 
> org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:844)
> at 
> org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:581)
> at 
> org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:447)
> at java.lang.Thread.run(Thread.java:801)
> Caused by: javax.ejb.EJBException: TradeBean.getClosedOrders - error
> at 
> org.apache.geronimo.samples.daytrader.TradeJPA.getClosedOrders(TradeJPA.java:491)
> at sun.reflect.GeneratedMethodAccessor324.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:615)
> at 
> org.a

[jira] Updated: (GERONIMO-1829) Service Plans should allow GBean references by interface (vs. by name)

2007-07-30 Thread Prasad Kashyap (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-1829?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Prasad Kashyap updated GERONIMO-1829:
-

Fix Version/s: (was: 2.0)
   2.0.x

> Service Plans should allow GBean references by interface (vs. by name)
> --
>
> Key: GERONIMO-1829
> URL: https://issues.apache.org/jira/browse/GERONIMO-1829
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>  Components: kernel
>Affects Versions: 1.0
>Reporter: Aaron Mulder
> Fix For: 2.0.x
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-2128) Allow user to specify the Isolation Level for a CMP bean's SQL access

2007-07-30 Thread Prasad Kashyap (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-2128?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Prasad Kashyap updated GERONIMO-2128:
-

Fix Version/s: (was: 2.0)
   2.0.x

> Allow user to specify the Isolation Level for a CMP bean's SQL access
> -
>
> Key: GERONIMO-2128
> URL: https://issues.apache.org/jira/browse/GERONIMO-2128
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>Reporter: Matt Hogstrom
> Fix For: 1.x, 2.0.x
>
>
> Currently CMP beans all execute in the default isolation level of the 
> datasource.  This means that in many situations higher level locks are 
> required for the database access than are required.  This improvement will 
> allow the user to specify a new attribute on a CMP entity bean 
>  that will be the isolation level used for database access 
> on this entity bean.  This will require updates to OpenEJB and TranQL.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-1807) Remove uses of ObjectName from core server

2007-07-30 Thread Prasad Kashyap (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-1807?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Prasad Kashyap updated GERONIMO-1807:
-

Fix Version/s: (was: 2.0)
   2.0.x

> Remove uses of ObjectName from core server
> --
>
> Key: GERONIMO-1807
> URL: https://issues.apache.org/jira/browse/GERONIMO-1807
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>  Components: kernel
>Reporter: Dain Sundstrom
> Fix For: 2.0.x
>
>
> Using the ObjectName class in the core server causes a dependency on the jmx 
> classes.  We are also in the process of moving away from these, but there are 
> still many places that use this class.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (GERONIMO-3270) Avoid CMP foreign key violations

2007-07-30 Thread Prasad Kashyap (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Prasad Kashyap updated GERONIMO-3270:
-

Fix Version/s: (was: 2.0)
   2.0.x

> Avoid CMP foreign key violations 
> -
>
> Key: GERONIMO-3270
> URL: https://issues.apache.org/jira/browse/GERONIMO-3270
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>Reporter: Kevan Miller
> Fix For: 2.0.x
>
>
> A recent OpenJPA change (see 
> https://issues.apache.org/jira/browse/OPENJPA-235) can cause foreign key 
> violations in CMP. May be that we (CMP) need to provide OpenJPA information 
> about foreign keys. We can, however, avoid the problem by specifying OpenJPA 
> to maintain operation-order. This is what I intend to do, in the short-term...

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: Claim your unassigned JIRAs targeted for 2.0

2007-07-30 Thread Prasad Kashyap
The following unassigned JIRAs have now been moved to the 2.0.x bucket.

GERONIMO-3363 
 ArrayList
thread safe problem in OpenJPA  YunFeng Ma  30/Jul/07

GERONIMO-1829   Service
Plans should allow GBean references by interface (vs. by name)  Aaron Mulder
12/Apr/06

GERONIMO-2128   Allow
user to specify the Isolation Level for a CMP bean's SQL access  Matt
Hogstrom 16/Jun/06

GERONIMO-1807   Remove
uses of ObjectName from core server  Dain Sundstrom  06/Apr/06

GERONIMO-3270   Avoid
CMP foreign key violations  Kevan  28/Jun/07

Cheers
Prasad

On 7/26/07, Prasad Kashyap <[EMAIL PROTECTED]> wrote:
> There are some 19 JIRAs in the 2.0 bucket (out of the present 57) that
> are unassigned. If you think they should really really go into 2.0,
> please find an owner for it.
>
> On Monday, 07/30 at 10 AM EST, I shall move the unassigned JIRAs to
> the 2.0.x bucket.
>
> Cheers
> Prasad
>


[jira] Closed: (GERONIMO-2878) JVM stats exposed through JMX are incorrect and missing init/max heap size

2007-07-30 Thread Donald Woods (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-2878?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods closed GERONIMO-2878.
--

   Resolution: Fixed
Fix Version/s: 2.1

Only updated the existing Uptime and Heap value calculations, as adding new 
Stats would break our compatibility with the JEE Management Spec 1.0.

Committed revision 561093 in branches/2.0
Committed revision 561094 in trunk (2.1)

Verified UpTime values are fixed and the reworked Heap values work correctly 
via JMX portlet -

 Name   Value  
JVM Heap Size:  Description: The memory usage of the JVM
Unit: BYTE
Start Time: Mon Jul 30 14:53:31 EDT 2007
Last Sample Time: Mon Jul 30 14:54:40 EDT 2007
Upper Bound: 532742144
Lower Bound: 0
High Water Mark: 27070456
Low Water Mark: 24755024
Current: 24755024
 
JVM Up Time:  Description: The length of time that the JVM has been running
Unit: MILLISECOND
Start Time: Mon Jul 30 14:53:31 EDT 2007
Last Sample Time: Mon Jul 30 14:54:40 EDT 2007
Count: 69500
 


> JVM stats exposed through JMX are incorrect and missing init/max heap size
> --
>
> Key: GERONIMO-2878
> URL: https://issues.apache.org/jira/browse/GERONIMO-2878
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: management
>Affects Versions: 2.0-M5
> Environment: WinXP, Java 1.5.0_11, JMX Viewer
>Reporter: Donald Woods
>Assignee: Donald Woods
> Fix For: 2.0, 2.1
>
>
> The statistics for geronimo:J2EEServer=geronimo,name=JVM,j2eeType=JVM are:
> 1) incorrect for JVM Up Time, which shows 1171999222859 msecs. after only 
> about 10 mins.
> 2) does not provide init or max Heap size values for health monitoring - 
> http://java.sun.com/j2se/1.5.0/docs/api/java/lang/management/MemoryUsage.html
> 3) does not provide all of the JVM attributes, as the JVM portlet does

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: svn commit: r561027 - in /geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector: BaseHttp11ConnectorGBean.java ConnectorGBean.java

2007-07-30 Thread Anita Kulshreshtha
   We could use that script to set svn:eol-style=native...

Thanks
Anita

--- [EMAIL PROTECTED] wrote:

> Author: jgenender
> Date: Mon Jul 30 09:06:27 2007
> New Revision: 561027
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=561027
> Log:
> Fix maxThreads issue
> 
> Modified:
>
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
>
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/ConnectorGBean.java
> 
> Modified:
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> URL:
>
http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java?view=diff&rev=561027&r1=561026&r2=561027
>
==
> ---
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> (original)
> +++
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> Mon Jul 30 09:06:27 2007
> @@ -320,7 +320,7 @@
>  
>  static {
>  GBeanInfoBuilder infoFactory =
> GBeanInfoBuilder.createStatic("Tomcat Connector",
> BaseHttp11ConnectorGBean.class, ConnectorGBean.GBEAN_INFO);
> -infoFactory.addInterface(Http11Protocol.class, 
> +infoFactory.addInterface(BaseHttp11Protocol.class, 
>  new String[] {
>  //HTTP Attributes
>  "acceptCount", 
> 
> Modified:
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/ConnectorGBean.java
> URL:
>
http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/ConnectorGBean.java?view=diff&rev=561027&r1=561026&r2=561027
>
==
> ---
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/ConnectorGBean.java
> (original)
> +++
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/ConnectorGBean.java
> Mon Jul 30 09:06:27 2007
> @@ -37,7 +37,7 @@
>  import org.apache.geronimo.tomcat.ObjectRetriever;
>  import org.apache.geronimo.tomcat.TomcatContainer;
>  
> -public abstract class ConnectorGBean extends BaseGBean implements
> GBeanLifecycle, ObjectRetriever, TomcatWebConnector {
> +public abstract class ConnectorGBean extends BaseGBean implements
> CommonProtocol, GBeanLifecycle, ObjectRetriever, TomcatWebConnector {
>  
>  private static final Log log =
> LogFactory.getLog(ConnectorGBean.class);
>  
> 
> 
> 



  

Shape Yahoo! in your own image.  Join our Network Research Panel today!   
http://surveylink.yahoo.com/gmrs/yahoo_panel_invite.asp?a=7 




Re: svn commit: r561104 - /geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

2007-07-30 Thread Anita Kulshreshtha
FYI: This is the relevant change:
+public boolean isStatisticsProvider() {
+return true;
+}

Thanks
Anita

--- [EMAIL PROTECTED] wrote:

> Author: akulshreshtha
> Date: Mon Jul 30 13:19:14 2007
> New Revision: 561104
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=561104
> Log:
>During Recent restructuring of tomcat connetors statisticsProvider
> attribute got lost. Reenable this attribute for HTTPConnectors
> 
> Modified:
>
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> 
> Modified:
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> URL:
>
http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java?view=diff&rev=561104&r1=561103&r2=561104
>
==
> ---
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> (original)
> +++
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> Mon Jul 30 13:19:14 2007
> @@ -1,409 +1,413 @@
> -/*
> - * Licensed to the Apache Software Foundation (ASF) under one
> - * or more contributor license agreements.  See the NOTICE file
> - * distributed with this work for additional information
> - * regarding copyright ownership.  The ASF licenses this file
> - * to you under the Apache License, Version 2.0 (the
> - * "License"); you may not use this file except in compliance
> - * with the License.  You may obtain a copy of the License at
> - *
> - *  http://www.apache.org/licenses/LICENSE-2.0
> - *
> - * Unless required by applicable law or agreed to in writing,
> - * software distributed under the License is distributed on an
> - * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> - * KIND, either express or implied.  See the License for the
> - * specific language governing permissions and limitations
> - * under the License.
> - */
> -package org.apache.geronimo.tomcat.connector;
> -
> -import java.net.InetAddress;
> -import java.net.InetSocketAddress;
> -import java.net.UnknownHostException;
> -import java.util.Map;
> -
> -import javax.management.j2ee.statistics.Stats;
> -import javax.net.ssl.KeyManagerFactory;
> -
> -import org.apache.geronimo.gbean.GBeanInfo;
> -import org.apache.geronimo.gbean.GBeanInfoBuilder;
> -import org.apache.geronimo.management.StatisticsProvider;
> -import org.apache.geronimo.system.serverinfo.ServerInfo;
> -import org.apache.geronimo.tomcat.TomcatContainer;
> -import org.apache.geronimo.tomcat.stats.ConnectorStats;
> -
> -public abstract class BaseHttp11ConnectorGBean extends
> ConnectorGBean implements BaseHttp11Protocol, StatisticsProvider {
> -
> -protected String connectHost;
> -
> -// JSR77 stats
> -private ConnectorStats connStatsProvider = new ConnectorStats();
> -
> -private boolean reset = true;
> -
> -public BaseHttp11ConnectorGBean(String name, Map initParams,
> String tomcatProtocol, String address, int port, TomcatContainer
> container, ServerInfo serverInfo) throws Exception {
> -super(name, initParams, tomcatProtocol, container,
> serverInfo);
> -
> -// Default the host to listen on all address is one was not
> specified
> -if (address == null) {
> -address = "0.0.0.0";
> -}
> -
> -// Must have a port
> -if (port == 0) {
> -throw new IllegalArgumentException("Must declare a
> port.");
> -}
> -
> -connector.setAttribute("address", address);
> -connector.setPort(port);
> -
> -}
> -
> -protected void initProtocol() {}
> -
> -public String getConnectUrl() {
> -if(connectHost == null) {
> -String host = getAddress();
> -if(host == null || host.equals("0.0.0.0") ||
> host.equals("0:0:0:0:0:0:0:1")) {
> -InetAddress address = null;
> -try {
> -address = InetAddress.getLocalHost();
> -} catch (UnknownHostException e) {
> -host = "unknown-host";
> -}
> -if(address != null) {
> -host = address.getCanonicalHostName();
> -if(host == null || host.equals("")) {
> -host = address.getHostAddress();
> -}
> -}
> -}
> -// this host address could be in IPv6 format, 
> -// which means we need to wrap it in brackets
> -if (host.indexOf(":") >= 0) {
> -host = "[" + host + "]"; 
> -}
> -connectHost = host;
> -}
> -return
> getScheme().toLowerCase()+"://"+

Re: svn commit: r561104 - /geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

2007-07-30 Thread Donald Woods
For branches/2.0, is there some reason the Tomcat stats are still disabled in 
TomcatContainer.java?


public boolean isStatisticsProvider() {
return false; // todo: return true once stats are integrated
}


-Donald

Anita Kulshreshtha wrote:

FYI: This is the relevant change:
+public boolean isStatisticsProvider() {
+return true;
+}

Thanks
Anita

--- [EMAIL PROTECTED] wrote:


Author: akulshreshtha
Date: Mon Jul 30 13:19:14 2007
New Revision: 561104

URL: http://svn.apache.org/viewvc?view=rev&rev=561104
Log:
   During Recent restructuring of tomcat connetors statisticsProvider
attribute got lost. Reenable this attribute for HTTPConnectors

Modified:
   


geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

Modified:


geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

URL:


http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java?view=diff&rev=561104&r1=561103&r2=561104
==

---


geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

(original)
+++


geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

Mon Jul 30 13:19:14 2007
@@ -1,409 +1,413 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *  http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.geronimo.tomcat.connector;
-
-import java.net.InetAddress;
-import java.net.InetSocketAddress;
-import java.net.UnknownHostException;
-import java.util.Map;
-
-import javax.management.j2ee.statistics.Stats;
-import javax.net.ssl.KeyManagerFactory;
-
-import org.apache.geronimo.gbean.GBeanInfo;
-import org.apache.geronimo.gbean.GBeanInfoBuilder;
-import org.apache.geronimo.management.StatisticsProvider;
-import org.apache.geronimo.system.serverinfo.ServerInfo;
-import org.apache.geronimo.tomcat.TomcatContainer;
-import org.apache.geronimo.tomcat.stats.ConnectorStats;
-
-public abstract class BaseHttp11ConnectorGBean extends
ConnectorGBean implements BaseHttp11Protocol, StatisticsProvider {
-
-protected String connectHost;
-
-// JSR77 stats

-private ConnectorStats connStatsProvider = new ConnectorStats();
-
-private boolean reset = true;
-
-public BaseHttp11ConnectorGBean(String name, Map initParams,
String tomcatProtocol, String address, int port, TomcatContainer
container, ServerInfo serverInfo) throws Exception {
-super(name, initParams, tomcatProtocol, container,
serverInfo);
-
-// Default the host to listen on all address is one was not
specified
-if (address == null) {
-address = "0.0.0.0";
-}
-
-// Must have a port
-if (port == 0) {
-throw new IllegalArgumentException("Must declare a
port.");
-}
-
-connector.setAttribute("address", address);
-connector.setPort(port);
-
-}
-
-protected void initProtocol() {}
-
-public String getConnectUrl() {

-if(connectHost == null) {
-String host = getAddress();
-if(host == null || host.equals("0.0.0.0") ||
host.equals("0:0:0:0:0:0:0:1")) {
-InetAddress address = null;
-try {
-address = InetAddress.getLocalHost();
-} catch (UnknownHostException e) {
-host = "unknown-host";
-}
-if(address != null) {
-host = address.getCanonicalHostName();
-if(host == null || host.equals("")) {
-host = address.getHostAddress();
-}
-}
-}
-// this host address could be in IPv6 format, 
-// which means we need to wrap it in brackets

-if (host.indexOf(":") >= 0) {
-host = "[" + host + "]"; 
-}

-connectHost = host;
-}
-retu

[jira] Assigned: (GERONIMO-3140) Artifact aliasing doesn't work for already-started modules

2007-07-30 Thread David Jencks (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Jencks reassigned GERONIMO-3140:
--

Assignee: David Jencks

> Artifact aliasing doesn't work for already-started modules
> --
>
> Key: GERONIMO-3140
> URL: https://issues.apache.org/jira/browse/GERONIMO-3140
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: deployment
>Affects Versions: 2.0-M5
>Reporter: David Jencks
>Assignee: David Jencks
> Fix For: Wish List
>
>
> We normally alias server modules on the client so that a reference to e.g. 
> the transaction-jta11 module gets turned into a reference to 
> client-transaction.  However if the module with this reference is already 
> started on the server while deployment is happening, there's no opportunity 
> to do this aliasing during deployment.  E.g. the openjpa module depends on 
> the tm module and is normally started: so any app client using opnejpa will 
> appear at deploy time to have 2 tms, one from tx-jta11 through the openjpa 
> dependency and one from client-transaction.
> One hacky workaround is to ignore duplicate gbean matches during deployment.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



Re: KEYS ?

2007-07-30 Thread Jason Dillon
Aighty, I'll keep that in mind and try to look for dangling  
references when I move it.


--jason


On Jul 30, 2007, at 9:35 AM, Dain Sundstrom wrote:

I'm not sure, but it may be mentioned in old binaries as the  
location for verifying the signatures.  I could be completely  
wrong... I just have a memory of some external references.  If you  
do move the file, I'd definitely search the website for references.


-dain

On Jul 27, 2007, at 12:35 PM, Jason Dillon wrote:


What external references are you referring to?

I believe that it does have symlink support, though I hardly ever  
use it.


--jason


On Jul 27, 2007, at 12:29 PM, Dain Sundstrom wrote:

I think there are external references to this file, but I'm not  
sure how we should deal with them.  Does svn have symlink support?


-dain

On Jul 26, 2007, at 3:52 PM, Jason Dillon wrote:

Should keys move up to a peer to the STATUS file too?  Seems  
these are project global, not specific to the server bits...


--jason










Re: svn commit: r561104 - /geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java

2007-07-30 Thread Anita Kulshreshtha
   This was probably added to provide/display Jetty (v 5.x) like
container wide statistics. Since tomcat does not provide container wide
statistics, this is set to false. Tomcat provides per connector
statistics which can be viewed using JMXViewer-->stats. One can always
compute container statistics by aggregating the connector statistics
and display them for the sake of uniformity.. I think per connector
statistics are more meaningful.

Thanks
Anita

--- Donald Woods <[EMAIL PROTECTED]> wrote:

> For branches/2.0, is there some reason the Tomcat stats are still
> disabled in 
> TomcatContainer.java?
> 
>  public boolean isStatisticsProvider() {
>  return false; // todo: return true once stats are integrated
>  }
> 
> 
> -Donald
> 
> Anita Kulshreshtha wrote:
> > FYI: This is the relevant change:
> > +public boolean isStatisticsProvider() {
> > +return true;
> > +}
> > 
> > Thanks
> > Anita
> > 
> > --- [EMAIL PROTECTED] wrote:
> > 
> >> Author: akulshreshtha
> >> Date: Mon Jul 30 13:19:14 2007
> >> New Revision: 561104
> >>
> >> URL: http://svn.apache.org/viewvc?view=rev&rev=561104
> >> Log:
> >>During Recent restructuring of tomcat connetors
> statisticsProvider
> >> attribute got lost. Reenable this attribute for HTTPConnectors
> >>
> >> Modified:
> >>
> >>
> >
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> >> Modified:
> >>
> >
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> >> URL:
> >>
> >
>
http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java?view=diff&rev=561104&r1=561103&r2=561104
> >
>
==
> >> ---
> >>
> >
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> >> (original)
> >> +++
> >>
> >
>
geronimo/server/trunk/modules/geronimo-tomcat6/src/main/java/org/apache/geronimo/tomcat/connector/BaseHttp11ConnectorGBean.java
> >> Mon Jul 30 13:19:14 2007
> >> @@ -1,409 +1,413 @@
> >> -/*
> >> - * Licensed to the Apache Software Foundation (ASF) under one
> >> - * or more contributor license agreements.  See the NOTICE file
> >> - * distributed with this work for additional information
> >> - * regarding copyright ownership.  The ASF licenses this file
> >> - * to you under the Apache License, Version 2.0 (the
> >> - * "License"); you may not use this file except in compliance
> >> - * with the License.  You may obtain a copy of the License at
> >> - *
> >> - *  http://www.apache.org/licenses/LICENSE-2.0
> >> - *
> >> - * Unless required by applicable law or agreed to in writing,
> >> - * software distributed under the License is distributed on an
> >> - * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> >> - * KIND, either express or implied.  See the License for the
> >> - * specific language governing permissions and limitations
> >> - * under the License.
> >> - */
> >> -package org.apache.geronimo.tomcat.connector;
> >> -
> >> -import java.net.InetAddress;
> >> -import java.net.InetSocketAddress;
> >> -import java.net.UnknownHostException;
> >> -import java.util.Map;
> >> -
> >> -import javax.management.j2ee.statistics.Stats;
> >> -import javax.net.ssl.KeyManagerFactory;
> >> -
> >> -import org.apache.geronimo.gbean.GBeanInfo;
> >> -import org.apache.geronimo.gbean.GBeanInfoBuilder;
> >> -import org.apache.geronimo.management.StatisticsProvider;
> >> -import org.apache.geronimo.system.serverinfo.ServerInfo;
> >> -import org.apache.geronimo.tomcat.TomcatContainer;
> >> -import org.apache.geronimo.tomcat.stats.ConnectorStats;
> >> -
> >> -public abstract class BaseHttp11ConnectorGBean extends
> >> ConnectorGBean implements BaseHttp11Protocol, StatisticsProvider {
> >> -
> >> -protected String connectHost;
> >> -
> >> -// JSR77 stats
> >> -private ConnectorStats connStatsProvider = new
> ConnectorStats();
> >> -
> >> -private boolean reset = true;
> >> -
> >> -public BaseHttp11ConnectorGBean(String name, Map initParams,
> >> String tomcatProtocol, String address, int port, TomcatContainer
> >> container, ServerInfo serverInfo) throws Exception {
> >> -super(name, initParams, tomcatProtocol, container,
> >> serverInfo);
> >> -
> >> -// Default the host to listen on all address is one was
> not
> >> specified
> >> -if (address == null) {
> >> -address = "0.0.0.0";
> >> -}
> >> -
> >> -// Must have a port
> >> -if (port == 0) {
> >> -throw new IllegalArgumentException("Must declare a
> >> port.");
> >> -}
> >> -
> >> -connector.setAttribute("address", address);
> >> -connector.setPort(port);
> >> -
> >> -}
> >> -

[jira] Closed: (GERONIMO-770) java.lang.IllegalStateException: More then one configuration mananger was found in kernel

2007-07-30 Thread David Jencks (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-770?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Jencks closed GERONIMO-770.
-

Resolution: Fixed
  Assignee: David Jencks

It's possible to make things like this happen if you manage to start an app 
client in the server, but it doesn't seem to happen unless you explicitly try 
to do that: certainly not as a result of a failed deployment.

> java.lang.IllegalStateException: More then one configuration mananger was 
> found in kernel
> -
>
> Key: GERONIMO-770
> URL: https://issues.apache.org/jira/browse/GERONIMO-770
> Project: Geronimo
>  Issue Type: Bug
>  Components: deployment
>Reporter: Jeremy Boynes
>Assignee: David Jencks
> Fix For: Wish List
>
>
> Seems to occur after unsuccessful deployment of an application client. Could 
> be that the org/apache/geronimo/Client configuration is left running so that 
> two GBeans are present.
> Once this occurs, the deployment tools are unable to locate the correct 
> configuration manager causing all deployment attempts to fail.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (GERONIMO-817) deploy-jsr88 module has dependency on openejb

2007-07-30 Thread David Jencks (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-817?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Jencks closed GERONIMO-817.
-

Resolution: Fixed

Gianni fixed this by making the deployer run off a config.xml so you can 
specify exactly which module jsr88 support gbeans you want.

> deploy-jsr88 module has dependency on openejb
> -
>
> Key: GERONIMO-817
> URL: https://issues.apache.org/jira/browse/GERONIMO-817
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: deployment
>Affects Versions: 1.0-M4, 1.0-M5
>Reporter: David Jencks
> Fix For: Wish List
>
> Attachments: nobuilderdependencies
>
>
> Apparently in an attempt to support dconfig beans, the deploy-jsr88 module 
> grew dependencies on several builder modules including openejb builder.  This 
> is a totally non-extensible archtecture and prevents building geronimo and 
> openejb in any semblance of a reasonable order.  I think that some kind of 
> registration scheme might be plausible but the current hard-coded 
> dependencies to specific builder implementations are not acceptable.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (GERONIMO-1393) Add test of EAR/WAR security options

2007-07-30 Thread David Jencks (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516582
 ] 

David Jencks commented on GERONIMO-1393:


Maybe these or something like them could be run against the new 
testsuite/enterprise-testsuite/sec-tests app?

> Add test of EAR/WAR security options
> 
>
> Key: GERONIMO-1393
> URL: https://issues.apache.org/jira/browse/GERONIMO-1393
> Project: Geronimo
>  Issue Type: Test
>  Security Level: public(Regular issues) 
>  Components: deployment, security
>Affects Versions: 1.0
>Reporter: Aaron Mulder
> Fix For: 1.x
>
> Attachments: ear-plan-no-realm-ear-security.xml, 
> ear-plan-no-realm-no-security.xml, ear-plan-war-realm-ear-security.xml, 
> ear-plan-war-realm-no-security.xml
>
>
> If we someday have a place to run deployment tests against a running server, 
> here are some plans that might come in handy.  These can be used with an EAR 
> wrapped around the LDAP-demo WAR (which I renamed to test-web.war for this 
> case),
> The test should try deploying that EAR with each of these plans in turn.  The 
> expected results are:
> ear-plan-no-realm-ear-security.xml: Fail with error
> ear-plan-no-realm-no-security.xml: Fail with error
> ear-plan-war-realm-no-security.xml: Fail with error
> ear-plan-war-realm-ear-security.xml: Deploy successfully
> To pass, there must be a security-realm-name element in the WAR and a 
> security element in either the EAR or the WAR.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (GERONIMO-1690) Additional support for Targets passed into JSR88

2007-07-30 Thread David Jencks (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-1690?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516583
 ] 

David Jencks commented on GERONIMO-1690:


Isn't this complete?

> Additional support for Targets passed into JSR88
> 
>
> Key: GERONIMO-1690
> URL: https://issues.apache.org/jira/browse/GERONIMO-1690
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>  Components: deployment
>Affects Versions: 1.1
>Reporter: Sachin Patel
>Priority: Minor
> Fix For: Wish List
>
> Attachments: patch.txt
>
>
> From an earlier dev list note...
> 1) The Target is currently discarded by our JSR-88 code because the
> Deployer GBean does not accept a Target/ConfigStore as an argument. 
> That needs to be changed, but it should be pretty straightforward.
> 2) I think the Target name is the full ObjectName of the ConfigStore,
> which means it would be pretty heinous to type on the command line. 
> Hopefully we can make it just the name= component of the ObjectName or
> something like that, but that would require some thought about what to
> do in case of non-unique names.
> 3) I don't think the Hot Deploy Dir GBean lets you specify a Target,
> and it probably should take that as a config param.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Assigned: (GERONIMO-1874) synthetic ears should allow use of modules outside the g repository

2007-07-30 Thread David Jencks (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-1874?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Jencks reassigned GERONIMO-1874:
--

Assignee: David Jencks

> synthetic ears should allow use of modules outside the g repository
> ---
>
> Key: GERONIMO-1874
> URL: https://issues.apache.org/jira/browse/GERONIMO-1874
> Project: Geronimo
>  Issue Type: Improvement
>  Security Level: public(Regular issues) 
>  Components: deployment
>Affects Versions: 1.1
>Reporter: David Jencks
>Assignee: David Jencks
> Fix For: Wish List
>
>
> Currently synthetic ears or external modules in an ear only accept external 
> artifacts that are in the geronimo repo, identified by an Artifact.  We 
> should also allow specification of a path that is not in the repository.  
> Perhaps this could be resolved using ServerInfo if the path is not absolute.  
> I think that changing the meaning of the current  to actually 
> mean a path and introducing a new element such as  perhaps with all 
> the parts to indicate an artifact in the repo would be a good idea.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (GERONIMO-2244) Explicit reference fail when module type is not "car"

2007-07-30 Thread David Jencks (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516585
 ] 

David Jencks commented on GERONIMO-2244:


No further work in the last year, and the actual problem was fixed.

> Explicit reference fail when module type is not "car"
> -
>
> Key: GERONIMO-2244
> URL: https://issues.apache.org/jira/browse/GERONIMO-2244
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: console, deployment
>Affects Versions: 1.1
>Reporter: Aaron Mulder
> Fix For: 1.1.x
>
>
> Currently the  used in the naming schema includes group, artifact, 
> and version, but not type.  In ENCConfigBuilder.buildAbstractNameQuery, the 
> type is hardcoded to "car".  This means that if you use a naming:pattern with 
> an artifactId, it only works if the target module's type is actually "car".  
> This is not true, for example, for several module types created by the admin 
> console, as well as in general for user-defined modules.
> The naming:pattern element should include a "type" element, and 
> ENCConfigBuilder (and any other necessary code) should respect it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Assigned: (GERONIMO-3314) (Sandbox) The Welcome portlet images and links are broken

2007-07-30 Thread Joe Bohn (JIRA)

 [ 
https://issues.apache.org/jira/browse/GERONIMO-3314?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Joe Bohn reassigned GERONIMO-3314:
--

Assignee: Joe Bohn

> (Sandbox) The Welcome portlet images and links are broken
> -
>
> Key: GERONIMO-3314
> URL: https://issues.apache.org/jira/browse/GERONIMO-3314
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: console
> Environment: WindowsXP
>Reporter: Daniel Larsen
>Assignee: Joe Bohn
>Priority: Trivial
> Attachments: Geronimo-3314.patch
>
>
> (sandbox/portals)
> The Welcome and Keystore portlet images are broken due to this change: 
> https://issues.apache.org/jira/browse/GERONIMO-3345
> The 3 links under "Common Console Actions" are also routed to the wrong place.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[VOTE] Release specs for JACC, JSP, Servlet

2007-07-30 Thread Tim McConnell

Hi, Please review and vote on the release of the following Geronimo specs:

-- geronimo-jacc_1.1_spec-1.0
-- geronimo-jsp_2.1_spec-1.0
-- geronimo-servlet_2.5_spec-1.1

The corresponding tar files are here:

http://people.apache.org/~mcconne/geronimo-jacc_1.1_spec-1.0.tar.gz
http://people.apache.org/~mcconne/geronimo-jsp_2.1_spec-1.0.tar.gz
http://people.apache.org/~mcconne/geronimo-servlet_2.5_spec-1.1.tar.gz

The current svn locations are here:

https://svn.apache.org/repos/asf/geronimo/specs/branches/geronimo-jacc_1.1_spec-1.0
https://svn.apache.org/repos/asf/geronimo/specs/branches/geronimo-jsp_2.1_spec-1.0
https://svn.apache.org/repos/asf/geronimo/specs/branches/geronimo-servlet_2.5_spec-1.1

The future svn locations will be here:

https://svn.apache.org/repos/asf/geronimo/specs/tags/geronimo-jacc_1.1_spec-1.0
https://svn.apache.org/repos/asf/geronimo/specs/tags/geronimo-jsp_2.1_spec-1.0
https://svn.apache.org/repos/asf/geronimo/specs/tags/geronimo-servlet_2.5_spec-1.1

The vote will conclude at 10:00 PM EST on Thursday, August 2nd.

--
Thanks,
Tim McConnell




[jira] Closed: (DAYTRADER-38) Cleanup ejb-jar.xml by removing one-phase remnants and fixing spec compliance warnings

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-38?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe closed DAYTRADER-38.
-

   Resolution: Fixed
Fix Version/s: 2.0
   1.2

Fixes now applied to both DayTrader 1.2 and 2.0

> Cleanup ejb-jar.xml by removing one-phase remnants and fixing spec compliance 
> warnings
> --
>
> Key: DAYTRADER-38
> URL: https://issues.apache.org/jira/browse/DAYTRADER-38
> Project: DayTrader
>  Issue Type: Improvement
>  Components: EJB Tier
>Affects Versions: 1.2, 2.0
>Reporter: Christopher James Blythe
>Assignee: Christopher James Blythe
>Priority: Minor
> Fix For: 1.2, 2.0
>
>
> While working to deploy Daytrader 1.2 on Sun Server 9 (Glassfish) with 
> NetBeans 5.5, the NetBeans validation utilities pointed out several errors 
> and spec compliance warnings in the ejb-jar.xml. To remove these warnings and 
> errors, the following changes were necessary...
> - Remove remnants of one-phase methods in the container transactions section
> - Remove the  tags from the query definitions
> - Add the  tag to the MDB definitions

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (DAYTRADER-40) Add vendor specific DDs for deployment on Sun Java System Application Server Platform Edition 9

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-40?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe updated DAYTRADER-40:
--

Affects Version/s: (was: 2.0)

Making this a 1.2 only item. May revisit this in the 2.0 time frame later.

> Add vendor specific DDs for deployment on Sun Java System Application Server 
> Platform Edition 9
> ---
>
> Key: DAYTRADER-40
> URL: https://issues.apache.org/jira/browse/DAYTRADER-40
> Project: DayTrader
>  Issue Type: New Feature
>  Components: EJB Tier, J2EE Application Clients, Web Tier
>Affects Versions: 1.2
>Reporter: Christopher James Blythe
>Assignee: Christopher James Blythe
>Priority: Minor
> Fix For: 1.2
>
>
> Add the vendor specific deployment descriptors required to deploy Daytrader 
> on Sun Java System Application Server Platform Edition 9. Since these do not 
> impact the based J2EE deployment descriptors in any way, they can coexist in 
> the project without impacting Geronimo, etc.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (DAYTRADER-40) Add vendor specific DDs for deployment on Sun Java System Application Server Platform Edition 9

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-40?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe closed DAYTRADER-40.
-

   Resolution: Fixed
Fix Version/s: 1.2

Completed in 1.2

> Add vendor specific DDs for deployment on Sun Java System Application Server 
> Platform Edition 9
> ---
>
> Key: DAYTRADER-40
> URL: https://issues.apache.org/jira/browse/DAYTRADER-40
> Project: DayTrader
>  Issue Type: New Feature
>  Components: EJB Tier, J2EE Application Clients, Web Tier
>Affects Versions: 1.2
>Reporter: Christopher James Blythe
>Assignee: Christopher James Blythe
>Priority: Minor
> Fix For: 1.2
>
>
> Add the vendor specific deployment descriptors required to deploy Daytrader 
> on Sun Java System Application Server Platform Edition 9. Since these do not 
> impact the based J2EE deployment descriptors in any way, they can coexist in 
> the project without impacting Geronimo, etc.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (DAYTRADER-39) Revise exception handling in EJB-tier to comply with spec

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe updated DAYTRADER-39:
--

Affects Version/s: (was: 2.0)

Making this a 1.2 only item for the time being. May revisit in 2.0.

> Revise exception handling in EJB-tier to comply with spec
> -
>
> Key: DAYTRADER-39
> URL: https://issues.apache.org/jira/browse/DAYTRADER-39
> Project: DayTrader
>  Issue Type: Improvement
>  Components: EJB Tier
>Affects Versions: 1.2
>Reporter: Christopher James Blythe
>Assignee: Christopher James Blythe
>Priority: Minor
> Fix For: 1.2
>
>
> While working to deploy Daytrader 1.2 on Sun Server 9 (Glassfish) with 
> NetBeans 5.5, the NetBeans validation utilities pointed out spec violations 
> associated with the exceptions thrown by classes in the EJB tier. To comply 
> with the spec, the following high level rules were followed...
> - TradeServices.java - all methods should throw Exception and RemoteException
> - Trade.java - all methods should throw Exception and RemoteException
> - TradeBean and TradeJDBCBean - methods should simply throw Exception (not 
> RemoteException)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (DAYTRADER-39) Revise exception handling in EJB-tier to comply with spec

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe closed DAYTRADER-39.
-

   Resolution: Fixed
Fix Version/s: 1.2

> Revise exception handling in EJB-tier to comply with spec
> -
>
> Key: DAYTRADER-39
> URL: https://issues.apache.org/jira/browse/DAYTRADER-39
> Project: DayTrader
>  Issue Type: Improvement
>  Components: EJB Tier
>Affects Versions: 1.2
>Reporter: Christopher James Blythe
>Assignee: Christopher James Blythe
>Priority: Minor
> Fix For: 1.2
>
>
> While working to deploy Daytrader 1.2 on Sun Server 9 (Glassfish) with 
> NetBeans 5.5, the NetBeans validation utilities pointed out spec violations 
> associated with the exceptions thrown by classes in the EJB tier. To comply 
> with the spec, the following high level rules were followed...
> - TradeServices.java - all methods should throw Exception and RemoteException
> - Trade.java - all methods should throw Exception and RemoteException
> - TradeBean and TradeJDBCBean - methods should simply throw Exception (not 
> RemoteException)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (DAYTRADER-47) Add full EJB 3 runtime mode (including EJB 3 based stateless session bean and MDBs)

2007-07-30 Thread Christopher James Blythe (JIRA)
Add full EJB 3 runtime mode (including EJB 3 based stateless session bean and 
MDBs)
---

 Key: DAYTRADER-47
 URL: https://issues.apache.org/jira/browse/DAYTRADER-47
 Project: DayTrader
  Issue Type: New Feature
  Components: EJB Tier
Affects Versions: 2.0
Reporter: Christopher James Blythe
 Fix For: 2.0


This code was committed into trunk in revision 559535 on 7/25/07

The following information was posted to the dev list regarding the changes. 
Since this basically replaces the existing TradeJPA implementation, I think  it 
should be removed (will address this in another JIRA).  

All,

As it currently stands, DayTrader 2.0 does not provide what I would consider a 
viable showcase application for Java EE 5 technology. The JPA mode that was 
added uses EJB3/JPA based entities; however, the mode still lacks a few key 
elements...

- EJB 3 based stateless session bean providing the business logic (current impl 
still uses EJB 2.1 session beans)
- EJB 3 based MDBs for Quote streamer and async order processing
- Improvements to JPA mode to ensure data consistency and improve performance

I have added a new EJB3 runtime mode that exposes the following...
- TradeSLSBBean (new EJB 3 based session bean for business logic)
- DTBroker3MDB (new EJB 3 based MDB for async order processing)
- DTStreamer3MDB (new EJB 3 based MDB for quote streamer)

I have made these updates and plan to commit them shortly, I have left the 
existing TradeJPA bean as is, in addition to the legacy Direct and EJB 2.1 
modes. I have pulled yesterday's build of Geronimo 2.0 and verified that the 
new EJB3 mode functionally works along with the legacy modes. The next step is 
load testing, so stay tuned...

Thanks...

Chris

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (DAYTRADER-47) Add full EJB 3 runtime mode (including EJB 3 based stateless session bean and MDBs)

2007-07-30 Thread Christopher James Blythe (JIRA)

 [ 
https://issues.apache.org/jira/browse/DAYTRADER-47?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher James Blythe closed DAYTRADER-47.
-

Resolution: Fixed

> Add full EJB 3 runtime mode (including EJB 3 based stateless session bean and 
> MDBs)
> ---
>
> Key: DAYTRADER-47
> URL: https://issues.apache.org/jira/browse/DAYTRADER-47
> Project: DayTrader
>  Issue Type: New Feature
>  Components: EJB Tier
>Affects Versions: 2.0
>Reporter: Christopher James Blythe
> Fix For: 2.0
>
>
> This code was committed into trunk in revision 559535 on 7/25/07
> The following information was posted to the dev list regarding the changes. 
> Since this basically replaces the existing TradeJPA implementation, I think  
> it should be removed (will address this in another JIRA).  
> All,
> As it currently stands, DayTrader 2.0 does not provide what I would consider 
> a viable showcase application for Java EE 5 technology. The JPA mode that was 
> added uses EJB3/JPA based entities; however, the mode still lacks a few key 
> elements...
> - EJB 3 based stateless session bean providing the business logic (current 
> impl still uses EJB 2.1 session beans)
> - EJB 3 based MDBs for Quote streamer and async order processing
> - Improvements to JPA mode to ensure data consistency and improve performance
> I have added a new EJB3 runtime mode that exposes the following...
> - TradeSLSBBean (new EJB 3 based session bean for business logic)
> - DTBroker3MDB (new EJB 3 based MDB for async order processing)
> - DTStreamer3MDB (new EJB 3 based MDB for quote streamer)
> I have made these updates and plan to commit them shortly, I have left the 
> existing TradeJPA bean as is, in addition to the legacy Direct and EJB 2.1 
> modes. I have pulled yesterday's build of Geronimo 2.0 and verified that the 
> new EJB3 mode functionally works along with the legacy modes. The next step 
> is load testing, so stay tuned...
> Thanks...
> Chris

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (DAYTRADER-48) Remove TradeJPA mode (replaced by full EJB 3 mode)

2007-07-30 Thread Christopher James Blythe (JIRA)
Remove TradeJPA mode (replaced by full EJB 3 mode)
--

 Key: DAYTRADER-48
 URL: https://issues.apache.org/jira/browse/DAYTRADER-48
 Project: DayTrader
  Issue Type: Task
  Components: EJB Tier
Affects Versions: 2.0
Reporter: Christopher James Blythe
Priority: Minor


The EJB 3 code that was recently committed provides a full EJB 3 implementation 
of the DayTrader workload using EJB 3 entities, sessions and MDBs. This 
improves upon the original Trade JPA mode (from which the business logic was 
originally ported) that used EJB 2.1 based session and MDBs. Furthermore, minor 
modifications were made to improve data consistency and performance (via 
eager/lazy loading of the entities).

I see no real need for the original JPA mode to stick around and feel it should 
be removed.

Any objections?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (DAYTRADER-49) Add EJB 3 based Session-to-Direct mode

2007-07-30 Thread Christopher James Blythe (JIRA)
Add EJB 3 based Session-to-Direct mode
--

 Key: DAYTRADER-49
 URL: https://issues.apache.org/jira/browse/DAYTRADER-49
 Project: DayTrader
  Issue Type: New Feature
  Components: EJB Tier
Affects Versions: 2.0
Reporter: Christopher James Blythe


This JIRA was opened to address the need for an EJB 3 based Stateless Session 
Bean to Direct JDBC mode (similar to what was added to the EJB 2.1 
implementation in DAYTRADER-15).

The code changes are relatively straight forward and simply require a new 
session bean which calls the service methods via TradeDirect.

As previously mentioned in DAYTRADER-15. This is known to be a common method 
for managing transactions commits/rollbacks for EJB 2.1 applications. I imagine 
this will not change for EJB 3.0.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.