BHoM / BHoM_Adapter

GNU Lesser General Public License v3.0
7 stars 5 forks source link

Add caching system for CRUD methods #337

Closed IsakNaslundBh closed 1 year ago

IsakNaslundBh commented 1 year ago

Issues addressed by this PR

Closes #336

Add caching system in the base adapter, reducing the times the same objects are read more than once from the model.

Can for some cases significantly speed up both push and pulls from the adapter, depending o the case being run.

For full effect, the concrete adapters need to be updated to make use of this new system.

Test files

Changelog

Additional comments

IsakNaslundBh commented 1 year ago

@BHoMBot check compliance @BHoMBot check core

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `core`
alelom commented 1 year ago

@BHoMBot check compliance @BHoMBot check core

bhombot-ci[bot] commented 1 year ago
@alelom to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `core`
IsakNaslundBh commented 1 year ago

@BHoMBot check required

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
bhombot-ci[bot] commented 1 year ago
The check `code-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 1 year ago
The check `documentation-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 1 year ago
The check `core` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 1 year ago
The check `installer` has already been run previously and recorded as a successful check. This check has not been run again at this time.
IsakNaslundBh commented 1 year ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `ready-to-merge`