Last modified: 2014-03-06 10:57:09 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T55987, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53987 - sulinfo is unusable (takes tens of seconds)
sulinfo is unusable (takes tens of seconds)
Status: VERIFIED FIXED
Product: Tool Labs tools
Classification: Unclassified
Quentinv57's tools (Other open bugs)
unspecified
All All
: Lowest major
: ---
Assigned To: Cyberpower678
:
Depends on: 55929 58890
Blocks: ts-migration
  Show dependency treegraph
 
Reported: 2013-09-10 08:55 UTC by Nemo
Modified: 2014-03-06 10:57 UTC (History)
12 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Nemo 2013-09-10 08:55:22 UTC
This tool is a must have, it has to work; it's even in the [[m:interwiki map]] (of course in the Toolserver version, which works perfectly).

$ curl http://tools.wmflabs.org/sulinfo/sulinfo.php?username=AnankeBot > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 34812    0 34812    0     0    833      0 --:--:--  0:00:41 --:--:--  9820

https://toolserver.org/~quentinv57/sulinfo/AnankeBot takes only 1-2 s.

Surely related to http://lists.wikimedia.org/pipermail/labs-l/2013-September/001572.html but better to address one tool at a time.
Comment 1 Andre Klapper 2013-09-11 19:39:26 UTC
Nemo: Why is this under the "Other" component instead of "tools" (or "Tool labs tools" product) if it's hosted on tools.wmflabs.org/* ?
Comment 2 Andre Klapper 2013-09-11 21:14:33 UTC
[Moving to "Tool Labs tools" as this is about a specific tool and not the general Wikimedia Tool Labs infrastructure. Petr and Yuvi are listed as maintainers on http://tools.wmflabs.org/ and hence CC'ed.]

If this requires fixing on an Labs Infrastructure level, please move to "Wikimedia Labs > tools". Thanks!
Comment 3 Yuvi Panda 2013-09-11 21:21:42 UTC
(I've no idea why I'm listed as maintainer)
Comment 4 Nemo 2013-09-13 05:19:30 UTC
(In reply to comment #2)
> If this requires fixing on an Labs Infrastructure level, please move to
> "Wikimedia Labs > tools". Thanks!
Comment 5 Nemo 2013-09-13 05:23:00 UTC
As I said, this is almost surely the same problem as in http://lists.wikimedia.org/pipermail/labs-l/2013-September/001572.html
A tool is 30 times slower on TL than on TS: until an alternative implementation/actionable solution is suggested to the maintainer, I consider this a bug in TL.

(In reply to comment #3)
> (I've no idea why I'm listed as maintainer)

Ah, sorry. No idea how to get that fixed.
Comment 6 MZMcBride 2013-09-14 13:30:48 UTC
(In reply to comment #0)
> This tool is a must have [...]

We already have [[m:Special:CentralAuth/Nemo_bis]]. If Special:CentralAuth is insufficient, I think we should focus on making it better rather than waste time porting tools between non-production hosts. If this is a must have, as you say.
Comment 7 Nemo 2013-09-14 22:00:21 UTC
(In reply to comment #6)
> (In reply to comment #0)
> > This tool is a must have [...]
> 
> We already have [[m:Special:CentralAuth/Nemo_bis]]. If Special:CentralAuth is
> insufficient, I think we should focus on making it better rather than waste
> time porting tools between non-production hosts. If this is a must have, as
> you
> say.

Feel free to propose a redesign of the special page and/or to submit patches. Tools are not going to disappear.
Comment 8 Marc A. Pelletier 2013-09-18 00:13:11 UTC
The 26ms of overhead it takes to have a query result cannot be circumvented; it is the actual network roundtrip between the database replicas and where the tool labs is currently located (in different data centers).

Moving tool labs to eqiad (where the replicas live) is on the roadmap for 4Q 2013, but in the meantime there is - literally - nothing that can be done to improve this short of increasing the speed of light.

That said, tools can exploit parallelism of the databases to good effect to circumvent the problem (i.e.: selecting from several databases in one query), or simply make the queries asynchronously (the results will still take 26ms to get back, but if you do them simultaneously rather than sequentially that is not an issue anymore).

I'm closing as "WONTFIX" since there isn't a "Unfixable-in-the-current-setup-but-will-not-be-an-issue-once-labs-moved" resolution.  :-)
Comment 9 Nemo 2013-09-18 05:53:59 UTC
(In reply to comment #8)
> I'm closing as "WONTFIX" since there isn't a
> "Unfixable-in-the-current-setup-but-will-not-be-an-issue-once-labs-moved"
> resolution.  :-)

I think --- is the best resolution for "should be fixed at some point but we'll need to check".
Comment 10 Andre Klapper 2013-09-18 10:40:54 UTC
Setting lowest priority, then.
Comment 11 Nemo 2013-10-13 09:48:25 UTC
In Toolserver top50 with about 16k requests per day, for the records.
http://lists.wikimedia.org/pipermail/toolserver-l/2013-October/006337.html
That's about 20 % of September 2013 Tool Labs hits.
http://tools.wmflabs.org/awstats/cgi-bin/awstats.pl?month=09&year=2013&output=urldetail&config=tools.wmflabs.org&framename=index
Comment 12 Marc A. Pelletier 2013-10-22 13:23:15 UTC
Regardless of popularity, this cannot be made faster as it is currently written before labs move save by changing the laws of physics.

The tools makes ~1000 database queries serially; unless and until it is made to do them with at least some parallelism, it will remain highly dependent on latency.
Comment 13 MZMcBride 2013-10-22 13:46:14 UTC
I hadn't realized until recently that [[m:Special:CentralAuth/Nemo_bis]] now includes an edit count column. What additional functionality is needed before we can point the "sulinfo" interwiki prefix to "Special:CentralAuth"?
Comment 14 Nemo 2013-10-22 22:34:50 UTC
(In reply to comment #12)
> Regardless of popularity, this cannot be made faster as it is currently
> written
> before labs move save by changing the laws of physics.

Platonides already pointed out on the mailing lists some solutions which require far less than breaking laws of physics.
Comment 15 MZMcBride 2013-10-23 01:52:08 UTC
(In reply to comment #13)
> I hadn't realized until recently that [[m:Special:CentralAuth/Nemo_bis]] now
> includes an edit count column. What additional functionality is needed before
> we can point the "sulinfo" interwiki prefix to "Special:CentralAuth"?

This should read "sulutil" where it says "sulinfo". (I could've sworn it was "sulinfo"....)

I've formally proposed this change at [[m:Special:Permalink/6129569#sulutil]] if anyone following this bug is interested.
Comment 16 MZMcBride 2013-10-23 02:05:26 UTC
(In reply to comment #12)
> Regardless of popularity, this cannot be made faster as it is currently
> written before labs move save by changing the laws of physics.

I'm reminded of bug 55929.
Comment 17 MZMcBride 2013-10-23 22:09:36 UTC
The Toolserver tool doesn't seem to be doing so great... when I tried to access <https://toolserver.org/~quentinv57/sulinfo/Nemo_bis> earlier today, it took 9m30.186s for curl to finish. I just tried again and it took 15m47.230s. :-/
Comment 18 Marc A. Pelletier 2013-12-24 14:19:32 UTC
*** Bug 58890 has been marked as a duplicate of this bug. ***
Comment 19 Cyberpower678 2014-01-04 00:25:42 UTC
For the record, quentinv57's tool has been migrated to https://tools.wmflabs.org/quentinv57-tools/tools/sulinfo.php
Comment 20 Cyberpower678 2014-01-04 00:26:18 UTC
(In reply to comment #19)
> For the record, quentinv57's tool has been migrated to
> https://tools.wmflabs.org/quentinv57-tools/tools/sulinfo.php

But it is also suffering from the latency issue.
Comment 21 Cyberpower678 2014-01-04 00:27:16 UTC
(In reply to comment #20)
> (In reply to comment #19)
> > For the record, quentinv57's tool has been migrated to
> > https://tools.wmflabs.org/quentinv57-tools/tools/sulinfo.php
> 
> But it is also suffering from the latency issue.

This tool is highly depended by ACC, so I was wondering when this can be resolved?

Sorry for constantly posting btw.
Comment 22 Cyberpower678 2014-03-05 18:42:09 UTC
It's been highly optimized and now also moved to eqiad.  Shouldn't take more that a few seconds to load SUL profiles.  High profile users may take a bit longer but the max I got for such users was 5-6 seconds.
Comment 23 Cyberpower678 2014-03-05 18:43:14 UTC
I'm now closing this as fixed.
Comment 24 Nemo 2014-03-06 10:57:09 UTC
Thanks!

$ curl https://tools.wmflabs.org/quentinv57-tools/tools/sulinfo.php?username=SieBot > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  100k    0  100k    0     0  33092      0 --:--:--  0:00:03 --:--:-- 33109

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links