aws / aws/bedrock-agentcore-sdk-python
Pagination discrepancy for MemorySessionManager ---> search_long_term_memories and list_long_term_memory_records
- Lenguaje dominante
- Python
- Estrellas
- 761
- Forks
- 147
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 7
Descripción
**Describe the bug**
MemorySessionManager has 2 memory listing methods named search_long_term_memories and list_long_term_memory_records
as per boto3 docs both the methods should support pagination with nextToken
i can see list_long_term_memory_records has some pagination inside it and fetches all the records in loop where as search_long_term_memories doesnt have one
both methods pagination logic should be similar as per boto3 actual methods? if yes, search_long_term_memories also needs same looping logic and fetching all records
Also as list_long_term_memory_records brings up all the records in a loop and then do the slicing as per max_results,
- what if the records are huge in number?
- Is sequential pagination will be available in future? as even if i increase max_results value to a bigger number, i will get the values from the beginning till max_result always
**Screenshots**
boto3 links:
https://docs.aws.amazon.com/boto3/latest/reference/services/bedrock-agentcore/client/retrieve_memory_records.html
https://docs.aws.amazon.com/boto3/latest/reference/services/bedrock-agentcore/client/list_memory_records.html
Guía de contribución
Línea de trabajo
Comienza por MemorySessionManager.search_long_term_memories y list_long_term_memory_records, y compara después el manejo de nextToken con la documentación enlazada de boto3 sobre retrieve_memory_records y list_memory_records. Confirma el comportamiento previsto de la paginación y max_results; se considera terminado cuando ambos métodos proporcionen resultados coherentes y documentados, sin una semántica de recuperación ambigua.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- aws, python
- Área
- api, backend
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100