nickva commented on code in PR #5036:
URL: https://github.com/apache/couchdb/pull/5036#discussion_r1577252745
##
src/couch_replicator/src/couch_replicator_job_sup.erl:
##
@@ -1,34 +0,0 @@
-% Licensed under the Apache License, Version 2.0 (the "License"); you may not
Review
nickva opened a new pull request, #5036:
URL: https://github.com/apache/couchdb/pull/5036
Use the scheduler as the job supervisor, since the scheduler is already a
fancy supervisor, with its own backoff logic, process monitoring, etc.
This simplifies the job starting/stopping logic
Boo, we failed.
https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/main/1078/display/redirect
Boo, we failed.
https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/main/1077/display/redirect
nickva merged PR #5035:
URL: https://github.com/apache/couchdb/pull/5035
--
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:
nickva opened a new pull request, #5035:
URL: https://github.com/apache/couchdb/pull/5035
Use maps and a few helper function to make it less verbose.
Otherwise it should be the exact same specs as before.
References: