linliu-code commented on code in PR #9593: URL: https://github.com/apache/hudi/pull/9593#discussion_r1315967441
########## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/Iterators.scala: ########## @@ -116,16 +116,16 @@ class LogFileIterator(logFiles: List[HoodieLogFile], // NOTE: This have to stay lazy to make sure it's initialized only at the point where it's // going to be used, since we modify `logRecords` before that and therefore can't do it any earlier - private lazy val logRecordsIterator: Iterator[Option[HoodieRecord[_]]] = + private lazy val logRecordsIterator: Iterator[scala.Option[HoodieRecord[_]]] = Review Comment: Because Hudi uses its own Option keyword; and scala has its own. There are two ways to differentiate them: one to add scala prefix to scala Option; one to using alias for Hudi Option, like HOption. I will probably do the later one and will update. -- 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]
