[Design of POJO Server] - Re: Updating docs
by adrian@jboss.org
If a metadata file is being parsed then it is potentially
something that could be passed in as a predetermined managed object.
i.e. You override the factory settings using the profile service
or a management tool.
To identify what is overridable (it is parsed from metadata files),
it populates the transient managed objects which the
profile service/front end can override by promoting the modified object to
the predetermined section.
They are different from the plain attachments which
are just temporary artifacts of the deployment that deployers
use to communicate with each other.
P.S. Not sure that managed object is a good name for this
since it conflicts with the real managed object created from
this object + annotations on the class.
Metadata is probably better but that also conflicts?
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3973512#3973512
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3973512
19 years, 6 months
[Design of Messaging on JBoss (Messaging/JBoss)] - Re: JBMESSAGING-545 - Leaks on XA transactions
by timfox
"clebert.suconic(a)jboss.com" wrote :
|
| So, summarizing the changes I'm making to fix this:
|
| I - I'm creating a method deleteTransaction on TransactionRepository.
|
| II - I also changed Transaction to accept a reference to TransactionRepository, as TransactionRepository is not static.
|
|
Sounds good
anonymous wrote :
| III - On commit/rollback transactionRepository.delete is being called
|
Fine. Make sure this is done at the end of the processing and not in a finally block - we don't want to remove the tx if it fails (the tx mgr can retry)
anonymous wrote :
|
| IV - Changed JDBCPersistenceManagerTest to perform a test on this
|
| V - Added a property method getNumberOfRegisteredTransactions() on TransactionRepository what is meant to be used only by tests. (I could use reflection on a private field for this, but I thought it was much cleaner to just add the property field. I could change this if someone doesn't like the exposed property)
|
|
|
Fine.
Also we need to make sure we return ok if a commit request comes in for a 2pc transaction and it isn't in the map.
There is a second part of this tasks which involves looking in the database if a request comes in for a transaction and it isn't in the map - but it's probably best to wait for Madhu to integrate his XA recovery work since you will need to re-use some of that.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3973509#3973509
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3973509
19 years, 6 months
[Design of Messaging on JBoss (Messaging/JBoss)] - Re: Incorporating Remoting http transport into Messaging
by ron_sigal
Here's what I've got so far.
As a first approximation, replacing the current Messaging push callbacks with Remoting's polling simulation of push callbacks is pretty easy. On the client side, CallbackManager is transformed from a ServerInvocationHandler to an InvokerCallbackHandler. When the Remoting poller pulls callbacks from the server, it passes them to CallbackManager, which proceeds as before.
On the server side, instead of passing around the remoting callback Client, which was being used to make explicit invocations that were processed by CallbackManager, the ServerInvokerCallbackHandler from which the callback Client was obtained is passed around. Instead of calling Client.invoker(), ServerConsumerEndpoint.Deliverer calls ServerInvokerCallbackHandler.handleCallback(), which, since it is configured for pull callbacks, just stores the callback with its load of messages. Eventually the client side poller will retrieve them and pass them to CallbackManager.
To some extent, these changes work. E.g., simple tests like HttpExample and QueueExample work. However, some of the more complex tests, for example, MessageConsumerTest.redel8(), fail. It looks like CallbackManager is trying to deliver the messages to a Consumer that has already closed.
I'll need to look at this some more.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3973481#3973481
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3973481
19 years, 6 months