getResultStream should use query.getResultStream instead of query.getResultList().stream

[th3nu11]

Hi all,

I am writing to you for help to check if my fixed is correct.

Seems that in the class/method GraphQLJpaQueryFactory::getResultStream that should return a stream but to do that the method convert the query stream to list.
See code here:

graphql-jpa-query/graphql-jpa-query-schema/src/main/java/com/introproventures/graphql/jpa/query/schema/impl/GraphQLJpaQueryFactory.java

Line 264 in aeb8766

return query.getResultList()

This thing resolves the entire query and does't use the jpa stream. I think the code should be

 return query.getResultStream()
                    .peek(entityManager::detach);

instead of the actual code:

 return query.**getResultList()
                    .stream()**
                    .peek(entityManager::detach);

Can you confirm if I am right or not?

Regards,
Carlo