refactor(backend): Refactor of ReindexThread (#30109)

[spbolton]

This pull request includes significant updates to the BulkProcessorListener class in the dotCMS project to improve concurrency handling and error logging. The most important changes involve replacing existing data structures with thread-safe alternatives, adding new atomic counters, and refactoring methods for better readability and functionality.

Improvements to concurrency handling:

• Replaced HashMap with ConcurrentHashMap for workingRecords to ensure thread-safe operations. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)
• Introduced AtomicInteger counters for tracking total responses, success, and failure counts. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)
• Added a BlockingQueue<ReindexResult> to manage reindexing results in a thread-safe manner. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)

Refactoring for readability and functionality:

• Extracted logic for obtaining IDs from BulkItemResponse into a new method getIdFromResponse. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)
• Refactored error handling and success logging into separate methods addErrorToQueue and addSuccessToQueue. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)
• Replaced handleFailure and handleSuccess methods with new logic to add results to the queue and increment counters. (dotCMS/src/main/java/com/dotmarketing/common/reindex/BulkProcessorListener.java)

Follows on from basic change in #30084

This PR resolves #30109 (Refactor of ReindexThread).

[dotcms-sonarqube]

Quality Gate passed

Issues
1 New issue
17 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube