[
https://issues.apache.org/jira/browse/HUDI-7799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
bradley updated HUDI-7799:
--------------------------
Description:
Resolved in PR: https://github.com/apache/hudi/pull/11308
AbstractHoodieLogRecordReader#processNextRecord access modifier is wrong and
should be changed from public to protected.
reason:
processNextRecord is used to process HoodieRecord inside the class. Derived
classes implement this abstract member method. The access modifier should not
be public. For example, the derived class HoodieMergedLogRecordScanner
implements processNextRecord, which is used to merge HoodieRecord.
was:Correct the access modifier of the processNextRecord member method of the
Scanner class
> Optimize the access modifier of
> AbstractHoodieLogRecordReader#processNextRecord
> -------------------------------------------------------------------------------
>
> Key: HUDI-7799
> URL: https://issues.apache.org/jira/browse/HUDI-7799
> Project: Apache Hudi
> Issue Type: Improvement
> Reporter: bradley
> Priority: Major
> Labels: pull-request-available
>
> Resolved in PR: https://github.com/apache/hudi/pull/11308
> AbstractHoodieLogRecordReader#processNextRecord access modifier is wrong and
> should be changed from public to protected.
> reason:
> processNextRecord is used to process HoodieRecord inside the class. Derived
> classes implement this abstract member method. The access modifier should not
> be public. For example, the derived class HoodieMergedLogRecordScanner
> implements processNextRecord, which is used to merge HoodieRecord.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
