Skip to content
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

Query|Receiver: Do not log full request on ProxyStore by default #7305

Merged
merged 2 commits into from
Apr 26, 2024
Merged

Query|Receiver: Do not log full request on ProxyStore by default #7305

merged 2 commits into from
Apr 26, 2024

Commits on Apr 25, 2024

  1. Query|Receiver|Store: Do not log full request on ProxyStore by default 

    We had a problem on our production where a sudden increase in requests with long matchers was putting our receivers under a lot of pressure.
Upon checking profiles we saw that the problem was calls to Log()

Signed-off-by: Pedro Tanaka <pedro.tanaka@shopify.com>
    @pedro-stanaka
    pedro-stanaka committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    10536c8 View commit details
    Browse the repository at this point in the history

  2. Adding changelog 

    Signed-off-by: Pedro Tanaka <pedro.tanaka@shopify.com>
    @pedro-stanaka
    pedro-stanaka committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    43519f5 View commit details
    Browse the repository at this point in the history