cashapp / cashapp/tempest

Improve the error message when a table does not have secondary index sort key annotation

Open
#78 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Kotlin
Stars
90
Forks
42
Avg merge
46m
Merged PRs (30d)
1

Description

[This line of code](https://github.com/cashapp/tempest/blob/master/tempest2/src/main/kotlin/app/cash/tempest2/internal/V2.kt#L82) currently throws a NPE instead of suggesting the developer to add a `@DynamoDbSecondaryIndexSortKey` annotation

Contributor guide

Open the contributing guide

Research direction

Start at tempest2/src/main/kotlin/app/cash/tempest2/internal/V2.kt line 82, where the missing secondary-index sort-key annotation currently causes an NPE. Trace the surrounding lookup to determine the expected missing-annotation case. Done means the error clearly suggests adding @DynamoDbSecondaryIndexSortKey instead of throwing an NPE.

Written by the indexing model from the issue text.

Assessment

Tech stack
aws, kotlin
Domain
database
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.