Optimizing layout
Pull request merged

Phil/controller backoffs

Description:

Resolves #2032
Adds backoffs to controller-initiated publications and discovers, so that they don't retry them too frequently when the controller is invoked frequently for various different reasons. This ought to dramatically reduce the rate of failed publications and discovers, which are being run due to frequent shard failures, which trigger controller runs.

In all of these, the goal is to still show the controller as being in error status when it's waiting on a backoff, so that failures are still obvious when looking at the overall status.

Also fixes a bug in the runtime, which caused us to fail to set lastSourcePublishedAt in materialization stats documents if no documents were read from the source.

see less
File matrix
14 files14 files
r1
r2
r3
r4
r5
r1
r2
r3
r4
r5
crates/crates/
agent/src/agent/src/
controllers/controllers/
r3
r4
r5
r1
r2
r3
r4
r5
r1
r2
r3
r4
r5
r1
r2
r3
r4
r5
capture/capture/
r2
r3
r4
r5
integration_tests/integration_tests/
r2
r3
r4
r5
r3
r4
r5
r3
r4
r5
r1
r2
r3
r4
r5
r3
r4
r5
r4
r5
models/src/status/models/src/status/
r1
r2
r3
r4
r5
runtime/src/materialize/runtime/src/materialize/
r5
+39
+39
adjust what's shown in the file matrix here.
Participants
waiting on
participant
status
role
contributor
last active
drafts
discussions
Diffs
  Reviewing the latest revision (r5) against its ⊥ base revision.

Next diffs to review (file selection:

)

Show other diffs: All changes

Preferences
Top level discussions
Review discussion
6 days ago
New comment
jgraettingerJohnny Graettinger

LGTM

r1
r2
r3
r4
r5
+39
Revision mapping:
Compact 5 revisions Click and hold to arm, release to activate.
r3
r4
r5
r1
r2
r3
r4
r5
r1
r2
r3
r4
r5
r1
r2
r3
r4
r5
r2
r3
r4
r5
r2
r3
r4
r5
r3
r4
r5
r3
r4
r5
r1
r2
r3
r4
r5
r3
r4
r5
r4
r5
r1
r2
r3
r4
r5
r5
Use of Reviewable is subject to the terms of useprivacy policy.