You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by bu...@apache.org on 2005/02/14 10:16:24 UTC

DO NOT REPLY [Bug 33553] - [configuration] Cache DatabaseConfiguration values for higher performance

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33553>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33553


ebourg@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Cache DatabaseConfiguration |[configuration] Cache
                   |values for higher           |DatabaseConfiguration values
                   |performance                 |for higher performance




------- Additional Comments From ebourg@apache.org  2005-02-14 10:16 -------
Adding a cache to DatabaseConfiguration is a good idea. However I'm not sure I
would rely solely on the cache for some operations like isEmpty() or
containsKey(). The cache should be first checked, and then the database should
be queried if necessary. Checking if the cache is empty isn't enough to make
sure the configuration is empty.

Just out of curiosity, why did you change the private members to final ?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org