Code review comment for lp:~vkolesnikov/pbxt/pbxt-type-enum

Revision history for this message
Paul McCullagh (paul-mccullagh) wrote :

I have seen this kind of thing before. I think it is a bug, and should
be investigated.

I all PBXT tables are dropped first, the it should always be possible
to do:

drop database pbxt;

On May 29, 2009, at 12:36 PM, Vladimir Kolesnikov wrote:

> yes, the problem was I got an error like: cannot drop dtabase pbxt,
> cannot delete table "statistics,locations"
>
> Paul McCullagh wrote:
>> Hi Vlad,
>>
>> Was there a reason for removing "drop database pbxt"?
>>
>> This change was required by MariaDB which tests that the state of the
>> server is the same as before and after each test.
>>
>> On May 29, 2009, at 10:40 AM, Vladimir Kolesnikov wrote:
>>
>>
>>> --disable_query_log
>>> -drop database pbxt;
>>> ---enable_query_log
>>> +#--disable_query_log
>>> +#drop database pbxt;
>>> +#--enable_query_log
>>> --echo End of 5.1 tests
>>>
>>
>>
>>
>> --
>> Paul McCullagh
>> PrimeBase Technologies
>> www.primebase.org
>> www.blobstreaming.org
>> pbxt.blogspot.com
>>
>>
>>
>>
>
>
> --
> --
> Best Regards,
> Vladimir
>
> https://code.launchpad.net/~vkolesnikov/pbxt/pbxt-type-enum/+merge/
> 6878
> Your team PBXT Core is subscribed to branch lp:pbxt.

--
Paul McCullagh
PrimeBase Technologies
www.primebase.org
www.blobstreaming.org
pbxt.blogspot.com

« Back to merge proposal