jnosql icon indicating copy to clipboard operation
jnosql copied to clipboard

CriteriaQuery API

Open amoscatelli opened this issue 3 years ago • 10 comments

This is only the first commit.

This is an example of what it will look like

CriteriaQuery<Person> createQuery = subject.createQuery(Person.class); Root<Person> from = createQuery.from(); CriteriaFunction<Person, Person, Number> count = from.count(); FunctionQuery<Person> select = createQuery.select(count); FunctionQueryResult<Person> query = subject.query(select);

QueryBuilder and metamodel generator are almost ready.

Now I am going to figure out how to propagate the CriteriaQuery to the communication layer.

@otaviojava what do you think ?

I may need an help about polishing what I am doing.

amoscatelli avatar Apr 22 '22 00:04 amoscatelli

Can one of the admins verify this patch?

genie-jnosql avatar Apr 22 '22 00:04 genie-jnosql

I liked @amoscatelli let's move it!

otaviojava avatar Apr 22 '22 17:04 otaviojava

@otaviojava I finished the select query implementation. I am working on missing comments/javadoc and on the TEST. Please have a look. Let's finish this e move to the unimplemented yet features !

amoscatelli avatar May 09 '22 13:05 amoscatelli

@otaviojava I need help with the test. What kind of test can I add ?

amoscatelli avatar May 09 '22 17:05 amoscatelli

Hey, @amoscatelli this week, I'm traveling; I'll take a look at it next week.

otaviojava avatar May 10 '22 04:05 otaviojava

@otaviojava hi there ! How was your traveling ? Can I have a feedback about how to move forward ?

amoscatelli avatar May 16 '22 10:05 amoscatelli

@amoscatelli Yes I did; I've created the first PR suggestion.

otaviojava avatar May 16 '22 13:05 otaviojava

@amoscatelli Yes I did; I've created the first PR suggestion.

Sorry, I can't find what you are talking about. What PR suggestion ?

amoscatelli avatar May 16 '22 14:05 amoscatelli

@amoscatelli this one.

https://github.com/amoscatelli/nosql/pull/1

Let's talk about the proposal first; then, we can talk about the implementation.

otaviojava avatar May 16 '22 21:05 otaviojava

Thank you, going to review the PR

amoscatelli avatar May 17 '22 07:05 amoscatelli