Take into account entity role for retrieval / use get_entity_by_name

Summary

Addresses https://gitlab.com/linkahead/linkahead-pylib/-/issues/123 and #198 (closed)
Container.retrieve now uses get_entity_by_name for retrieval of entities without id and matches the role if given.

Test-MR: caosdb-pyinttest!87 (merged)

Focus

• This is likely significantly slower than before, and makes a lot more requests for large Containers with entities without ids. Is this acceptable? If not, would an additional parameter to specify which behaviour is wanted be a usable compromise?
• To offset the performance hit, get_entity_by_id is not used. Is this ok?
• The manual query for file retrieval with a path was replaced by get_entity_by_path, is this likely to break anything?

Test Environment

Manual Tests

Check List for the Author

• All automated tests pass
• Reference related issues
• Up-to-date CHANGELOG.md (or not necessary)
• Up-to-date JSON schema (or not necessary)
• Appropriate user and developer documentation (or not necessary)
• Annotations in code (Gitlab comments)

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md (or not necessary)
• Appropriate user and developer documentation (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?

assigned to @i.nueske

mentioned in merge request caosdb-pyinttest!87 (merged)

changed the description

mentioned in issue #198 (closed)

marked the checklist item All automated tests pass as completed

marked the checklist item Annotations in code (Gitlab comments) as completed

requested review from @salexan

changed the description

marked the checklist item I understand the intent of this MR as completed

added 6 commits

• 880159e6...c7e4fa23 - 5 commits from branch dev
• 41b0d0be - Merge branch 'dev' into f-entity-retrieve-with-get_entity_by

Compare with previous version

added 24 commits

• 41b0d0be...600505d1 - 23 commits from branch dev
• 8f1be0e0 - Merge branch 'dev' into 'f-entity-retrieve-with-get_entity_by'

Compare with previous version

added 26 commits

• 8f1be0e0...c2062ba5 - 25 commits from branch dev
• f9d3b8ac - Merge branch 'dev' into f-entity-retrieve-with-get_entity_by

Compare with previous version

Unfortunately, this needs to be discussed in general (Deprecate retrieve?)

resolved all threads

mentioned in issue #222

closed