admin@glassfish.java.net

Re: Please review fix for 6467835

From: Kedar Mhaswade <Kedar.Mhaswade_at_Sun.COM>
Date: Thu, 14 Sep 2006 08:31:17 -0700

Please go ahead.
BTW, does this affect all types of brokers?

Kedar

Sivakumar Thyagarajan wrote:
> Could you please review this fix for issue 6467835?
> http://blrshare.india.sun.com/~st148833/codereview/6467835/
>
> Issue: The varhome option was not sent while trying to init the
> imqbroker, during domain creation, and this resulted in the imqbrokerd
> using the default varhome [domain1's varhome] always.
>
> Thanks
> --Siva.
>
> uw-diff for non-swan users:
>
>> Index:
>> servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java
>>
>> ===================================================================
>> RCS file:
>> /cvs/glassfish/admin/servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java,v
>>
>> retrieving revision 1.6
>> diff -u -w -r1.6 RepositoryManager.java
>> ---
>> servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java
>> 8 May 2006 16:27:12 -0000 1.6
>> +++
>> servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java
>> 14 Sep 2006 11:23:12 -0000
>> @@ -771,6 +771,8 @@
>> final List cmdInput = new ArrayList();
>> cmdInput.add(broker.getAbsolutePath());
>> cmdInput.add("-init");
>> + cmdInput.add("-varhome");
>> + cmdInput.add(mqVarHome.getAbsolutePath());
>> ProcessExecutor pe = new ProcessExecutor
>> ((String[])cmdInput.toArray(new String[cmdInput.size()]));
>> pe.execute(false, false);
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>