Re: [Sks-devel] Re: [PATCH] Proper case handling for words index

[Kim Minh Kaplan]

Jason Harris:

> I would suggest the output go to its own logfile, like update_subkeys:
>
>   update_subkeys.ml:121:  set_logfile "update_subkeys";
>
> and be verbose by default (assuming I got my extra output because I set
> "debuglevel: 5" in sksconf):

This fixup script does not really deserve to be maintained.  Once the
database is fixed its of no more use.  I hope we will quickly forget
about it.

> I removed Dbenv.RECOVER from my flags a long time ago and running
> ./fix_word_index in parallel with "sks db" seemed to work fine.

Nice.  By the way does removing Dbenv.RECOVER permit you to dump your db
live, without stopping the sks server?

> Deadlock can still easily occur, however, since SKS doesn't yet do
> full deadlock setup/handling in the BDB API, IIRC.

Right.  In this case one would have to launch it again.  Mmm, I am not
sure a deadlock would be autodected.  OK, you should stop sks db while
running this script.  Or run the Berkeley DB deadlock detection utility.

> BTW, I would like to pull these patches directly from your repo. for
> better tracking.  Thanks.

I am very hesitant to fork SKS.  But as Yaron Minsky has been very quiet
these last months it could be the only solution.  What do people think
about this here?

Kim Minh.