_count_known_servers delay

[heftig]

synapse/synapse/storage/databases/main/roommember.py

Lines 88 to 95 in 1c6a190

	self.hs.get_clock().looping_call(
	self._count_known_servers,
	60 * 1000,
	)
	self.hs.get_clock().call_later(
	1000,
	self._count_known_servers,
	)

Is this really supposed to be 1000 seconds? looping_call uses milliseconds and call_later uses seconds.

[callahad]

FWIW, this was introduced almost three years ago in 55d5b3a as part of #5981

[anoadragon453]

@heftig You're correct, and that definitely shouldn't be 1,000 seconds 🙂

Would you like to PR a fix? Otherwise we can take care of it.

[heftig]

Would you like to PR a fix? Otherwise we can take care of it.

I would if I just had to make a commit with the fix, but your guidelines require a lot more work I don't have time for.

[anoadragon453]

Understandable, thanks for bringing it to our attention regardless 🙂

I've opened a PR to fix this here: #10195