[01:07:22] <grrrit-wm>	 (PS1) Bmansurov: Add support for timezones while creating reports [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/175169 (https://bugzilla.wikimedia.org/72116)
[01:08:27] <grrrit-wm>	 (CR) Bmansurov: "It's a hacky solution. Maybe we I should do it some other way? Any suggestions?" [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/175169 (https://bugzilla.wikimedia.org/72116) (owner: Bmansurov)
[01:15:38] <grrrit-wm>	 (PS2) Bmansurov: Add support for timezones while creating reports [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/175169 (https://bugzilla.wikimedia.org/72116)
[06:05:56] <rtnpro>	 YuviPanda|zzzz, ping
[06:06:03] <YuviPanda|zzzz>	 rtnpro: pong
[06:06:06] <YuviPanda|zzzz>	 good morning
[06:06:13] <rtnpro>	 YuviPanda|zzzz, Good morning :)
[06:06:27] <rtnpro>	 YuviPanda|zzzz, I thought we were sleeping :P
[06:06:37] <rtnpro>	 s/we/you/g
[06:07:40] <rtnpro>	 YuviPanda|zzzz, I had some query regarding this line: https://github.com/wikimedia/analytics-quarry-web/blob/master/quarry/web/app.py#L292
[06:07:53] <rtnpro>	 YuviPanda|zzzz, AFAIK, "joins" are bad
[06:08:59] <rtnpro>	 YuviPanda|zzzz, when the tables grow, this is gonna have its toll
[06:09:09] <YuviPanda>	 indeed
[06:09:16] <YuviPanda>	 but they haven't grown yet
[06:09:18] <YuviPanda>	 so it's ok :)
[06:09:23] <rtnpro>	 YuviPanda|zzzz, ok
[06:09:24] <YuviPanda>	 don't want to denormalize yet
[06:09:31] <rtnpro>	 YuviPanda|zzzz, ok
[06:09:42] <YuviPanda>	 and denormalizing is the way to fix it
[06:09:57] <rtnpro>	 YuviPanda|zzzz, yes
[06:10:24] <rtnpro>	 YuviPanda|zzzz, since, READs are way lot greater than WRITEs, we need to optimize it that way
[06:10:48] <YuviPanda>	 true, but we can do that when we hit a few hundred thousand records, I think
[06:11:06] <rtnpro>	 YuviPanda|zzzz, yes, I agree
[06:11:11] <rtnpro>	 YuviPanda|zzzz, also, regarding the pagination for all queries page, I have used the following URL pattern
[06:11:47] <rtnpro>	 YuviPanda|zzzz, /query/runs/all?from=<query_id>&direction=<prev|next>
[06:12:02] <YuviPanda>	 I think just having 'next' is ok
[06:12:03] <YuviPanda>	 ?
[06:12:32] <rtnpro>	 YuviPanda|zzzz, I do not follow
[06:12:45] <YuviPanda>	 as in
[06:12:59] <YuviPanda>	 ?from=query_id&limit=<number>
[06:13:01] <rtnpro>	 YuviPanda|zzzz, 'next' in place of 'direction'?
[06:13:32] <YuviPanda>	 as in, don't support 'previous'
[06:13:33] <rtnpro>	 YuviPanda|zzzz, 'next' is always on by default
[06:13:37] <rtnpro>	 ok
[06:13:38] <rtnpro>	 ok
[06:13:42] <YuviPanda>	 why support previous?
[06:14:16] <rtnpro>	 YuviPanda|zzzz, that's what pagination is for, you need to be able to go backwards and forwards
[06:14:49] <rtnpro>	 YuviPanda|zzzz, it will just involve one more line
[06:14:53] <YuviPanda>	 rtnpro: you go backwards by knowing the query_id of the page that was 'previous'
[06:15:00] <YuviPanda>	 rtnpro: but hmm, if this is simpler, sure.
[06:16:23] <rtnpro>	 YuviPanda|zzzz, so, let's finalize on this URL pattern: query/runs/all?from=<query_id>&limit=<number>&prev
[06:16:44] <YuviPanda>	 how about
[06:16:47] <YuviPanda>	 just setting negative limit
[06:16:50] <YuviPanda>	 to go prev?
[06:17:16] <rtnpro>	 YuviPanda|zzzz, hmm, makes sense
[06:17:18] <rtnpro>	 +1
[06:17:31] <YuviPanda>	 :)
[06:19:58] <rtnpro>	 YuviPanda|zzzz, I still don't have access for: https://www.mediawiki.org/wiki/Special:OAuthConsumerRegistration/propose :(
[06:20:09] <YuviPanda>	 rtnpro: bah.
[06:20:27] <YuviPanda>	 rtnpro: send me your email, I will send you some test credentials
[06:20:54] <rtnpro>	 YuviPanda, it's rtnpro AT gmail DOT com
[06:22:44] <YuviPanda>	 rtnpro: sent
[06:22:45] <YuviPanda>	 try those?
[06:22:58] <rtnpro>	 YuviPanda, thanks :)
[06:23:17] <YuviPanda>	 rtnpro: tell me if it does work, tho
[06:23:35] <rtnpro>	 YuviPanda, ok
[06:28:21] <rtnpro>	 YuviPanda, it worked :)
[06:28:40] <YuviPanda>	 yay
[06:28:43] <YuviPanda>	 cool\