Roiocam commented on code in PR #199:
URL: 
https://github.com/apache/pekko-persistence-jdbc/pull/199#discussion_r1617416082


##########
core/src/main/scala/org/apache/pekko/persistence/jdbc/query/scaladsl/JdbcReadJournal.scala:
##########
@@ -119,7 +117,9 @@ class JdbcReadJournal(config: Config, configPath: 
String)(implicit val system: E
   override def persistenceIds(): Source[String, NotUsed] =
     Source
       .repeat(0)
-      .flatMapConcat(_ => delaySource.flatMapConcat(_ => 
currentPersistenceIds()))

Review Comment:
   I have been roll back this change, and then increasing the 
readJournalConfig.refreshInterval to 10x times, but it will still be flaky. And 
then I made a change like this.
   
   
https://github.com/apache/pekko-persistence-jdbc/pull/199/commits/e91daa09dfb87e7ccf39f3f382fc77ba3e035027
   
   seems like `throttle` was useful than Source.tick()



-- 
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: notifications-unsubscr...@pekko.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org
For additional commands, e-mail: notifications-h...@pekko.apache.org

Reply via email to