Common DynamoDb Enhanced interface for "queryable"
- Lenguaje dominante
- Java
- Estrellas
- 2.6k
- Forks
- 1k
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 51
Descripción
### Describe the feature
A common interface for `DynamoDbTable` and `DynamoDbIndex` defining the `scan` and `query` methods.
### Use Case
I have some pipeline methods that build `QueryConditional`s, execute them, and unpack the results. Sometimes these queries are aimed at the base table, and sometimes at GSIs, depending on the sorting criteria. Currently, the table and index interfaces share no common base interface, resulting in code duplication.
### Proposed Solution
`DynamoDbTable`'s query and scan methods return `PageIterable`, which is a subtype of `DynamoDbIndex`'s return type of `SdkIterable>`. A common interface `DynamoDbQueryable` could be extended by both the table and index interfaces with no impact to existing code.
### Other Information
_No response_
### Acknowledgements
- [X] I may be able to implement this feature request
- [ ] This feature might incur a breaking change
### AWS Java SDK version used
2.20.50
### JDK version used
openjdk version "17.0.6" 2023-01-17 OpenJDK Runtime Environment Temurin-17.0.6+10 (build 17.0.6+10) OpenJDK 64-Bit Server VM Temurin-17.0.6+10 (build 17.0.6+10, mixed mode, sharing)
### Operating System and version
Gentoo Linux
Guía de contribución
Línea de trabajo
Empieza localizando las interfaces DynamoDbTable y DynamoDbIndex y revisando sus firmas de scan y query, especialmente PageIterable frente a SdkIterable>. Comprueba si hay pruebas cercanas para estas interfaces. Se considera terminado cuando ambas interfaces exponen el comportamiento consultable común solicitado sin romper los tipos de retorno existentes.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- databases
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100