[Engine-devel] Questions about database changes
Yair Zaslavsky
yzaslavs at redhat.com
Wed Aug 28 06:56:26 UTC 2013
----- Original Message -----
> From: "Vitor de Lima" <vitor.lima at eldorado.org.br>
> To: engine-devel at ovirt.org
> Sent: Tuesday, August 27, 2013 4:18:12 PM
> Subject: [Engine-devel] Questions about database changes
>
> Hi everyone,
>
> I would like some feedback about how to create the upgrade script that
> creates columns for the architecture of each VM, template and cluster in the
> engine database.
>
> The changes #17853 and #16700 introduce a field (and the respective Enum)
> used to store the target architecture of a cluster, VM or template (which
> currently can be either x86_64, ppc64 or undefined).
> In their current state, these changes introduce a VARCHAR column storing the
> architecture, but now I wanted to implement an autocompleter for this field
> in the search backend, and it would be massively cleaner and easier to use
> the architecture field as an integer (since Enums that implement the
> Identifiable interface can use the EnumValueAutoCompleter class).
>
> Considering that these two changes are already in review, should I modify
> them directly to use an integer or should I create another patch that
> changes the column in the database? If I create another patch, should it
> modify the upgrade script from change #16700 or it must create another
> script that migrates the column from a VARCHAR to an INTEGER?
>
> Thanks,
> Vitor de Lima
If still under review, why not modify the existing patches for review? (i.e - use the same change-id where needed)
Cheers,
Yair
>
>
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
>
More information about the Devel
mailing list