F find query
Attention: f06ca944 must be checked carefully. Two separate changes affected the "WITH" statement. @daniel can you help/check?
Summary
The default behavior of the query FIND SomeName [...]
(as well as COUNT
and SELECT
) is being made configurable and changes
-
FIND SomeName
will be interpreted asFIND $FIND_QUERY_DEFAULT_ROLE SomeName
from now on whereFIND_QUERY_DEFAULT_ROLE
is a newly introduced server property. - The new
FIND_QUERY_DEFAULT_ROLE
server property defaults toRECORD
which is why the behavior of the server api has a breaking change. - Of course, administrators can choose to retain the old behavior by setting
FIND_QUERY_DEFAULT_ROLE=ENTITY
.
As requested per caosdb/customers/leibniz-zmt/management#98
Focus
Since the default role is configurable the changed code is executed after the parsing of the query string. This way, the parser did not change at all.
Test Environment
Tests in caosdb-pyinttest!45 (merged)
Check List for the Author
Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines
-
All automated tests pass -
Reference related issues -
Up-to-date CHANGELOG.md (or not necessary) -
Annotations in code (Gitlab comments) - Intent of new code
- Problems with old code
- Why this implementation?
Check List for the Reviewer
-
I understand the intent of this MR -
All automated tests pass -
Up-to-date CHANGELOG.md (or not necessary) -
The test environment setup works and the intended behavior is reproducible in the test environment -
In-code documentation and comments are up-to-date. -
Check: Are there specifications? Are they satisfied?
For further good practices have a look at our review guidelines.