I found that your solution to the problem of default values for properties with DefaultPropertyPlaceholderConfigurer is the best I have seen. Both simple and efficient.

Bravo !

I use it now, I hope don’t mind !

Thanks for your creativity

Jean-Marie Galliot
System Architect

Turn-around times for spring-based apps are currently quite bad - my IDE knows how to update and reload a class file, but not how to reload a spring bean definition, which ought to be much simpler. It may take a different shape in the end, but if Spring is not supposed to go down as another Java Dinosaur Framework, there has to be some more dynamicity.

I don’t understand why there’s no comment there from any Spring people. Don’t they see the value of this??

The demon flag should not be necessary if the application context is
closed when the webapp context is destroyed. This should happen
automatically if you use a
ContextLoaderListener or a servlet-specific application context.

/**
* Load properties into the given instance.
*
* @param props
*            the Properties instance to load into
* @throws java.io.IOException
*             in case of I/O errors
* @see #setLocations
*/
@Override
protected void loadProperties(Properties props) throws IOException {
	if (this.locations != null) {
		for (int i = 0; i < this.locations.length; i++) {
				Resource location = this.locations[i];
				if (logger.isInfoEnabled()) {
					logger.info(”Loading properties file from ” + location);
				}
				InputStream is = null;
				try {
					is = new FileInputStream(location.getFile().getAbsolutePath());
					if (location.getFilename().endsWith(XML_FILE_EXTENSION)) {
						this.propertiesPersister.loadFromXml(props, is);
					} else {
						if (this.fileEncoding != null) {
							this.propertiesPersister
									.load(props, new InputStreamReader(is,
											this.fileEncoding));
						} else {
							this.propertiesPersister.load(props, is);
						}
					}
				} catch (IOException ex) {
					if (this.ignoreResourceNotFound) {
						if (logger.isWarnEnabled()) {
							logger.warn(”Could not load properties from ”
									+ location + “: ” + ex.getMessage());
						}
					} else {
						throw ex;
					}
				} finally {
					if (is != null) {
						is.close();
					}
				}
			}
		}
	}

Also, we must put the attribute <property name=”daemon” value=”true”></property> of the timer in the dynamic.xml example so Tomcat shuts down properly.

Thanks,
Prakash

Note I’ve added a link to the source code of DefaultPropertyPlaceholderConfigurer. I’ll have to check with my employer about the copyright of the Reloading… source code.

Cheers,
Axel

Merry X-MAS,
-Stefan