Add a note on MongoDB Panache doc about query on same field #13804
Conversation
ghost
added
|
/cc @evanchooly @FroMage |
| @@ -513,6 +513,10 @@ Here are some query examples: | |||
| - `lastname is not null` will be mapped to `{'lastname':{'$exists': true}}` | |||
| - `status in ?1` will be mapped to `{'status':{$in: [?1]}}` | |||
|
|
|||
| WARNING: MongoDB queries must be valid JSON documents, | |||
| using multiple times the same field on a query is not allowed using PanacheQL as is it would generate an invalid JSON | |||
| (see link:https://github.com/quarkusio/quarkus/issues/12086[this issue on github]). | |||
There was a problem hiding this comment.
Can we avoid referencing an issue? Better put an example in the doc of what's not valid.
Btw, I think it's a pretty annoying limitation given it can be a very common use case.
There was a problem hiding this comment.
Btw, I think it's a pretty annoying limitation given it can be a very common use case.
That's why there is an open issue about it, so people can give feedback on it and maybe work on it. I think it's important to link to the openned issue as we could gather feedback about this issue and decide if we need to work on it (this would not be easy).
There was a problem hiding this comment.
I can off course add an example but I think the issue still needs to be linked here.
There was a problem hiding this comment.
@gsmet what your takes on my previous comment ?
...s/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/package-info.java
Outdated
Show resolved
Hide resolved
| @@ -85,6 +85,11 @@ | |||
| * </ul> | |||
| * </p> | |||
| * <p> | |||
| * <b>WARNING</b>: MongoDB queries must be valid JSON documents, | |||
| * using multiple times the same field on a query is not allowed using PanacheQL as is it would generate an invalid JSON | |||
| * (see this issue on github: https://github.com/quarkusio/quarkus/issues/12086). | |||
loicmathieu
force-pushed
the
doc/note-on-mongodb-query
branch
from
42cd1ba to
983192b
Compare
Relates to #12086 that we keep open to maybe fix it some day.