Add GSI for Slack channel lookups and replace scan with query #66
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Made changes to Slack App logic. Slack channel look up now use GSI, allowing for faster query as opposed to scan
Type of change (Choose 1)
Reason for this change
This allows for more efficient use of DDB, in relation to Slack. With GSI, there is faster lookups by Slack for DynamoDB, which allows the slack integration to perform faster and reduce possible latency, which becomes only visible with scale.
Related Issue (if applicable)
Asana Link
Contributor Task List
Other: Provide a brief description or attach an architecture diagram to this PR.How is state / caching maintained in the integration
N/A
How much do you expect this integration to cost
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the MIT No Attribution license