Closed fanantoxa closed 8 years ago
@soulcutter Updated code with your additions. Please check.
Looks good to me.
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ Benchmark with :nokogiri parser @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ user system total real for_tag 6.100000 0.040000 6.140000 ( 6.162233) at_depth 4.830000 0.030000 4.860000 ( 4.868037) within 6.180000 0.040000 6.220000 ( 6.222976) composite 4.660000 0.020000 4.680000 ( 4.698063) ########################################################## for_tag: 100000 artist elements parsed at_depth: 400000 elements parsed within: 100000 artists children parsed composite: 0 names within artist nested 3 tags deep parsed @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ Benchmark with :ox parser @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ user system total real for_tag 4.000000 0.030000 4.030000 ( 4.035326) at_depth 2.940000 0.020000 2.960000 ( 2.961429) within 3.920000 0.020000 3.940000 ( 3.951661) composite 2.330000 0.020000 2.350000 ( 2.352623) ########################################################## for_tag: 100000 artist elements parsed at_depth: 400000 elements parsed within: 100000 artists children parsed composite: 0 names within artist nested 3 tags deep parsed
I've finished fir performance update. Benchmarks from master:
Benchmarks with changes on this PR:
@soulcutter I've got some performance increase, but I'm not sure that my environment was good enough prepared for benchmark. Please checn it on you PC. And if works faster - please merge it.