I’m sifting through sonar qube to create JIRA issues from the rules that are broken in the code.
I see that there are lots of uses of synchronized classes and wanted to talk about how to best clean that up.
Am I right to say that whenever we use one of the synchronized classes like
Vector for local variables we can safely replace them with the non-synchronized such as
ArrayList since those local variables are not shared state between threads there is no need to synchronize every operation on this container.
Such a use can be seen here
According to a github search for vector there are 45 occurences of vector. Should I create one JIRA issue with subtasks or simply one JIRA issue with separate pull requests one for each class thats affected?