manojpec opened a new pull request #4505:
URL: https://github.com/apache/hudi/pull/4505
## What is the purpose of the pull request
- HoodieMetadataMergedLogRecordReader#getRecordsByKeys() and its parent
class methods
are not thread safe. When multiple queries come in for gettting log
records
by keys, they all operate on the same log record reader instance provided
by
HoodieBackedTableMetadata#openReadersIfNeeded() and they trip over each
other
as they clear/put/get the same class memeber records.
## Brief change log
- The fix is to streamline the mutatation to class member records. Making
HoodieMetadataMergedLogRecordReader#getRecordsByKeys() a synchronized
method
to avoid concurrent log records readers getting into NPE.
## Committer checklist
- [ ] Has a corresponding JIRA in PR title & commit
- [ ] Commit message is descriptive of the change
- [ ] CI is green
- [ ] Necessary doc changes done or have another open PR
- [ ] For large changes, please consider breaking it into sub-tasks under
an umbrella JIRA.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
