On Sep 22, 2010, at 4:18 AM, Piotr Solnica wrote: > On Sep 22, 12:53 pm, postmodern <[email protected]> wrote: >>>> Also, I thought we could eventually kick the Enum/Flags types out of >>>> dm-types and into >> dm-types-legacy. > > That's a tough one. Dan always advocates people to use String > with :set option instead of the properties you mention. Personally I > tend to agree as right now there's no big value in using them and I > actually don't consider them as stable :/ Fortunately there is a plan > to make things better (talking again about final Property/adapter API > changes) and having that done Enum could be better supported so that > various adapters could use native Enum fields for this property. The > question is do we really need that? I've heard various opinions about > Enum fields so I'm not sure if it's worth the effort.
My $0.02: two thumbs down on the deprecation / separation / removal / obfuscation / etc of Enum. I use them everywhere, and to great effect. So far the only arguments I've heard against Enum are either philosophical in nature, or "it's hard to make them work well" / etc. One of DM's great strengths is flexibility and it seems completely counter to this value to deprecate / excise / further decouple Enum from the core libraries on the basis of philosophy alone. I haven't yet seen any real meat to the "hard to make work well" / "it's a PITA" / etc argument, though I'm definitely interested in hearing a cogent explanation. Finally, the String set idea is less performant at the RDBMS level, yet I don't understand what benefit it brings over Enum? Can someone explain what aspects of a set of strings make it a better design choice than a set of integers, especially over large data sets? cheers, --jordan -- You received this message because you are subscribed to the Google Groups "DataMapper" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/datamapper?hl=en.
