Eddy Ashton
Eddy Ashton
This was done as part of #2801. In moving `NodeToNodeChannelManager` to its own file, we also removed the `destroy_channel` paths which filled the map with tombstone null pointers.
@achamayou - Correct, this is not yet resolved. I tried removing those lines in #5160, but that resulted in test failures as some nodes are _still_ emitting this on shutdown.
The original goal is now a non-goal. We found a use for the single central list of all service tables - auto-generating HTTP wrappers for them (#4659). So this issue...
Protobuf v22 includes a hard dependency on Abseil, so we don't want to include that as a dependency for all CCF apps. We think that's fine - the framework can...
This was completed in #4316. (Caveat - we don't support "forwarding" with executors, but that's because we don't support forwarding over HTTP/2 at all, and are looking at replacing forwarding...
TODO before merging: - [x] Rename `IExtension` - [x] Rename the extensions for less `Ccf` repetition - [x] Manually verify what we expose in each execution context hasn't changed, see...
> take a look at Basic too, which seems to have dropped, perhaps because of programmability Fiddling locally, I'm seeing ~56k with the current `basic` (with programmability). If I remove...
Update after further discussion: 1. It's tricky to show _votes_ for **open** proposals, since the _votes_ (boolean result) might change at any time. But we can show the IDs of...
Another repro here: https://dev.azure.com/MSRC-CCF/CCF/_build/results?buildId=66163&view=logs&j=8f3dc89c-3708-5926-47e7-27120a268dab&s=96ac2280-8cb4-5df5-99de-dd2da759617d&t=bb1a7e6d-8f5b-56e4-638c-b498b20b4b62&l=1060 Looks extremely rare, but still present. I'll reopen and try to look at this soon.