admin@glassfish.java.net

Re: Code Review: Bug fix for: 6505045 ...

From: Kedar Mhaswade <Kedar.Mhaswade_at_Sun.COM>
Date: Tue, 02 Oct 2007 16:21:27 -0700

I agree, but we need to fix a few bugs.

They are significantly important and *very* low risk as far as I can tell.

Request: please consider these:

https://glassfish.dev.java.net/issues/show_bug.cgi?id=3719
https://glassfish.dev.java.net/issues/show_bug.cgi?id=3715

Thanks,
Kedar

Shreedhar Ganapathy wrote:
>
>> PS - Can I trade some other bugs for UR1, instead? :)
> :)
> At the moment we are exceeding the max threshold we had in mind for bugs
> that must go into UR1. Our preference at this point is to hold back
> lesser issues for another release.
>>
>> Mark A. Basler wrote:
>>> Hi Guys,
>>>
>>> Has this issue been resolved?
>>>
>>> Does this need to go into Glassfish V2 update 1?
>>>
>>> Please let me know...
>>>
>>> Thanks - Mark
>>>
>>> Kedar Mhaswade wrote:
>>>>
>>>>> Re-reading the original bug report, I think the right solution is
>>>>> to simply refuse to even *attempt* to change domain.xml if we
>>>>> cannot create a temporary backup file (a temporary backup file has
>>>>> other safety benefits too).
>>>>
>>>> A question:
>>>>
>>>> Do you mean don't even attempt
>>>> - to "actually write" the domain.xml to disk, or
>>>> - to modify the memory that represents the domain.xml's contents,
>>>>
>>>> when you can't create the backup file?
>>>>
>>>>
>>>>> Lloyd
>>>>>
>>>>> On Sep 20, 2007, at 5:50 PM, Kedar Mhaswade wrote:
>>>>>
>>>>>> Finally, I have been able to fix this problem (reasonably).
>>>>>>
>>>>>> All about it can be found at:
>>>>>> http://wiki.glassfish.java.net/attach/GlassFishAdminReferences/6505045.html
>>>>>>
>>>>>>
>>>>>> Please go through the background and code changes if you have time
>>>>>> and let me
>>>>>> know if you have any comments, by tomorrow 12.00 noon, pacific.
>>>>>>
>>>>>> I am planning to check this in then, on GlassFish trunk.
>>>>>>
>>>>>> Mark Basler: I leave it up to you to approve the fix for 9.1 UR1.
>>>>>> If you do
>>>>>> want to do that, let me know and I can port the changes there.
>>>>>>
>>>>>> I have run the tests on my 128M Zip-drive on Mac OS X by observing
>>>>>> domain.xml go
>>>>>> to zero-bytes and domain.xml.protect coming to rescue, making
>>>>>> every attempt
>>>>>> to persist the domain.xml.
>>>>>>
>>>>>> Regards,
>>>>>> Kedar
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
>>>>>> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
>>>>> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
>>> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>