Re: DayTrader failures on 1.2 for SLSB JDBC Mode

2006-12-09 Thread Christopher Blythe

Matt...

Everything I've been doing this far has been in the Daytrader 1.2 branch
running against Geronimo 1.1. Still trying to get Geronimo 1.2 built per my
other emails.

I specifically removed those methods in TradeJDBC remote interface so they
would not have to be implemented (in Daytrader-28). From what I could tell
they they were never used, but let me revisit. I think Geronimo 1.1 never
checked to see if there was an impl for them and hence never complained. My
guess is that Geronimo 1.2/2.0 now checks, but I haven't' had a chance to
run.

Actually, I ran into something similar when I tried to deploy on
WebSphere... it threw warnings at deployment, but instead of providing the
impl like you did, I just removed the method signatures from the interface.

Anyway, I'll take a look on Monday and try to sort out what happened... Just
curious, did you apply Daytrader-28?

Thanks...

Chris

On 12/8/06, Matt Hogstrom [EMAIL PROTECTED] wrote:


Chris/Piyush,

I've incorporated a few of the patches which included the new Session
JDBC work as well as the new table creation code from Piyush.  I was
getting some errors in deploying the application from trunk and had
to add a few business nethods to the TradeBean class.  Chris, did you
deploy your changes on Geronimo?  I'm wondering if there is some
oddness in the way I applied the patch or in Geronimo.

also, I'm getting errors in EJB mode and JPA mode I'm debugging.
Before I go too far down the rat hole it would be comforting to know
you looked at trunk and the problems are mine and not some
misapplication of the patch.  In the interim I'll continue to debug.

It looks like were missing the business methods from:

modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/
session/TradeJDBCBean.java
+   public OrderDataBean completeOrderOnePhase(Integer orderID)
throws Exception, RemoteException {

+   public void cancelOrderOnePhase(Integer orderID) throws
Exception, RemoteException {

+   public double investmentReturn(double investment, double
NetValue) throws Exception {

+   public void publishQuotePriceChange(QuoteDataBean quoteData,
java.math.BigDecimal oldPrice, java.math.BigDecimal changeFactor,
double sharesTraded) throws Exception {

Did I miss something?

Matt Hogstrom
[EMAIL PROTECTED]






--
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: DayTrader failures on 1.2 for SLSB JDBC Mode

2006-12-09 Thread Matt Hogstrom


On Dec 9, 2006, at 5:10 PM, Christopher Blythe wrote:


Matt...

Everything I've been doing this far has been in the Daytrader 1.2  
branch running against Geronimo 1.1. Still trying to get Geronimo  
1.2 built per my other emails.


I specifically removed those methods in TradeJDBC remote interface  
so they would not have to be implemented (in Daytrader-28). From  
what I could tell they they were never used, but let me revisit. I  
think Geronimo 1.1 never checked to see if there was an impl for  
them and hence never complained. My guess is that Geronimo 1.2/2.0  
now checks, but I haven't' had a chance to run.




Actually, I ran into something similar when I tried to deploy on  
WebSphere... it threw warnings at deployment, but instead of  
providing the impl like you did, I just removed the method  
signatures from the interface.



Ok, I'll remove them as well...just wanted to make sure.


Anyway, I'll take a look on Monday and try to sort out what  
happened... Just curious, did you apply Daytrader-28?




Not yet...was verifying the other patches...I'll get to it once I get  
all modes working.



Thanks...

Chris



Matt Hogstrom
[EMAIL PROTECTED]




DayTrader failures on 1.2 for SLSB JDBC Mode

2006-12-08 Thread Matt Hogstrom

Chris/Piyush,

I've incorporated a few of the patches which included the new Session  
JDBC work as well as the new table creation code from Piyush.  I was  
getting some errors in deploying the application from trunk and had  
to add a few business nethods to the TradeBean class.  Chris, did you  
deploy your changes on Geronimo?  I'm wondering if there is some  
oddness in the way I applied the patch or in Geronimo.


also, I'm getting errors in EJB mode and JPA mode I'm debugging.   
Before I go too far down the rat hole it would be comforting to know  
you looked at trunk and the problems are mine and not some  
misapplication of the patch.  In the interim I'll continue to debug.


It looks like were missing the business methods from:

modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/ 
session/TradeJDBCBean.java
+   public OrderDataBean completeOrderOnePhase(Integer orderID)  
throws Exception, RemoteException {


+   public void cancelOrderOnePhase(Integer orderID) throws  
Exception, RemoteException {


+   public double investmentReturn(double investment, double  
NetValue) throws Exception {


+   public void publishQuotePriceChange(QuoteDataBean quoteData,  
java.math.BigDecimal oldPrice, java.math.BigDecimal changeFactor,   
double sharesTraded) throws Exception {


Did I miss something?

Matt Hogstrom
[EMAIL PROTECTED]