-
Notifications
You must be signed in to change notification settings - Fork 4.1k
colexec: 'Operator.Init' should only be called once #44145
Copy link
Copy link
Closed
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
Description
I think we should modify Operator interface so that Init could be called at most once. It would have prevented a crash like #44133, but also it is beneficial from the memory allocations stand point (since we often allocate new stuff on Init).
If we do decide to modify the interface, one way to enforce that we follow it is to modify invariantsChecker to call Init multiple times on its input.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.