David M Johnson wrote:
On Mar 28, 2006, at 12:26 PM, Allen Gilliland wrote:
sounds good. i'll wait until that's done then.
It would be helpful if you could start reviewing this stuff now, I'd
like to make the move before COB today (and produce another RC today)
since I'm out for the rest of the week.
we might want to consider making the enable/disable flags part of the
runtime config rather than the static config. it is sometimes tough
to decide where certain properties should go, but these are my
thoughts ..
Yes I agree, but since these are experimental features I want to keep
the profile low and not have them appear in the UI. Once Atom protocol
is final and Admin protocol has had sufficient review, I'd like to make
them runtime configs and document them.
eeks, now we are working backwards. if these things are still
"experimental features" and need to be kept low profile then i am -1 for
moving them out of the sandbox.
also, after "Admin protocol has had sufficient review"? that review
needs to happen before we move it out of the sandbox and package the
code into a release.
i am okay with commiting them to the core code if the implementation is
complete but may have some changes in the future, but we still need to
treat them as if these are fully working and fully functional features.
if we don't believe that is true then we can't release them.
now that i think about it, maybe it is not a good idea to squeeze this
in at such a late time in the release process.
-- Allen
- Dave