Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The number of reported installs on the project page, as in "Reported installs: 1,722 sites currently report using this module." is the number for last week (as seen on https://drupal.org/project/usage/MODULE), not the current week.
Comment | File | Size | Author |
---|---|---|---|
#16 | 2129811_fix-undefined-db_affected_rows.patch | 1.66 KB | jthorson |
Comments
Comment #1
kreynen CreditAttribution: kreynen commentedComment #2
tvn CreditAttribution: tvn commentedComment #3
a.ross CreditAttribution: a.ross commentedLooks like it isn't updated at all.
Comment #4
rjacobs CreditAttribution: rjacobs commentedSetting #2128619: Project usage/download/install statistics not being generated in a timely manner as "parent" issue as that currently seems most logical.
Comment #5
RdeBoer@rjacobs, #4:
That's ok, but it may not be the correct causal relationship I'm seeing. The statistics graph DOES get updated (albeit days after for some modules), but the "reported installs count" on the project page itself is frozen on 20 Oct 2013.
Rik
Comment #6
rjacobs CreditAttribution: rjacobs commentedAgreed. I guess I was thinking more on a conceptual level than a technical one, since the tech specifics are still not clear. #2128619: Project usage/download/install statistics not being generated in a timely manner is has become something of a "master" description of all the outdated update stat issues, and until it's clear which sub-issues are dependent on others (at a technical level) I was just trying to tell the right story for user's who land on #2128619. In this case I just wanted to get the parent/child/related box to align with the issue description (and it seemed that this issue was spawned from that one).
In the case of these d.o. (site-specific upgrade) issue queues I'm honestly not sure what the process is. The proper analysis for some of this stuff seems to require special system/code access that I guess only d.o. site admin have. I figured that the best way I could contribute would be to just try and enhance some of the summary info for now.
Comment #7
davidmac CreditAttribution: davidmac commentedI don't think the previous description of 'not being updated in a timely manner' can hold when we are seeing projects where the reported installs and downloads are not updated at all (as stated in #3 on Nov 21st). The timely manner relates more to the statistics 'graph page' which has seen varying degrees of lag in updates and is dealt with under issue #2128619.
There is also the added irk that the 'last updated' date in this section of the project page seems to be 'updating' whilst the reported statistics have been static for over a month now.
Given the ongoing problem, and the relationship of this issue to https://drupal.org/node/2128619 which is marked as 'major', I've amended its priority to bring them in line with each other.
Comment #8
RdeBoerCould not agree more, David!
While stats for the module as a whole still seem to tick along (in the graphs only), any RELEASE-SPECIFIC data for releases after 20 Oct isn't on the project page or anywhere.
An important special case of this is that any new modules released after 20 Oct 2013 have NO download stats or usage stats at all.
Newbie developers as well as old developers with newbie modules thrive on this sort of initial feedback.
Please don't deprive them!
Stats are important in creating early-adaptor momentum. Not having stats could kill off a module (and its maker's spirit)....
Especially sad when it is a really great module!
Comment #9
davidmac CreditAttribution: davidmac commentedI agree Rik, the importance of these statistics to newly released modules can't be underestimated. I have, in the past, seen comments where people don't want to use a particular module because it's usage statistics are low, and I have had to point out that it's a new module. Delaying stats on new modules can be damaging to their perceived popularity/quality. It is also very disheartening for the developer and will make people think twice about putting the time and effort into developing further modules.
I would prefer to have this section of the project page hidden until it is fixed, thereby forcing people to look at the graph (which I suspect remains buggy). Presumably this could be done easily enough?
What would be really useful would be some feedback as to what caused these issues, what are the likely timescales/obstacles to fixing them. I also think that issues which remain unresolved for longer and/or which effect a very wide user-base should receive higher priority.
Any feedback from the Drupal.org upgrade/webmaster team would be welcome.
Comment #10
davidmac CreditAttribution: davidmac commentedSome summarising across issues: https://drupal.org/comment/8255729#comment-8255729
Comment #11
drummDownloads should be solved with #2133385: project_generate_download() query needs porting to D7 structure. That code is committed and running now. It took 15 minutes on D6; D7 may be slower or faster. If downloads don't update soon, that issue, or the parent issue should be updated.
Comment #12
drummThe key problem is
Error: Call to undefined function db_affected_rows() in /var/www/drupal.org/htdocs/sites/all/modules/project/usage/project_usage.drush.inc, line 104
Comment #13
lolandese CreditAttribution: lolandese commentedAt the bottom of my project pages I added:
It turns into a link to this issue with the title 'Reported installs count on project page not being updated'. Furthermore on hover the issue status is shown, 'Status: Active' at time of writing.
This way the issue gets known by site builders that currently might be unaware of it.
Thanks.
Comment #14
davidmac CreditAttribution: davidmac commentedRe: #11
@drumm are you saying that this is fixed given the removal of 'Download Statistics' from the issue title? I'm not seeing any fix to the downloads aspect of this issue. Also, I don't get what you mean by referring to D6 when this issue relates to the current version of Drupal.org i.e. D7, can you explain?
Also, the parent issue relates to Drupal.org D7 upgrade QA but you've changed this to Drupal.org customizations which is confusing given that the problem arose from the upgrade issue, and is not a new feature.
Comment #15
jthorson CreditAttribution: jthorson commenteddavidmac: Consider D7QA a 'triage' area for upgrade problems ... once the root cause is discovered, we move to the issue to whatever project actually hosts the affected code.
And the title change was to indicate that there are two problems here ... download statistics and reported install counts. They have two different causes/solutions, and the download statistics solution was delivered in the linked issue.
Comment #16
jthorson CreditAttribution: jthorson commentedSomething like this should fix #12 (untested!)
Comment #17
davidmac CreditAttribution: davidmac commented@jthorson Many thanks for the explanation, however the issue is also for those that are experiencing the problem, so IMO, care should be taken when stating that it is fixed.
It may be fixed in the code but is not fixed for those who are reporting the issue in the first instance, because the problem persists. Removing aspects of the description makes it difficult to find for others, and leads to duplicate/overlapping issues being raised as evidenced by the parent and child issues here.
Comment #18
rjacobs CreditAttribution: rjacobs commentedI'm trying to keep tabs on all this, partly to ensure the info in the parent issue (#2128619: Project usage/download/install statistics not being generated in a timely manner), which is starting to feel like a "meta" issue, is comprehensive. What I've been able to glean from all of this is roughly the following:
How did I do? Am I way off?
Comment #19
drummThis issue is *usage* stats, not download. Download stats are #2133385: project_generate_download() query needs porting to D7 structure.
The UI says "install", which is a synonym for usage here.
Comment #20
drummSorry, I got the wrong project.
I went ahead and committed #16. Leaving open since there could easily be other problems uncovered when this is deployed and runs.
rjacobs - otherwise, your summary is okay.
Comment #21
davidmac CreditAttribution: davidmac commented@drumm @rjacobs
Thanks for clarifying the latest state of affairs.
As you know, the download field has updated over night, but will need to be monitored to see if it's in line with trends.
As far as I'm aware there has been no loss of data throughout this hiccup, is that correct?
Comment #22
drummNo data should be lost. The scripts are designed to parse all the logs available.
Comment #23
lolandese CreditAttribution: lolandese commentedCorrected typo in issue title.
Comment #24
drummI took care of the last notice on line 490 of sites/all/modules/project/usage/project_usage.drush.inc. https://drupal.org/drupal-4.7-cvs thought it was a release of the #4878: Support file uploads via blogapi project, but that is an issue. I corrected it to be a Drupal core release.
We still have some
Notice: Undefined offset: 1 in _project_usage_process_varnish_file() (line 264 of /var/www/drupal.org/htdocs/sites/all/modules/project/usage/project_usage.drush.inc).
But that went way down. Either that is fixed by fixing the bad data for line 490, or there were just fewer lines in the parsed logfile triggering this notice.
The big problem is that the query killer is killing this while storing results.
Comment #25
drummLet's wait a day for the next run to see what notices remain. Once the notices are gone, I think we can close this, and leave the parent issue. The blocker is the query killer, which exists because of #2136119: Text search on issue queues is slow and sometimes WSODs.
Comment #26
jthorson CreditAttribution: jthorson commentedWhat if we were to get rid of the database transaction and the db_delete line, and use db_merge instead of db_insert? I know this would result in 1000 times more individual database transactions (since db_merge() doesn't support multi-values); but at the same time, the fact that our insert is running longer than the killer script timeout shows that the status quo isn't exactly efficient either.
Comment #27
drummThanks to #2178437: Add a DB user for cron, this has gotten slightly further. The next problem is
I believe I have fixed this, http://drupalcode.org/project/project.git/commit/6219465.
Comment #28
drummSeems this is working: https://drupal.org/project/usage/drupal
Comment #29
davidmac CreditAttribution: davidmac commentedI also am seeing this as fixed.
Thanks
Comment #30
RdeBoerYay! And it works for new modules created after 20 Oct 2013 too!
These new babies now have install counts on their project pages for the first time in their youg lives.
Sensational work drumm and all involved.