You don't have to change this, it is more for your information

The ConnectionPool class isn't really necessary because it only configures the BasicDataSource which could be done using Spring instead.  Also it has a super class called DataSource which could be passed to DocController.  A sub-class of DataSource is org.apache.commons.dbcp.PoolingDataSource which could be used in the future to pool connections.