kowshik commented on a change in pull request #10280:
URL: https://github.com/apache/kafka/pull/10280#discussion_r602056245
##########
File path: core/src/main/scala/kafka/log/Log.scala
##########
@@ -213,32 +208,38 @@ case object SnapshotGenerated extends
LogStartOffsetIncrementReason {
}
/**
- * An append-only log for storing messages.
+ * A log which presents a unified view of local and tiered log segments.
*
- * The log is a sequence of LogSegments, each with a base offset denoting the
first message in the segment.
+ * The log consists of tiered and local segments with the tiered portion of
the log being optional. There could be an
+ * overlap between the tiered and local segments. The active segment is always
guaranteed to be local. If tiered segments
+ * are present, they always appear at the head of the log, followed by an
optional region of overlap, followed by the local
Review comment:
Done. Good point. I've fixed the doc to refer to `beginning of 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:
[email protected]
