Get rid of the duplicate users cache #2662

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

arqunis merged 1 commit into serenity-rs:next from GnomedDev:remove-users-cache

Dec 28, 2023

Member

GnomedDev commented Dec 13, 2023 •

edited

Loading

This cache was just duplicating information already in Guild::members and therefore should be removed. This saves 700mb for my bot (pre-fixedstring). This has to refactor content_safe to always take a Guild instead of Cache, but in practice it was mostly pulling from guild cache and this means it is more likely to respect nicknames and other information, while losing the ability to clean mentions from DMs (which don't matter).

github-actions bot added model cache utils examples labels

GnomedDev force-pushed the remove-users-cache branch from 77c480f to 03e2d39 Compare

December 13, 2023 11:54


          Get rid of the duplicate users cache

GnomedDev force-pushed the remove-users-cache branch from 03e2d39 to 9765654 Compare

December 28, 2023 01:05

arqunis added enhancement breaking change labels

arqunis approved these changes

View reviewed changes

arqunis merged commit c0ab15a into serenity-rs:next

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

cf66540

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

fb9cc79

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

9c13cd0

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev deleted the remove-users-cache branch

January 14, 2024 10:26

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

5cc4e70

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

371e77c

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

90d8a39

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

arqunis pushed a commit to arqunis/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

7055a6e

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

5fa809a

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

45b3638

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

88ef106

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed. 

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit to GnomedDev/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

b1c51e3

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit to GnomedDev/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

f381f1e

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit to GnomedDev/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

5da921a

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

jamesbt365 mentioned this pull request

Correct docs for UserId::to_user #2809

Merged

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

0ced192

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

4fe1897

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

08d9ab1

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

cc119f4

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

a5d83ab

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

49ac2aa

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

f722c51

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

a178423

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

349826c

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

f439936

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

ce8af61

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

236ec21

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

9d6f014

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

f6f2038

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

f140a66

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

024426c

The documentation was never updated after serenity-rs#2662 removed the user cache.

GnomedDev added a commit to GnomedDev/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

b500bac

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev pushed a commit to GnomedDev/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

6ba1005

The documentation was never updated after serenity-rs#2662 removed the user cache.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

7d99d8f

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev pushed a commit that referenced this pull request


          Correct documentation for UserId::to_user (#2809)

7d87a85

The documentation was never updated after #2662 removed the user cache.

GnomedDev added a commit that referenced this pull request


          Get rid of the duplicate users cache (#2662)

de4c358

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

GnomedDev pushed a commit that referenced this pull request


          Correct documentation for UserId::to_user (#2809)

The documentation was never updated after #2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

a85c10c

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

5120a4a

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

4bf63b5

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

8eef723

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

2835e42

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

18aeeef

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

668c750

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

8499e78

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

ad0ff7c

The documentation was never updated after serenity-rs#2662 removed the user cache.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Get rid of the duplicate users cache (serenity-rs#2662)

a5d9ed0

This cache was just duplicating information already present in `Guild::members`
and therefore should be removed.

This saves around 700 MBs for my bot (pre-`FixedString`).

This has to refactor `utils::content_safe` to always take a `Guild` instead
of`Cache`, but in practice it was mostly pulling from the guild cache anyway
and this means it is more likely to respect nicknames and other information,
while losing the ability to clean mentions from DMs, which do not matter.

mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request


          Correct documentation for UserId::to_user (serenity-rs#2809)

9b2f1d1

The documentation was never updated after serenity-rs#2662 removed the user cache.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change cache enhancement examples model utils