StackExchange / StackExchange/StackExchange.Redis
Make `ServerMaintenanceEvent` constructor to public.
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 6.2k
- Forks
- 1.6k
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 43
Description
StackExchange.Redis supports Azure maintenance events, but not Amazon ElastiCache.
ElastiCache maintenance events are notified via SNS, not Redis Pub/Sub, and it seems that there are no plans to support this notification as an out-of-the-box feature.
I would like to subscribe to maintenance event notifications via SNS as a third-party extension, but because the constructor of the ServerMaintenanceEvent class is internal, I cannot derive it to create a maintenance event class for ElastiCache, and I cannot call OnServerMaintenanceEvent.
Please make the constructor of ServerMaintenanceEvent public so that instances can be created from third-party code.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the ServerMaintenanceEvent class and inspect its internal constructor and the OnServerMaintenanceEvent usage described in the issue. Confirm the public API change supports creating a derived maintenance event from third-party code, then run the relevant existing test suite; done means external SNS integration code can instantiate or derive the event and invoke the notification path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- backend-api-design
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100