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

Reply via email to