On Wed, 11 Mar 2015, Ning Yao wrote: > Can we also consider to coalesce two OP_SETATTR transaction to a > single OP_SETATTRS transaction?
Yeah, definitely! sage > Regards > Ning Yao > > > 2015-03-05 15:04 GMT+08:00 Haomai Wang <haomaiw...@gmail.com>: > > I think the performance improvement can be refer to > > https://github.com/ceph/ceph/pull/2972 which I did a simple benchmark > > comparison. > > > > This coalescing should get a lighter improvement but I think obviously > > it should be better. > > > > On Thu, Mar 5, 2015 at 8:10 AM, Sage Weil <s...@newdream.net> wrote: > >> On Tue, 3 Mar 2015, Sage Weil wrote: > >>> I think we should try to avoid the dups in the first place in the upper > >>> layers before resorting to deduping in the FileStore. > >> > >> I took a stab at this: > >> > >> https://github.com/ceph/ceph/pull/3878 > >> > >> Here's what the txns look like now: > >> > >> { > >> "ops": [ > >> { > >> "op_num": 0, > >> "op_name": "omap_setkeys", > >> "collection": "0.2_head", > >> "oid": "2\/\/head\/\/0", > >> "attr_lens": { > >> "0000000005.00000000000000000002": 178, > >> "_epoch": 4, > >> "_info": 729, > >> "can_rollback_to": 12, > >> "rollback_info_trimmed_to": 12 > >> } > >> }, > >> { > >> "op_num": 1, > >> "op_name": "write", > >> "collection": "0.2_head", > >> "oid": > >> "8b08fa52\/benchmark_data_maetl_15609_object5\/head\/\/0", > >> "length": 123, > >> "offset": 0, > >> "bufferlist length": 123 > >> }, > >> { > >> "op_num": 2, > >> "op_name": "setattr", > >> "collection": "0.2_head", > >> "oid": > >> "8b08fa52\/benchmark_data_maetl_15609_object5\/head\/\/0", > >> "name": "_", > >> "length": 269 > >> }, > >> { > >> "op_num": 3, > >> "op_name": "setattr", > >> "collection": "0.2_head", > >> "oid": > >> "8b08fa52\/benchmark_data_maetl_15609_object5\/head\/\/0", > >> "name": "snapset", > >> "length": 31 > >> } > >> ] > >> } > >> > >> Care to give that a spin and see if it give syou a similar (or better) > >> improvement? > >> > >> sage > >> -- > >> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > >> the body of a message to majord...@vger.kernel.org > >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > > > > > -- > > Best Regards, > > > > Wheat > > -- > > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > > the body of a message to majord...@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html