spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [spark] warrenzhu25 opened a new pull request #28887: [SPARK-32044][SS] Kakfa continuous processing print mislead initial o…
Date Sun, 21 Jun 2020 18:29:15 GMT

warrenzhu25 opened a new pull request #28887:
URL: https://github.com/apache/spark/pull/28887


   ### What changes were proposed in this pull request?
   
   Use `Optional.orElseGet` instead of `Optional.orElse` to fix unnecessary kafka offset fetch
and misleading info log
   
   ### Why are the changes needed?
   
   Fix mislead initial offsets log and unnecessary kafka offset fetch
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   ### How was this patch tested?
   
   Currently, no test for KafkaContinuousReader. Also it's hard to test log.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message