Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mail_queue: fill maq_message_id column #352

Merged
merged 10 commits into from
Mar 8, 2018
Merged

mail_queue: fill maq_message_id column #352

merged 10 commits into from
Mar 8, 2018

Conversation

glensc
Copy link
Member

@glensc glensc commented Mar 6, 2018 •
edited
Loading

#140 added maq_message_id column (released as 3.1.1), but it was never filled by app itself.

this PR will fill the column in mail queue and restore filling it from maq_headers column

 == 20180306202659 EventumMaqMessageId: migrating
Total 278353 rows, this may take time. Please be patient.
...
 == 20180306202659 EventumMaqMessageId: migrated 4261.0148s

All Done. Took 4261.0750s```

NOTE: it's ok to hit `^C` to abort this patch, it can resume if retried.

@glensc glensc added this to the 3.4.0 milestone Mar 6, 2018
@glensc glensc self-assigned this Mar 6, 2018
@glensc glensc force-pushed the fill-maq_message_id branch 3 times, most recently from 6b6b1aa to 3450f0c Compare March 8, 2018 23:00
@glensc glensc force-pushed the fill-maq_message_id branch from 3450f0c to aa3f995 Compare March 8, 2018 23:05
@glensc glensc force-pushed the fill-maq_message_id branch from aa3f995 to d36f039 Compare March 8, 2018 23:05
@glensc glensc merged commit 733d166 into eventum:master Mar 8, 2018
@glensc glensc deleted the fill-maq_message_id branch March 8, 2018 23:07
@glensc glensc mentioned this pull request Mar 21, 2018
Merged
@glensc glensc mentioned this pull request Dec 26, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@glensc glensc

Labels
None yet
Milestone
3.4.0
Development

Successfully merging this pull request may close these issues.
1 participant
@glensc