868167 - Move the current cf_due_date from type DATETIME to DATE once bug 866248 has landed on prod

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Administration, task)

Description

[David Lawrence [:dkl]]

Once we add support for new date only custom field (bug 866248) we should convert the current Due Date (cf_due_date) to it. Looking at the raw data, no one so far has stored an actual time in the column so converting will not cause any data loss.

We should of course try this on the test instances to verify before product. 

This will also require a custom one-off script to update the BZ database schema properly and will be run by IT.

dkl

Comment 1

[David Lawrence [:dkl]]

Attachment: Patch to convert a datetime column to date (v1)

Comment 2

[Tara (musingt)]

(yay - I can't wait).

Comment 3

[:glob ✱]

Comment on attachment 745168 [details] [diff] [review]
Patch to convert a datetime column to date (v1)

Review of attachment 745168 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

::: contrib/reorg-tools/convert_date_time_date.pl
@@ +18,5 @@
> +use Bugzilla::Constants;
> +
> +sub usage() {
> +  print <<USAGE;
> +Usage: convert_date_time_date.pl <table> <column>

<table> is always going to be bugs, there's no need to require this as a parameter.

@@ +24,5 @@
> +E.g.: convert_date_time_date.pl bugs cf_due_date
> +Converts a datetime field (FIELD_TYPE_DATETIME) to a date field type
> +(FIELD_TYPE_DATE).
> +
> +Note: Any specific time portion will be lost but the specific date portion will be preserved.

nit: s/specific//g

@@ +44,5 @@
> +
> +print <<EOF;
> +Converting ${table}.${column} from FIELD_TYPE_DATETIME to FIELD_TYPE_DATE.
> +
> +Note: Any specific time portion will be lost but the specific date portion will be preserved.

nit: s/specific//g

@@ +56,5 @@
> +                  undef, FIELD_TYPE_DATE, $column);
> +
> +print "\ndone.\n";
> +
> +exit(0);

exit() at the end of a script doesn't do anything :)

Comment 4

[David Lawrence [:dkl]]

Committing to: bzr+ssh://[e-mail]/bmo/4.2             
added contrib/reorg-tools/convert_date_time_date.pl
Committed revision 8780.

Comment 5

[Tara (musingt)]

wohoo !!!