Hi, On 2023-03-03 11:18:04 +1100, Peter Smith wrote: > - Why is reducing members of LogicalDecodingContext even a goal? I > thought the LogicalDecodingContext is intended to be the one-stop > place to hold *all* things related to the "Context" (including that > member that was deleted).
There's not really a reason to keep it in LogicalDecodingContext after this change. It was only needed there because of the broken architectural model of calling UpdateProgress from within output plugins. Why set a field in each wrapper that we don't need? > - How is reducing one member better than introducing one new parameter > in multiple calls? Reducing the member isn't important, needing to set it before each callback however makes sense. Greetings, Andres Freund