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.
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
Don't ask the database for user ids which are not emails #676
Don't ask the database for user ids which are not emails #676
Conversation
User ids in the guest app are always emails, so there is no use in querying the database when there is no @ in the user id. Signed-off-by: Joas Schilling <coding@schilljs.com>
…eady Signed-off-by: Joas Schilling <coding@schilljs.com>
Regarding the second commit, we queried the user id twice all the time: queries from our famous query log:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change looks fine 👍
Signed-off-by: Joas Schilling <coding@schilljs.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 for the tests
Can we release a new version until monday so we have a new deployment in the next company call? |
I can do that tomorrow |
User ids in the guest app are always emails, so there is no
use in querying the database when there is no @ in the user id.
Commit 1 saves 18.061 queries from our famous query log
Commit 2 saves 17.898 queries from our famous query log