Skip to content

Add a second elasticache connection for dual write#41

Draft
joaojunceira-zendesk wants to merge 3 commits into
masterfrom
silk-1730-add-new-redis-connector
Draft

Add a second elasticache connection for dual write#41
joaojunceira-zendesk wants to merge 3 commits into
masterfrom
silk-1730-add-new-redis-connector

Conversation

@joaojunceira-zendesk

@joaojunceira-zendesk joaojunceira-zendesk commented May 23, 2024

Copy link
Copy Markdown
Contributor

Description

  • Add second elasticache connection for dual write;

References

  • Jira:
  • Github issue:

Risks

  • High | Medium | Low : How might failures be experienced? All code changes
    carry a minimum of risk of Low, and None should be a rare exception.
  • Rollback:

sergionoviello
sergionoviello previously approved these changes May 28, 2024

@sergionoviello sergionoviello left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread lib/connection.js Outdated
@sergionoviello sergionoviello force-pushed the silk-1730-add-new-redis-connector branch 3 times, most recently from b49aef8 to a5bbf97 Compare June 5, 2024 10:28
@sergionoviello sergionoviello force-pushed the silk-1730-add-new-redis-connector branch from 71fa171 to 0973b39 Compare June 5, 2024 15:59
niall3rs
niall3rs previously approved these changes Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants