objectbox / objectbox/objectbox-java
Can't use query ForEach() with Order()
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Java
- Estrellas
- 4.6k
- Forks
- 311
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
I got that exception message when i tried to use foreach function with order list:
"This method is currently only available for unordered queries"
Its not make any sense because I can just query with order and make forEach manually in my code and it will work. Weird a bit.
Issue Basics
ObjectBox v1.1.0
It can reproduce.
Code
final Box<CallRemindersData> box = objectBox.boxFor(CallRemindersData.class);
final QueryBuilder<CallRemindersData> callRemindersDataQueryBuilder = box.query().order(CallRemindersData_.notificationTime);
callRemindersDataQueryBuilder .build().forEach()
The code that make it (Query.java):
if (hasOrder) {
throw new UnsupportedOperationException("This method is currently only available for unordered queries");
}
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en Query.java, en la comprobación de hasOrder que lanza UnsupportedOperationException, y después inspecciona la implementación de forEach circundante y el comportamiento de ejecución de la query. Reproduce el ejemplo de Java con una query ordenada y verifica que forEach se complete en orden sin la excepción, añadiendo o ejecutando las pruebas existentes relevantes si están disponibles.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- databases
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 45/100