Tom Lane wrote:
Heikki Linnakangas <[EMAIL PROTECTED]> writes:
Here's a fix for the problem that on a busy system, mdsync never finishes. See the original problem description on hackers:


This leaks memory, no?  (list_delete_cell only deletes the ListCell.)

Oh, I just spotted another problem with it and posted an updated patch, but I missed that.

But I dislike copying the table entries anyway, see comment on -hackers.

Frankly the cycle id idea sounds more ugly and fragile to me. You'll need to do multiple scans of the hash table that way, starting from top every time you call AbsorbFsyncRequests (like we do know). But whatever...

BTW, it's very hard to see what a patch like this is actually changing.
It might be better to submit a version that doesn't reindent the chunks
of code you aren't changing, so as to reduce the visual size of the
diff.  A note to the committer to reindent the whole function is
sufficient (or if he forgets, pg_indent will fix it eventually).

Ok, will do that. Or would you like to just take over from here?

--
  Heikki Linnakangas
  EnterpriseDB   http://www.enterprisedb.com

---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to