viirya commented on a change in pull request #31275:
URL: https://github.com/apache/spark/pull/31275#discussion_r563331257



##########
File path: 
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/consumer/KafkaDataConsumer.scala
##########
@@ -192,6 +197,13 @@ private[consumer] case class FetchedData(
    * Returns the next offset to poll after draining the pre-fetched records.
    */
   def offsetAfterPoll: Long = _offsetAfterPoll
+
+  /**
+   * Returns the tuple of earliest and latest offsets that is the available 
offset range when

Review comment:
       Hm, I think this just follows other methods above and below.

##########
File path: 
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/consumer/FetchedDataPool.scala
##########
@@ -170,11 +170,14 @@ private[consumer] object FetchedDataPool {
   }
 
   private object CachedFetchedData {
+    import KafkaDataConsumer.AvailableOffsetRange

Review comment:
       Moved. Thanks.




----------------------------------------------------------------
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]

Reply via email to