warrenzhu25 edited a comment on pull request #28887: URL: https://github.com/apache/spark/pull/28887#issuecomment-647238451
> Probably, it would be nice to explain the difference between Optional.orElse vs Optional.orElseGet for Java a bit in PR description. It's very confusing in Scala developers as we expect lazy evaluation natively. > > And one more, please rebase the target branch to master. We normally receive patches against master branch. If the problem no longer exists in master branch, we may want to find and port back the commit which fixed the issue previously, instead of picking up newer commit. 1. Updated description to explain more about Optional.orElse vs Optional.orElseGet 2. In master branch, KafkaContinuousReader.scala has been refactored out in e75488718. This file is only in branch-2.4. Any suggestions about how should I do? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
