Aiven-Open / klaw

Klaw, the latest OS tool by Aiven, helps enterprises cope with Apache Kafka(r) topics, schema registry and connectors governance by introducing roles/authorizations to users of various teams of an org.
https://www.klaw-project.io/
Apache License 2.0
142 stars 58 forks source link

Review usage of LinkedHashMap, synchronized blocks, exception handling in clusterapi in the repo #173

Closed muralibasani closed 1 year ago

muralibasani commented 1 year ago

What is currently missing?

Usage of LinkedHashMap in connector calls can be replaced with HashMap ? Review usage of synchronized Exception handling in clusterapi

How could this be improved?

Usage of LinkedHashMap in connector calls can be replaced with HashMap ? Review usage of synchronized, verify if only one instance of jvm is allowed to run Exception handling in clusterapi for controllers with better try/catch or global exception handler

Is this a feature you would work on yourself?

sekaiser commented 1 year ago

Regarding exception handling, maybe it makes sense to consider the following as well: