objectbox / objectbox/objectbox-java

Can't use query ForEach() with Order()

Aberta
#238 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

enhancement
Linguagem predominante
Java
Estrelas
4.6k
Forks
311
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

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");
 }

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece em Query.java, na verificação de hasOrder que lança UnsupportedOperationException, e então inspecione a implementação de forEach ao redor e o comportamento da execução da query. Reproduza o exemplo Java com uma query ordenada e verifique se forEach é concluído na ordem sem a exceção, adicionando ou executando os testes existentes relevantes, se disponíveis.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
java
Domínio
databases
Tipo de issue
Bug
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.