Page MenuHomePhabricator

[lib] remove checking for message IDs collision
ClosedPublic

Authored by kamil on Oct 2 2024, 6:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 2:24 PM
Unknown Object (File)
Mon, Dec 16, 2:24 PM
Unknown Object (File)
Mon, Dec 16, 2:24 PM
Unknown Object (File)
Mon, Dec 16, 2:24 PM
Unknown Object (File)
Wed, Dec 11, 9:30 PM
Unknown Object (File)
Dec 4 2024, 12:19 AM
Unknown Object (File)
Nov 29 2024, 3:04 AM
Unknown Object (File)
Nov 27 2024, 8:11 AM
Subscribers
None

Details

Summary

Short term fix to: ENG-9464.

There is a comment in ENG-9468 to came up with correct solution.

Test Plan

On the master:

  1. Open physical device
  2. In DM thread send a test message
  3. Notif delivered
  4. Message not visible in UI
  5. Seeing log: Discarded a send_text_message operation because message with the same ID 27caa155-d435-4ef8-9d8e-5b0862861e02 already exists in the store
  6. Kill/restart the app - message is visible regardless of network/TB connection.

Apply this diff:

  1. Open physical device
  2. In DM thread send a test message
  3. Notif delivered
  4. Message immediately visible in UI
  5. Not seeing any log about the discarded operation.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Oct 2 2024, 6:27 AM
kamil edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Oct 2 2024, 6:32 AM