Skip to content

Update SyncStatusManagerImpl and LoggingIdentifierController to use StateFlow instead of AtomicReference #4249

Closed
@BenHenning

Description

@BenHenning

SyncStatusManagerImpl and LoggingIdentifierController (introduced as part of fixing #4064) need to track thread-safe state. Doing this correctly without running into ordering problems in its fake requires leveraging a StateFlow property (which can't be introduced until coroutines are updated which is being done as part of #4239.

Activity

changed the title Update SyncStatusManagerImpl to use StateFlow instead of AtomicReference Update SyncStatusManagerImpl and LoggingIdentifierController to use StateFlow instead of AtomicReference on Mar 15, 2022
added a commit that references this issue on May 5, 2022

Fix #4249, part of #4064: Domain components for learner analytics (#4267

a7c8e6c
added
Z-ibtTemporary label for Ben to keep track of issues he's triaged.
on Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Z-ibtTemporary label for Ben to keep track of issues he's triaged.

    Type

    No type

    Projects

    • Status

      No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Update SyncStatusManagerImpl and LoggingIdentifierController to use StateFlow instead of AtomicReference · Issue #4249 · oppia/oppia-android