Page MenuHomePhabricator

Android schemas - add `is_temp` column for currently active MEP schemas
Closed, ResolvedPublic

Description

In preparation for the addition of Temporary Account data we will need to add a column, is_temp (boolean datatype) to currently active schemas to accommodate this data. This change only needs to be made on MEP schemas as MP schemas already contain the performer.is_temp column (these already migrated schemas are not listed below).

SchemaColumn to adddatatypeStatus
app_sessionis_tempbooleanTO DO
app_interaction (stream template)is_tempbooleanTO DO
android_breadcrumbs_eventis_tempbooleanTO DO
android_image_recommendation_eventis_tempbooleanTO DO
android_daily_statsis_tempbooleanTO DO
android_install_referrer_eventis_tempbooleanTO DO
android_notification_interactionis_tempbooleanTO DO
android_user_contribution_screenis_tempbooleanTO DO

Note: There are several schemas that are still collecting data and may still come into play further down the road but for now these are low priority, depending on difficulty/time required to adjust these schemas we should revisit after the above list is complete.

Android Not Currently Used Schemas

  • android_create_account_interaction
  • android_customize_toolbar_interaction
  • android_edit_history_interaction
  • android_app_appearance_settings_interaction
  • android_article_page_scroll_interaction
  • android_reading_list_interaction|

Event Timeline

@SNowick_WMF Note that android_article_page_scroll_interaction and android_reading_list_interaction do not seem to be used in the app.

@Dbrant per T374940: Ensure performer attributes in schemas clarify if the user is a temporary account, there is a fragment fragment/mediawiki/state/entity/user in the primary schemas repo for a performer object, which contains already is_temp. I'm not sure if you want to use that fragment, but just commenting here as a heads up in case it's useful.

Change #1077738 had a related patch set uploaded (by Dbrant; author: Dbrant):

[schemas/event/secondary@master] Add is_temp field to Android schema fragment.

https://gerrit.wikimedia.org/r/1077738

Change #1077738 merged by jenkins-bot:

[schemas/event/secondary@master] Add is_temp field to Android schema fragment.

https://gerrit.wikimedia.org/r/1077738