Hey all! First off, love quarry, much shinier than my JSON-based backdoor server I'd run from time to time. While importing some of my favorite queries (e.g., http://quarry.wmflabs.org/query/1778) I was somewhat surprised to see that the resultset displayed below was not respecting the ORDER BY field, instead being sorted by the very first field. The CSV data I downloaded did, however, so I'm hopeful that the displayed data will follow suit. :)
Thanks!
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Do not sort by first column by default | analytics/quarry/web | master | +2 -1 |
Change 222775 had a related patch set uploaded (by Yuvipanda):
Do not sort by first column by default
Just fixed it :) Thanks for reporting the issue, and apologies for such late resolution.
Awesome dude. Appears to be working on my end from the queries I tried. And hey, believe me, I know a thing or two about software development processes and associated delays, and it's always better late than never ;) Much appreciated.
:) part of the problem was the lack of a local dev environment - I just fixed that problem yesterday (vagrant!) - fixing things should be easier now :)
Yuvi I hope you documented somewhere how one can set up Quarry on vagrant. Quarry has a tone of dependencies, so having that documentation somewhere would really help.
PS: (and sorry for spamming, I just realized I cannot edit this task anymore) I think you should remove the "patch for review" once the issue is fixed.
@Huji: Just running 'vagrant up' on a quarry checkout works :) I'll write up a README soon