(redshift-alpha): setLoggingProperties method for Clusters
- Dominant language
- TypeScript
- Stars
- 12.9k
- Forks
- 4.6k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 83
Description
### Describe the feature
Add a `setLoggingProperties` method for Clusters that enables setting the Cluster logging configuration post construction.
### Use Case
1. I have S3 buckets created later in my application
2. It's easier to conditionally set the logging configuration this way (dev, prod)
3. I have Redshift Clusters that get passed into a construct that must be configured in certain ways (ex. logging)
### Proposed Solution
1. Add `setLoggingProperties` to `Cluster` construct
2. Method takes in an `IBucket` and string prefix as parameters
3. Method throws an error if there is already a logging configuration set, otherwise it adds the logging configuration to the underlying `CfnCluster`
### Other Information
_No response_
### Acknowledgements
- [X] I may be able to implement this feature request
- [ ] This feature might incur a breaking change
### CDK version used
N/A
### Environment details (OS name and version, etc.)
N/A
Contributor guide
Research direction
Locate the redshift-alpha Cluster construct and its underlying CfnCluster, then inspect how existing logging configuration and nearby mutator methods are implemented and tested. Done means adding setLoggingProperties with an IBucket and prefix, rejecting an existing logging configuration, and covering the behavior with tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- aws, typescript
- Domain
- cloud, databases
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100