Skip to content

Removing synchronized keyword

Abhishek Patil requested to merge abpatil/storage_perf_impvmnt into master

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [NO]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this introduce a breaking change?

  • [NO]

What is the current behavior?

The method get() of RecordMetadataRepository class was marked synchronized due to which performance of GET /records/{id} and PUT /records api was affected.

What is the new/expected behavior?

There is no need for the method to be synchronized and hence after removing synchronized keyword the performance of the api's is improved.

Have you added/updated Unit Tests and Integration Tests?

NA. All the existing ITs are passing.

Any other useful information

NA.

Merge request reports