On Feb 1, 2012, at 3:35 PM, Christophe Laprun wrote:


On Feb 1, 2012, at 2:51 PM, Julien Viet wrote:

and I think we should make it part of the existing configuration with proper prefixing in a dedicated section.

The point of having a dedicated configuration file is that:
1. it's easy to change for users as they don't risk messing up the rest of the configuration
2. it's faster to load since it doesn't need to load things that have nothing to do with the specific configuration we're interested in
3. it doesn't pollute the more important configuration by avoiding putting the validation configuration there when it's likely going to be removed when we switch to Bean Validation (which I understand is the long term goal). The main configuration.properties file should be used for "vital" configuration things, not optional ones, in my opinion.
4. it's easier to check whether the default behavior can be used or not (otherwise, need to check that *all* the possible configuration keys are missing)

That said, if people think it's better to put it in the configuration.properties file, I'll change the implementation accordingly.

We keep a single file that is the place where you will find any configuration property.

We thought about modularizing at some point, but not according to the function but instead to the scope of the property (admin, dev).


however for the case we need to avoid case like "minLength" and stick to lower case (except the mistake that was done for gatein.email.smtp.*).

Is there a point to using only lower case (apart from consistency with the other keys, which will not be an issue if the configuration is made in a separate file)?

consistency matters, it's not only narcissistic and assumed, it also allows one to not wonder if a letter of a property should be upper or lower case, it is just lower case.

it should be used whenever we define a property name (configuration file, runtime properties like <qname xmlns:prp='http://www.gatein.org/xml/ns/prp_1_0'>prp:navigation_uri</qname>, etc...). 


Cordialement / Best,
Chris

==
Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc.
Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein
Follow me: http://metacosm.info/metacosm / http://twitter.com/metacosm