It is a simple change for me to put together in my branch, and will do
so today once I get feedback from tom.jenkinson(a)redhat.com, since he is
taking over the tm integration from Jonathan.
On 6/15/11 9:45 AM, Brian Stansberry wrote:
If someone has time to rework it for 7.0, I'd love it. I wanted
to do
some work on it this week, but time is looking tight.
On 6/15/11 11:37 AM, David M. Lloyd wrote:
> On 06/15/2011 11:24 AM, Scott Stark wrote:
>> I have the ability to disable the remote interfaces to the recovery
>> manager working in the following pull request:
>>
https://github.com/jbossas/jboss-as/pull/58
>>
>> In this request I raise the question of whether the current
>> configuration is too low level in that it is largely a mapping of the tm
>> configuration beans, and it takes a combination of two properties to
>> achieve the result of disabling the recovery manager remote interfaces.
>> Perhaps just a single enable-remote-access on the recovery-environment
>> that sets appropriate properties on the underlying
>> RecoveryEnvironmentBean and CoordinatorEnvironmentBean would be a better
>> solution.
> Yeah it's always been my wish to present a higher-level configuration.
> The config beans have a lot of problems and reflecting them in the
> configuration itself is a mistake.
>
> The configuration should be use-case driven, and the subsystem API
> should be a reflection of that. The config beans layer really seems to
> have little to do with use-cases or implementation for that matter.
>
> All that said though - I'm not sure we have enough time before 7.0 to
> make this change. It might have to wait for 7.1. We'll have to see
> what Jason and Brian say.