[DBUTILS-52] QueryRunner is not thread-safe

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2
• Assignee: Daniel Fabulich
• Reporter: Daniel Fabulich
• Created: 11/Mar/09 16:54
• Updated: 15/Mar/09 19:05
• Resolved: 15/Mar/09 19:05

Description

sebb pointed out: "QueryRunner Javadoc says the class is thread-safe. However it has a protected mutable variable DataSource which can also be set/got via public methods. If one thread sets the variable, another may not see the correct value, so the class is not thread-safe."

We should make the DataSource final and remove the setter.