Ask remote servers for individual unknown events, take 2 #976
Labels
No labels
Bug
Cherry-picking
Database
Dependencies
Dependencies/Renovate
Difficulty
Easy
Difficulty
Hard
Difficulty
Medium
Documentation
Enhancement
Good first issue
Help wanted
Inherited
Matrix/Administration
Matrix/Appservices
Matrix/Auth
Matrix/Client
Matrix/Core
Matrix/Federation
Matrix/Hydra
Matrix/MSC
Matrix/Media
Meta
Meta/CI
Meta/Packaging
Priority
Blocking
Priority
High
Priority
Low
Security
Status/Blocked
Status
Confirmed
Status
Duplicate
Status
Invalid
Status
Needs Investigation
Support
To-Merge
Wont fix
old/ci/cd
old/rust
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
continuwuation/continuwuity!976
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "ginger/continuwuity:single-event-backfill"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
From #899:
dist/and consolidate the service filesdist/topkg/39f92f107007d0e672af07d0e672af21dd09a6b6@ -21,2 +3,2 @@}}/// Parse a pagination tokenpub(crate) fn pagination_token_to_count(token: &str) -> Result<PduCount> { token.parse() }these are just wrappers around the functions on the
PduCountstruct. why even keep this file if these are the only functions in it?@ -135,0 +140,4 @@if local.is_ok() {// We already have this PDU, no need to backfilldebug!("We already have {event_id} in {room_id}, no need to backfill.");return local;strange that a function with
remotein the name can return local PDUs sometimesalso, this function is similar in implementation and purpose to
fetch_and_handle_outliersThis entire system needs to be revisited once continuwuity has better support for holes in the local event timeline. Outlier events work here but they're kind of a hack, and the context route is pretty much impossible to implement.
WIP: Ask remote servers for individual unknown events, take 2to Ask remote servers for individual unknown events, take 28d353c90f3a82470633ea82470633e8e7ee1e210Just re-running CI then I think we're good for merge
That was supposed to be an approval
8e7ee1e210350aa040a2445b8d030f90fd92977esrc/api/client/utils.rs1e541875adfullfeature