detail.php
OW2 Forge: Detail: 303537 jotm.properties discovery algorithm is too rigid

Advanced - Powered by Google


   
Log In
New Account
  
 
Home
My Page
Project Tree
Project Openings
JOTM
          
 
 
Summary
Tracker
Lists
Tasks
News
CVS
Files
SVN
                
 

Tracker: Bugs

Submit New | Browse | Admin | ExportToXml

[ #303537 ] jotm.properties discovery algorithm is too rigid

Date:
2005-04-30 23:05
Priority:
2
Submitted By:
Ludovic Orban (lorban)
Assigned To:
David A. Egolf (degolf)
Category:
JOTM
State:
Open
Summary:
jotm.properties discovery algorithm is too rigid

Detailed description:
org.objectweb.jotm.TransactionRecoveryImpl looks for a jotm property file under some folder which name is built in a too rigid way (jotm.base or jonas.base system propery + hardcoded conf subfolder). It should be possible to include jotm.properties in the classpath or in a folder that would be specified with another system property. This problem bloats JettyPlus a bit because a conf folder needs to be created and a command-line system property must be passed-in if TX recovery is wanted. That's not critical of course but this is a nice to have. I have patched the TransactionRecoveryImpl class to be able to read the properties from classpath. I can send you a patch if you want.

Add A Comment:

Please login

Followup

Message
Date: 2005-05-01 11:38
Sender: lorban
Logged In: YES 
user_id=6112

Here I attached my simplistic patch. It just tries to read a
jtom.properties file from classpath if it could not find it
in the conf folder. I also fixed a small bug: you forgot to
close the file after reading it.

Yes, I'm attempting to implement TX recovery within
JettyPlus. More details about this here:
http://lorban.blogspot.com/2005/04/joy-of-xa.html

Thank YOU for being so responsive.
Date: 2005-05-01 09:02
Sender: degolf
Logged In: YES 
user_id=736

Hi Ludovic,

Yes!  Please send a patch for us to see.  We were not 
happy with the properties file situation, but we needed 
some direction.

The hardcoded "conf" folder concept came from jonas,
which
is the first platform which actually supports recovery.  
Therefore, for jonas users this was not a problem.

For other users, recovery is not implemented on other 
platforms.  We didn't expect this to be an issue until May 
after we publish the interface for the resource manager 
registrations and the administrative reporting.

Meanwhile, please send the patches so that we can consider 
your implementation.

Also, please let us know if you are attempting to 
implement recovery on some other application platform.

Thank you,

David Egolf
david.egolf@objectweb.org
jotm project

Attached Files:

Name Description Download
TransactionFactoryImpl.patch Q&D patch to load props from classpath Download

Changes:

Field Old Value Date By
priority42009-03-10 17:02durieuxp
artifact_group_idNone2009-03-10 17:02durieuxp
File Added228: TransactionFactoryImpl.patch2005-05-01 11:33lorban
priority52005-05-01 09:02degolf
assigned_tonone2005-05-01 09:02degolf
priority42005-04-30 23:06lorban
category_idNone2005-04-30 23:05lorban
priority52005-04-30 23:05lorban

Copyright © 1999-2008, OW2 Consortium | contact | You have a difficulty, a problem ? Please report an issue using your OW2 forge account credentials