couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mark Hammond (JIRA)" <>
Subject [jira] Commented: (COUCHDB-86) (CouchDB on Windows) compaction can not be done.
Date Sun, 07 Feb 2010 23:31:28 GMT


Mark Hammond commented on COUCHDB-86:

Thanks very much for running with this.  I had a bit of a play over the weekend.  First, a
few comments about the patch itself:

* The force_close_file changes in couch_file seem to be unused and should be deleted.
* force_close_file in couch_db seems to be poorly named - it actually calls couch_file:delete
and best I can tell, makes no attempt at closing the file at all.
* couch_file:delete now seems to be mis-named - it only does a deletion on non-Windows platforms.
 I wonder if the logic for checking Windows shouldn't just go in couch_db, leaving couch_file
with less magic and doing what people would expect it to do (ie, expect couch_file:delete
to actually delete and may be surprised at the new behaviour.
* Some of the functions named in couch_file should probably be renamed, even if they get longer
names.  eg, delete_all implies from the name if will delete all files from some location,
when what it really does it delete all "versioned" files given a base-name.  Ditto next_filepath
- that problably wants a name closer to 'next_versioned_filepath'
* I'm very inexperienced with erlang, but wouldn't it be better to use linked processes rather
than using process_flag(trap_exit,true)?

Also, the futon tests get further, but still fail for me.  The couch log shows:

1> [info] [<0.1606.0>] - - 'GET' /test_suite_db/_changes?filter=changes_filter/conflicted
=INFO REPORT==== 8-Feb-2010::10:23:32 ===
1> [info] [<0.1609.0>] - - 'DELETE' /test_suite_db/ 200
1> [info] [<0.1606.0>] - - 'PUT' /test_suite_db/ 412
1> [error] [<0.1622.0>] ** Generic server <0.1622.0> terminating
** Last message in was {'EXIT',<0.1626.0>,shutdown}
** When Server state == {file,{file_descriptor,prim_file,{#Port<0.4173>,712}},
** Reason for termination ==
** {{badmatch,{error,eacces}},

While futon reports # Exception raised: {"error":"file_exists","reason":"The database could
not be created, the file already exists."}

On the plus side, many tests now pass which failed before, and no stale 'versioned' files
appear to have been left behind :)

> (CouchDB on Windows) compaction can not be done.
> ------------------------------------------------
>                 Key: COUCHDB-86
>                 URL:
>             Project: CouchDB
>          Issue Type: Bug
>          Components: Build System
>    Affects Versions: 0.8
>         Environment: Windows XP,    Erlang/OTP R12B-3
>            Reporter: Li Zhengji
>            Assignee: Paul Joseph Davis
>            Priority: Blocker
>             Fix For: 1.0
>   Original Estimate: 5h
>  Remaining Estimate: 5h
> During compacting, rename the current DB file to a .old file is not allowed on Windows.
> A possible workaround for this could be: 
> 1. Close current DB file (.couch);
> 2. Send db_updated to update to use .compact;
> 3. After 5sec, delete the .couch file;  ---- This is done in a linked process, after
that, this process send a message to update_loop;
> 4. After received the message in update_loop, close current DB file which is a .compact
file, then rename it to .couch;
> 5. Finally, db_updated again to use this new .couch file.
> Maybe, there would be a "pause" in service?

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message