Closed amoscatelli closed 1 year ago
Can one of the admins verify this patch?
I liked @amoscatelli let's move it!
@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 !
@otaviojava I need help with the test. What kind of test can I add ?
Hey, @amoscatelli this week, I'm traveling; I'll take a look at it next week.
@otaviojava hi there ! How was your traveling ? Can I have a feedback about how to move forward ?
@amoscatelli Yes I did; I've created the first PR suggestion.
@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 this one.
https://github.com/amoscatelli/nosql/pull/1
Let's talk about the proposal first; then, we can talk about the implementation.
Thank you, going to review the PR
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.