Hi!

I think the direction of moving to Discovery pretty much precludes us doing anything but this. Files that are built by the engine have to be released by the engine in the end. We can't/won't be in the book keeping business for them.

Cheers,
        -Brian

On Dec 16, 2008, at 7:09 AM, Jim Starkey wrote:

Barry Leslie wrote:
Hi,

Is there a reason why mysql_rm_db() only calls ha_drop_database() after
mysql_remove_known_files() returns with out an error?

I would think it would make more sense to call ha_drop_database() first to allow the handlers a chance to cleanup before mysql_remove_known_files() is
called.

The way it is currently being done makes it complicated for handlers that
create sub folders in the database to remove them properly before the
database is dropped.


I think it would be appropriate if the drizzle server got out of the business of dropping files on behalf of storage engines. It's dangerous, saves a trivial amount of code, and violates the layering of server on storage engine. The whole idea that a) users can control database file names and b) the server stupidly deletes files by extension wasn't a really clever idea on day one and hasn't got a whole lot smarter since.

If the philosophy is to defer to storage engines, then defer to storage engines and stop trying to do them favors.

[And this strong opinion is not strictly from theory.]

--
Jim Starkey
President, NimbusDB, Inc.
978 526-1376


_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

--
_______________________________________________________
Brian "Krow" Aker, brian at tangent.org
Seattle, Washington
http://krow.net/                     <-- Me
http://tangent.org/                <-- Software
_______________________________________________________
You can't grep a dead tree.




_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

Reply via email to