@Daimona I seem to recall I had the same question when we all discussed it! 😆

In T312870#8197555, @MHorsey-WMF wrote:

@Daimona I seem to recall I had the same question when we all discussed it! 😆

You definitely did, I remember that! And I also remember me answering that question, just to realize that the use case I had in mind was actually wrong. Then I think I came up with some possible alternative use case, but it didn't convince me. All in all, I think we don't need this field, as it would only make the queries more complex.

In T312870#8194071, @Daimona wrote:

I'm working on this, but I'm wondering if we need the "updated_at" field. What was its purpose? Also, this is a field that we could add in the future if need be, and it could be backfilled with the value of created_at. @cmelo @MHorsey-WMF @ifried @vyuen any thoughts?

I think we really don’t need the "updated_at" it will not make any difference, also in the future we will create logs of user actions (probably), so this is really not needed.
I think you can go ahead and create only the other columns, thanks @Daimona !

Change 828027 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Add creation and deletion timestamp to ce_organizers table

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

Change 828027 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Add creation and deletion timestamp to ce_organizers table

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

Change 827580 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Add creation and deletion timestamp to ce_organizers table

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

These are the commands that I will run on beta:

ALTER TABLE ce_organizers ADD COLUMN ceo_created_at BINARY(14) NOT NULL DEFAULT '20220718130000';
ALTER TABLE ce_organizers ADD COLUMN ceo_deleted_at BINARY(14) DEFAULT NULL;

Then, after the patch is merged:

ALTER TABLE ce_organizers ALTER ceo_created_at DROP DEFAULT

Mentioned in SAL (#wikimedia-releng) [2022-08-31T16:21:18Z] <Daimona> Applying schema change to the wikishared DB on beta for the CampaignEvents extension (1/2) # T312870

Change 827580 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Add creation and deletion timestamp to ce_organizers table

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

Mentioned in SAL (#wikimedia-releng) [2022-08-31T16:37:04Z] <Daimona> Applying schema change to the wikishared DB on beta for the CampaignEvents extension (2/2) # T312870

Patch merged, and beta DB updated. This is now ready for QA.

@vaughnwalters To see the schema: SSH to deploy03 and connect to the DB with sql wikishared.

Then you want to check that table definitions (column types, indexes etc.) and make sure that they match the code definition.

To see a table definition: DESCRIBE my_table_name;. To see the indexes on a table: SHOW INDEX FROM my_table_name;

SHOW INDEX FROM ce_organizers;

DESCRIBE ce_organizers;

From the code definitions

AC passes. ce_organizers table now has fields ceo_created_at_and ceo_deleted_at. Marking this as done.