Skip to content

Adds seen entities to vitess #2363

Mark Harding requested to merge feat/2363-vitess-seen-entities into master

Ticket(s) / Related Merge Requests

#2363 (closed)

Summary of Changes

Refactors 'seen entities' logic to use Vitess as storage instead of Redis.

Testing Considerations

Browse the sandbox and check to see if rows are added to the table.

select * from pseudo_seen_entities;

Deployment Considerations

Ensure new table is added to prod.

CREATE TABLE IF NOT EXISTS pseudo_seen_entities(
  pseudo_id varchar(128),
  entity_guid bigint,
  last_seen_timestamp timestamp,
  PRIMARY KEY(pseudo_id, entity_guid)
) ENGINE=InnoDB;

Regression Scope

Seen entity logic

Platform Affected (web, mobile, etc)

API

Developer Testing Completed

@markharding 👍

Screenshots / Screen Recording

Screenshot_2022-07-27_at_14.13.42

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Mark Harding

Merge request reports