* State of the erasure code pyramid plugin
@ 2014-01-30 8:04 Loic Dachary
0 siblings, 0 replies; only message in thread
From: Loic Dachary @ 2014-01-30 8:04 UTC (permalink / raw)
To: Andreas-Joachim Peters; +Cc: Ceph Development
[-- Attachment #1: Type: text/plain, Size: 1219 bytes --]
Hi,
Your last comment made me realize I took an approach that is more complex than needed (the decode_chunks / encode_chunks methods are not necessary), which results in something that is also sub-optimal (it decodes too much and has no way to limit decoding to what is strictly necessary although this is precisely the point of the pyramid plugin). I see how to resolve it (rewrite recursion in terms of the encode()/decode() API instead of the encode_chunks()/decode_chunks() API and I'm half way thru). But since the feature freeze is Friday and given than it's not the only topic I care about, it is probably better to postpone.
I'm quite happy that we now have a sound implementation plan. Even more that the current API is good enough. What remains to be done is:
* Rewrite of https://github.com/ceph/ceph/pull/1142 with encode()/decode()
* Complete the XOR plugin https://github.com/apeters1971/ceph/tree/wip-xor
* Implement a way ( moving the primary in the same datacenter as a failing OSD ? ) for pools to take advantage of the provided locality
I reorganized the tasks under http://tracker.ceph.com/issues/7266 to keep track of this.
Cheers
--
Loïc Dachary, Artisan Logiciel Libre
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 263 bytes --]
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2014-01-30 8:04 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-30 8:04 State of the erasure code pyramid plugin Loic Dachary
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.