From: Peter Braam <Peter.Braam@Sun.COM>
To: lustre-devel@lists.lustre.org
Subject: [Lustre-devel] Commit on share
Date: Tue, 27 May 2008 18:44:18 +0800 [thread overview]
Message-ID: <C4620704.5499%peter.braam@sun.com> (raw)
This HLD is definitely not ready at all. It is very short, lacks
interaction diagrams and the arguments made are not sufficiently detailed.
* the second sentence is not right. Commit should happen before
un-committed data coming from a client is shared with a 2nd client.
* Is COS dependent on VBR ? no it is not, and can equally apply to normal
recovery
* Section 3.2 is wrong: the recovery process will not fail with gaps in the
sequence when there is VBR. It only fails if there are gaps in the
versions, and this is rare.
* 3.3 parallel creations in one directory are protected with different,
independent lock resources. Isn?t that sufficient to allow parallel
operations with COS?
* 3.6 provide a detailed explanation please
* GC thread is wrong mechanism this is what we have commit callbacks for
* Why not use the DLM, then we can simply keep the client waiting ? the
mechanism already exists for repack; I am not convinced at all by the
reasoning that rep-ack is so different ? no real facts are quoted
* It is left completely without explanation how the hash table (which I
think we don?t need/want) is used
Regards,
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lustre.org/pipermail/lustre-devel-lustre.org/attachments/20080527/11a2ef8f/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: commit-on-sharing-simple-tracking-hld.pdf
Type: application/octet-stream
Size: 63433 bytes
Desc: not available
URL: <http://lists.lustre.org/pipermail/lustre-devel-lustre.org/attachments/20080527/11a2ef8f/attachment.obj>
next reply other threads:[~2008-05-27 10:44 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-27 10:44 Peter Braam [this message]
2008-05-27 15:43 ` [Lustre-devel] Commit on share Mikhail Pershin
2008-06-01 5:00 ` Peter Braam
2008-05-29 17:42 ` Mikhail Pershin
2008-05-31 2:45 ` Andreas Dilger
2008-05-31 9:37 ` Alex Zhuravlev
2008-06-01 7:03 ` Mikhail Pershin
2008-06-03 18:41 ` Andreas Dilger
2008-06-03 18:56 ` Alex Zhuravlev
2008-06-01 16:54 ` Alex Zhuravlev
2008-06-02 8:42 ` Alex Zhuravlev
2008-06-03 18:50 ` Andreas Dilger
2008-06-04 1:11 ` Peter Braam
2008-06-04 10:50 ` Nikita Danilov
2008-06-11 14:21 ` Alexander Zarochentsev
2008-06-11 14:35 ` Alex Zhuravlev
2008-06-11 15:26 ` Peter Braam
2008-06-11 16:27 ` Alex Zhuravlev
2008-06-11 16:28 ` Peter Braam
2008-06-11 16:46 ` Alexander Zarochentsev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=C4620704.5499%peter.braam@sun.com \
--to=peter.braam@sun.com \
--cc=lustre-devel@lists.lustre.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.