[erlang-patches] patch: Allow a user_defined function to wrap mnesia_schema:merge_schema()
Ulf Wiger
ulf.wiger@REDACTED
Tue Jun 1 20:04:55 CEST 2010
git fetch git://github.com/uwiger/otp.git schema_merge
"remove debug printout and accidental variable name reuse"
http://github.com/uwiger/otp/commit/b4844311746b44e85aa49d6e7bd68703301998e0
Thanks Fredrik Thulin for helping me figure out what I did wrong.
BR,
Ulf W
On 06/01/2010 05:57 PM, Raimo Niskanen wrote:
> On Tue, Jun 01, 2010 at 04:19:36PM +0200, Ulf Wiger wrote:
>>
>> What is the easiest way to address it?
>>
>> The debug printout:
>> Line 2663 in
>> http://github.com/erlang/otp/blob/pu/lib/mnesia/src/mnesia_schema.erl
>> should be deleted (I've not seen any other debug printouts)
>>
>> The crash:
>> Line 2676, 2679 - rename NewNodes to something else, e.g. OtherNodes.
>>
>> I could resubmit, but my git fu is still low enough that I'm
>> likely to make a mess of things without detailed instructions. :)
>
> The easiest for you (and for us to follow) would be to
> just keep on working on the same branch:
>
> In your local repository (from which you pushed the local branch
> schema_merge) to your github repository 'origin' (assumed).
>
> $ git checkout schema_merge
>
> Do the changes above.
> Test it.
>
> $ git push origin schema_merge
>
> Send a mail to erlang-patches saying you have new commits
> available with:
>
> git fetch git://github.com/uwiger/otp.git schema_merge
>
>>
>> I have done the above and tested with four nodes (it did in fact
>> crash before; now it doesn't).
>>
>> BR,
>> Ulf W
>>
>> Dan Gudmundsson wrote:
>>> There are debug printouts left.
>>> Starting a cluster of more than 2 nodes (i.e. 3) crashes for me.
>>>
>>> /Dan
>>>
>>> 2010/5/10 Björn Gustavsson <bgustavsson@REDACTED>:
>>>> On Sun, May 9, 2010 at 11:40 AM, Ulf Wiger
>>>> <ulf.wiger@REDACTED> wrote:
>>>>> http://github.com/uwiger/otp/tree/schema_merge
>>>> Thanks! Will include in 'pu'.
>>>>
>>>> For future contributions, please also include a "git fetch" line in
>>>> the email that I can
>>>> copy/paste into the shell. For instance:
>>>>
>>>> git fetch git://github.com/uwiger/otp.git schema_merge
>>>>
>>>> --
>>>> Björn Gustavsson, Erlang/OTP, Ericsson AB
>>>>
>>>> ________________________________________________________________
>>>> erlang-patches (at) erlang.org mailing list.
>>>> See http://www.erlang.org/faq.html
>>>> To unsubscribe; mailto:erlang-patches-unsubscribe@REDACTED
>>>>
>>>>
>>
>>
>> --
>> Ulf Wiger
>> CTO, Erlang Solutions Ltd, formerly Erlang Training & Consulting Ltd
>> http://www.erlang-solutions.com
>>
>> ---------------------------------------------------
>>
>> ---------------------------------------------------
>>
>> WE'VE CHANGED NAMES!
>>
>> Since January 1st 2010 Erlang Training and Consulting Ltd. has become
>> ERLANG SOLUTIONS LTD.
>>
>> www.erlang-solutions.com
>>
>>
>> ________________________________________________________________
>> erlang-patches (at) erlang.org mailing list.
>> See http://www.erlang.org/faq.html
>> To unsubscribe; mailto:erlang-patches-unsubscribe@REDACTED
>>
>
---------------------------------------------------
---------------------------------------------------
WE'VE CHANGED NAMES!
Since January 1st 2010 Erlang Training and Consulting Ltd. has become ERLANG SOLUTIONS LTD.
www.erlang-solutions.com
More information about the erlang-patches
mailing list