I think the requirements for user properties (CAY-400) and comments/description (CAY-659) are different.
While CAY-400 could be used to support comments, and other things like meta-data, I think getting the comments/description done as a modeler enhancement is better done separately. Editing a simple description field will be easier to use than arbitrary lists of user defined properties. The danger with CAY-400 is the use-cases/requirements are pretty vague, making it hard to dermine what's needed. I think this is why this feature has stalled. Adding description fields to the Modeler is a much simpler requirement, which shouldn't be stalled as well. Regarding the design, loading the comments only when using the modeler sounds fine to me. I can't imagine people pasting a Word document into a 30 character length text field, however I could be wrong. regards Malcolm Edgar On 2/1/07, Aristedes Maniatis <[EMAIL PROTECTED]> wrote:
On 01/02/2007, at 9:28 PM, Andrus Adamchik wrote: > We are talking about BLOBS of text. Consider people using this for > javadocs, with each attribute having a 100 char comment field. For > the model of 50 entities with 20 attributes each, we have (50 + > 50*20) * 100 = 102K. Not crucial, but still keeping this stuff > around in runtime seems wrong. Those things add up over time, > resulting in framework becoming heavier with every new release. Not to mention it might contain private notes we may not want in a public release of a product. I don't want our entity documentation released to the world. How about a velocity(?) script which could strip some parts of the XML file for deployment? As long as they were easily identifiable, we could even put a little regex into our main ant build script for deploying the application. On the other hand, a separate config file for comments would make this easier... Ari --------------------------> ish http://www.ish.com.au Level 1, 30 Wilson Street Newtown 2042 Australia phone +61 2 9550 5001 fax +61 2 9550 4001 GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
