cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 31359] - [PATCH] SOAP Flowscript Integration
Date Wed, 22 Sep 2004 12:25:36 GMT
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=31359>.
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=31359

[PATCH] SOAP Flowscript Integration





------- Additional Comments From vgritsenko@apache.org  2004-09-22 12:25 -------
Couple of suggestions:
 * WebServiceLoader can extend AbstractLogEnabled instead of
   implementing LogEnabled.
 * Please use org.apache.excalibur.store.Store.TRANSIENT_STORE
   instead of local cache Map.
 * IIUC, Cache lookup in WebServiceLoader.load() can go before
   attempt at reading (remote) WSDL.
 * IMHO, exceptions should extend ProcessingException
 * As a future extension, probably, WebServiceLoader should
   (sometimes) check modification timestamp of WSDL and attempt
   to re-compile and re-load it. It should also check if files
   are already generated and skip generation step - should be
   useful on restarts.

Other than that, it sounds like fun :)

Mime
View raw message