I see that NotifUser::resetNotificationCount does that in different context.

Any reasons why looping on the users and calling it would be a bad idea?

It has unintended side-effects that we probably don't want like setting it in memcached and invalidating the user cache. Also if possible we should aim to do batch inserts.

Change 259265 had a related patch set uploaded (by Matthias Mullie):
Backfill echo_unread_wikis table

https://gerrit.wikimedia.org/r/259265

Change 259265 merged by jenkins-bot:
Backfill echo_unread_wikis table

https://gerrit.wikimedia.org/r/259265

wikishared.echo_unread_wikis table is properly updated for the following cases:

1. when a user initiates an action that sends alerts/notifications to another user
2. when a recipient of notifications from the previous step marks some(all) messages/alerts
3. in flyout or
4. go to Notifictions page.

Technically, that is testing the normal flow, not the backfill.

We should be able to test the backfill as part of the MediaWiki-Vagrant stuff. I'm working on a patch for this now (to auto-run it on provision).

Change 264257 had a related patch set uploaded (by Mattflaschen):
Backfill echo_unread_wikis table after it is created

https://gerrit.wikimedia.org/r/264257

@Etonkovidova If you want to test this on MediaWiki-Vagrant:

1. DROP the echo_unread_wikis table in the wikishared DB (it will probably only exist if you tested the prior patch in this chain).
2. Create whatever notifications you want to test.
3. Checkout https://gerrit.wikimedia.org/r/264257
4. vagrant provision

It will create and backfill the table. You can also run the script manually at any time if you want.

To connect to wikishared:

mysql -u extensionadmin wikishared -pextensionpassword

In T119523#1937102, @Mattflaschen wrote:

To connect to wikishared:

mysql -u extensionadmin wikishared -pextensionpassword

Can we add that to the welcome message when we access the VM with vagrant ssh?

Sorry, when I posted the above, my brain was stuck on "test the users" mode, since part of the point was to have the PHP app use separated users.

However, the root user still exists (even if the app doesn't use it), so you can do:

mysql wikishared

I can still add that to MOTD if you think it's useful (there's a manifest, so it can easily be done only if Echo is installed).

Nice catch on the Puppet ordering problem. Fixed in latest patch.

So, you can:

1. DROP again.
2. Create notifications.
3. Checkout latest https://gerrit.wikimedia.org/r/#/c/264257
4. vagrant provision

Rechecked locally actual backfill action with the steps given by @Mattflaschen

1. wikishatred.echo_unread_wikis is created upon Notifcations are triggered
2. e.g. euw_user=6 has Alerts form frwiki and Messages from wiki, It is be recorded in wikishatred.echo_unread_wikis as the following:

root@localhost:[wikishared]> select * from echo_unread_wikis;

After euw_user=6 viewed Alerts in frwiki, the table is updated.

Change 264257 merged by jenkins-bot:
Backfill echo_unread_wikis table after it is created

https://gerrit.wikimedia.org/r/264257