YahooArchive / boomerang

End user oriented web performance testing and beaconing
http://lognormal.github.com/boomerang/doc/
Other
1.18k stars 447 forks source link

Adapting sendBeacon function for customizing output format of the request #21

Open ninodafonte opened 13 years ago

ninodafonte commented 13 years ago

Refactoring of the sendBeacon function for having the option for customizing formatResponse inside plugins.

This open the posibilities for modifying the output format and adapt it to the required response.

I'm trying to keep the gathering of statistical data as fast as possible and the goal is to insert the data in some noSql engine or Hadoop framework with a single request letting me make some post-processing. To do that, the url has to be in a specific format.

That's the reason for customizing the url format. I hope this could be helpful for another purposes.

Best regards, thank you very much.

pd: sorry for the bunch of line changes. My editor kills spaces and change them for tabs. I just wrote a comment to make the pull request more clear.

bluesmoon commented 12 years ago

I think maybe this should be split into two. The StatsPlus plugin which only adds the new parameters will be one part. For the second part, we'll need to look at making this into a MVC pattern, where the formatter would be the view.

ninodafonte commented 12 years ago

Yes, you are right, I'll try to split this functionality as you say and i'll be back in a bit

thank you Philip!

bluesmoon commented 9 years ago

Just wanted to let you know that this project is now at lognormal/boomerang, so would be good if you made your changes against that repository.