rmatharu commented on code in PR #1598:
URL: https://github.com/apache/samza/pull/1598#discussion_r864343575
##########
samza-core/src/main/scala/org/apache/samza/checkpoint/OffsetManager.scala:
##########
@@ -461,12 +470,19 @@ class OffsetManager(
val checkpoint = checkpointManager.readLastCheckpoint(taskName)
- if (checkpoint != null) {
- Map(taskName -> checkpoint.getOffsets.asScala.toMap)
- } else {
- info("Did not receive a checkpoint for taskName %s. Proceeding without a
checkpoint." format taskName)
- Map(taskName -> Map())
+ if (!elasticityCheckpointsEnabled) {
Review Comment:
Could just be elasticityEnabled ?
Branching could occur only when it is enabled, otherwise default to default
path?
--
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: commits-unsubscr...@samza.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
