Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MODDATAIMP-372 Data Import job creates SRS records but not all expected Inventory records #141

Merged
merged 1 commit into from
Mar 25, 2021

Conversation

OleksiiKuzminov
Copy link
Contributor

Purpose

Data Import job creates SRS records but not all expected Inventory records
org.apache.kafka.common.errors.UnknownProducerIdException: This exception is raised by the broker if it could not locate the producer metadata associated with the producerId in question. This could happen if, for instance, the producer's records were deleted because their retention time had elapsed. Once the last records of the producerId are removed, the producer's metadata is removed from the broker, and future appends by the producer will return this exception. for topic-partition folio.pub-sub.fs00001019.DI_SRS_MARC_BIB_RECORD_CREATED-0 with producerId 538357, epoch 0, and sequence number 4"

Approach

Remove pull of producers and generate producer by demand using shared vert.x pull

mod-data-import
Data Import job creates SRS records but not all expected Inventory records
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@OleksiiKuzminov OleksiiKuzminov merged commit f3cb9b3 into master Mar 25, 2021
@OleksiiKuzminov OleksiiKuzminov deleted the MODDATAIMP-372 branch March 25, 2021 08:19
KaterynaSenchenko pushed a commit that referenced this pull request Mar 25, 2021
(cherry picked from commit f3cb9b3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants