From b1000cd5c62da6d537f281688b338955f0f5e748 Mon Sep 17 00:00:00 2001
From: Keith Donald <kdonald@vmware.com>
Date: Wed, 22 Apr 2009 17:47:44 +0000
Subject: [PATCH] embedded

---
 .../jdbc/datasource/embedded/EmbeddedDatabaseBuilder.java       | 2 ++
 .../jdbc/datasource/embedded/EmbeddedDatabaseConfigurer.java    | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseBuilder.java b/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseBuilder.java
index ce77dd67f9..cb51166c87 100644
--- a/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseBuilder.java
+++ b/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseBuilder.java
@@ -29,6 +29,8 @@ import org.springframework.core.io.ResourceLoader;
  * db.shutdown();
  * </pre>
  * 
+ * TODO - should we replace schema/testdata with more general 'script' method?
+ * 
  * @author Keith Donald
  */
 public class EmbeddedDatabaseBuilder {
diff --git a/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseConfigurer.java b/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseConfigurer.java
index 534277a864..a96c753970 100644
--- a/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseConfigurer.java
+++ b/org.springframework.jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseConfigurer.java
@@ -23,6 +23,8 @@ import org.springframework.jdbc.datasource.SimpleDriverDataSource;
  * Encapsulates the configuration required to create, connect to, and shutdown a specific type of embedded database such as HSQLdb or H2.
  * Create a subclass for each database type we wish to support.
  * 
+ * TODO - introduce ConfigurableDataSource strategy for configuring connection properties?
+ * 
  * @see EmbeddedDatabaseConfigurerFactory
  */
 abstract class EmbeddedDatabaseConfigurer {