Pull Request Reviews
Ralph Bean
rbean at redhat.com
Thu Aug 22 13:04:03 UTC 2013
Hi infra team-
We have a handful of pull requests open on our github repo for which
I'd like to ask for help in reviewing:
1) fedbadges 4d PR#22 - Allow lambda expressions in the dat .. http://bit.ly/1aeY7qh
2) tahrir-api 4d PR#19 - Feature/ranking .. http://bit.ly/1dhjI0G
3) fedbadges 3d PR#23 - Publish a fedmsg message when a use .. http://bit.ly/14UUlkP
4) tahrir 3d PR#199 - Feature/publish message on rank cha .. http://bit.ly/14sVXfq
5) fedmsg 2d PR#173 - Feature/idempotent .. http://bit.ly/1643CQX
-
Number 1 is an independent change and is an enhancement to the badges
awarder that will make it so we can perform much more flexible
datanommer queries. (It makes it so we can do flexible operations in
python after making the actual datanommer query but before we
conclude that a badge should or should not be awarded).
Number 2 is a requirement for numbers 3 and 4. It does a number of things:
- it pulls the leaderboard calculation code from tahrir (the web
frontend) into tahrir-api (the underlying lib used by both the
badges awarder backend and the web frontend)
- it caches users' rank in the database which should improve
performance on the webapp significantly.
- it adds a new `notification_callback` which we'll use to publish
fedmsg messages about users' rank changing. We want to award
badges based on this kind of stuff, so its a nice bonus.
Number 3 simply makes use of Number 2. It removes the old hardcoded
fedmsg stuff from the backend badges awarder.
Number 4 simply makes use of Number 2. It removes the old leaderboard
code and the old hardcoded fedmsg stuff from the badges web frontend.
Number 5 is an independent change that will ultimately fix a bug in
the datagrepper web api.
-
Any help providing review would be much appreciated.
Cheers-
-Ralph
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.fedoraproject.org/pipermail/infrastructure/attachments/20130822/581098c5/attachment.sig>
More information about the infrastructure
mailing list