Hey,
we have many duplicate attributes at connector and protocolhandler classes-
I have spent a lot of time to fix that at my new server.xml/context.xml storage
module.
Current design based on factory model with external description file to handle those issues.
Very strange, but my only chance to implement the handler, without break the current codebase.
We must refactor the Connector/Protocoll class to make a clear cut,
between old and correct attributes. We drop maxProcessors and minProcessors attributes as example.
My current questions are:
Why we handle all connectors attributes as real class attributes
and dymamic properties?
Can we drop the attribute translation between Connector and ProtocolHandler?
Regards Peter PS: I am waiting to get cvs write access....
Shapira, Yoav schrieb:
Hi,
I'm going to wait for Peter for save-to-XML rewriting, before bothering
with fixing the various issues.
(right now, saving data sources needs to be redone as well)
I contemplated that, but it seems that here the core problem is not the persisting itself: it's the duplication of attributes. Even if the XML persistence issue is fixed soon (I have faith ;)), it'll be hackish to have it check the buffer to see if an attribute has been persisted, and only persist it if not. It's an OK fix, but doesn't address the core issue in this particular case, does it? ;)
Yoav
This e-mail, including any attachments, is a confidential business communication, and may contain information that is confidential, proprietary and/or privileged. This e-mail is intended only for the individual(s) to whom it is addressed, and may not be saved, copied, printed, disclosed or used by anyone else. If you are not the(an) intended recipient, please immediately delete this e-mail from your computer system and notify the sender. Thank you.
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
