{"push_id":2093809935,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"63ee43353bd3b753beb0233f64865e658abeb1c3","before":"4dc90f0214f784245226b3445cd7dbf12f51604b","commits":[{"sha":"63ee43353bd3b753beb0233f64865e658abeb1c3","author":{"name":"Fernando Seror","email":"9d1c84d67c1d37e657203757907a8b6df42111d0@gmail.com"},"message":"JobSet#each goes through elements in descending score order\n\nThe current implementation of the #each method uses Redis.zrange to\npaginate the iteration and use multiple lightweight calls. It performs\nthis pagination in descending score order, but each page is returned\nfrom Redis in ascending order. The result is that the final iteration\nthrough the whole set is not sorted properly. Here's an example with a\npage of size 3:\n\nRedis set: 1, 2, 3, 4, 5, 6, 7, 8, 9\nJobSet.to_a: 7, 8, 9, 4, 5, 6, 1, 2, 3\n\nThis fixes it with barely no performance cost (each page is reverted in\nRuby) and all the items are perfectly sorted in descending score order.","distinct":true,"url":"https://api.github.com/repos/mperham/sidekiq/commits/63ee43353bd3b753beb0233f64865e658abeb1c3"}]}
{"actor":{"display_login":"mperham"}}