On Wed, 5 Aug 2015, Loic Dachary wrote: > Hi Sam, > > How does this proposal sound ? It would be great if that was done before > the feature freeze. I think it's a good time. Takeshi, note that what this really means is that the on-disk encoding needs to remain fixed. If we decide to change it down the line, we'll have to make a 'shec2' or similar so that the old format is still decodable (or ensure that existing data can still be read in some other way). Sound good? sage > > Cheers > > On 29/07/2015 11:16, Loic Dachary wrote: > > Hi Sam, > > > > The SHEC plugin[0] has been running in the rados runs[1] in the past few months. It also has a matching corpus verification which runs on every make check[2] as well as its optimized variants. I believe the flag "experimental" can now be removed. > > > > In order to do so, we need to use a PLUGINS_V3 feature, in the same way we did back in Giant when the ISA and LRC plugins were introduced[3]. This won't be necessary in the future, when there is a generic plugin mechanism, but right now that's what we need. It would be a commit very similar to the one implementing PLUGINS_V2[4]. > > > > Is this agreeable to you ? Or would you rather see another way to resolve this ? > > > > Cheers > > > > [0] https://github.com/ceph/ceph/tree/master/src/erasure-code/shec > > [1] https://github.com/ceph/ceph-qa-suite/tree/master/suites/rados/thrash-erasure-code-shec > > [2] https://github.com/ceph/ceph-erasure-code-corpus/blob/master/v0.92-988/non-regression.sh#L52 > > [3] http://tracker.ceph.com/issues/9343 > > [4] https://github.com/ceph/ceph/commit/9687150ceac9cc7e506bc227f430d4207a6d7489 > > > > -- > Loïc Dachary, Artisan Logiciel Libre > >
