Part of what makes Lemmy (and other voting link aggregators) work is the voting aspect. By taking away outbound vote federation, it forces further consolidation into these popular instances. Thereby further exacerbate the problem because now they’re even more consolidated and the posts and comments eventually becomes the bottleneck for the exact same underlying chatty protocol. For this reason, I’d be vehemently against this change without a pairing PR that allows this information to be requested via a batch pull that the protocol makes available.
this post was submitted on 04 Jul 2023
5 points (100.0% liked)
Lemmy Server Performance
6 readers
1 users here now
Lemmy Server Performance
lemmy_server uses the Diesel ORM that automatically generates SQL statements. There are serious performance problems in June and July 2023 preventing Lemmy from scaling. Topics include caching, PostgreSQL extensions for troubleshooting, Client/Server Code/SQL Data/server operator apps/sever operator API (performance and storage monitoring), etc.
founded 1 year ago
MODERATORS
Thanks for doing all this.
Do we have any real numbers from a real server? How many votes are trying to be federated to how many servers?
Just ballparking some approximate numbers:
- !technology@lemmy.world
- 15k subscribers
- 4000 subscribed servers
- 10 votes per subscriber per day
15000 * 4000 * 10 = 600,000,000 federated actions. That is around 7,000 per second 24/7 for one community.
IMO, this real time federation just doesn't scale. We need to start planning the specs for federation batching.
I'm hoping the 'subscribed servers' is maybe only 300 or so? But I don't know, the big sites haven't been sharing information like that in my experience. They did say there were "millions" of outbound federation tasks. I expect the number of votes by user is higher than your number. They did put in code changes to detect servers they can't reach and to stop attempting delivery.
We need to start planning the specs for federation batching.
I think a pull app that goes around to servers with content and uses the front-end API to grab 300 or more comments at a time, etc is the way to go. The client API is geared toward batch delivery. since lemmy.ml is so unstable for discussion, I opened a topic on GitHub: https://github.com/RocketDerp/lemmy_helper/discussions/4 - where I proposed some new /api/syncshare to get more raw data out of the PostgreSQL tables.
prototype pull
pull request for prototype: https://github.com/LemmyNet/lemmy/pull/3475
The environment variable LEMMY_SKIP_FEDERATE_VOTES is as good a way as any to reference this code hack.
I don't think this is a good change. It would make communities basically no more than an RSS feed and I don't think that's quite the point..
Somewhat related, but why are we federating votes? Why not just federate the upvote count and downvote count? Does each server need to track the identity of every voter on a subscribed community?
Each server will track votes from their own users, preventing duplicate votes.
Why not just federate the upvote count and downvote count?
I think the answer to that is that it isn't an optimized design.
Does each server need to track the identity of every voter on a subscribed community?
I think so. Which isn't a terrible assumption that user who votes will eventually comment/post and that profile will be of use.
I have no idea why the file is named "mod.rs", as normal non-admin non-moderator users seem to go through this code path.
ok, I figured out how to get Rust to match the enum, is there a way to do this with match instead of if statements?
+ if let AnnouncableActivities::UndoVote(_) = activity {
+ warn!("zebratrace310 SKIP UndoVote");
+ } else if let AnnouncableActivities::Vote(_) = activity {
+ warn!("zebratrace310A SKIP Vote");
+ } else {
+ warn!("zebratrace311 send");
+ AnnounceActivity::send(activity.clone().try_into()?, community, context).await?;
+ };
Code seems to work great, blocks UndoVote/Vote but does the send on comment reply.
more "correct" way would be this:
match activity {
AnnouncableActivities::UndoVote(_) => warn!("zebratrace310 SKIP UndoVote"),
AnnouncableActivities::Vote(_) => warn!("zebratrace310A SKIP Vote"),
_ => {
warn!("zebratrace311 send");
AnnounceActivity::send(activity.clone().try_into()?, community, context).await?;
},
}
here it is in the rust book: https://doc.rust-lang.org/stable/book/ch06-02-match.html
ool. I found the syntax for multiple hits, so I was looking for:
match activity {
AnnouncableActivities::UndoVote(_) |
AnnouncableActivities::Vote(_) => {
warn!("zebratrace310 SKIP federating Vote/UndoVote");
},
_ => {
warn!("zebratrace311 send");
AnnounceActivity::send(activity.clone().try_into()?, community, context).await?;
},
}
Thank you.
view more: next ›