Conexão fechada

Em que momento você abre a conexão, na instanciação do Servlet?
Você poderia abrir a conexão quando o serviço é solicitado, preparar o retorno e fechar a conexão, mas isso pode fazer seu serviço ficar lento.

Abro aqui ó:

public class VisitorService {

    private Connection connection = new ConnectionDBFactory().getConnection();

Isso aí equivale a inicializar na instanciação.

Poderia sobrescrever os métodos init e destroy da classe Servlet.
Inicializar a conexão dentro do método init e fechar dentro do método destroy.

poque está dando commit ? o seu get está alterando algo ? o certo seria fechar no seu próprio dao ou talvez criar um filter.
Não entendo nem porque essa connection esta sendo passada como parametro, isola logo ela dentro do dao
vai ficar repetindo esse código a cada service ?

É fogo @igomes, vejo muitos colegas que fazem isso, o pior é que os professores ensinam dessa forma…
Ou seja, jogam a abstração e encapsulamento no lixo.

Estou usando um service para dar o commit ou o rollback e não no DAO pq tenho situações que preciso inserir ou alterar varias tabelas…ou seja… preciso garantir que tudo foi feito ou nada foi feito… se eu deixar com auto commit ou commit no método do DAO… se algo der errado no meio…algumas coisas ja estarão commitadas, correto? Por usar esse service antes do DAO criei a conexão nele e nao no DAO…

Mas estou aberto a sugestões…

Em algum lugar você chama explicitamente new VisitorService() ?

Cada vez que você fizer isso ele vai abrir uma conexão pelo que eu estou entendo,
cada vez que ele faz

private Connection connection = new ConnectionDBFactory().getConnection();

ele abre uma conexão, você pode trabalhar com uma conexão aberta só, garantindo que o getConnection seja chamado só uma vez, usando uma estrategia singleton.

ou utilizar alguma biblioteca para gerenciar os seus pulls de conexão.

Sim… no Servlet eu faço esse new…

Considerando que na sua infraestrutura de conexão seja algo parecido com pelo menos esse exemplo simples:

public class ConnectionFactory {
    public Connection getConnection() {
        try {
            return DriverManager.getConnection(
                "jdbc:mysql://localhost/seubanco", "root", "");
        } catch (SQLException e) {
            throw new RuntimeException(e);
        }
    }
}

E na sua classe de negócio:

public List<Tipo> atenderRequisicaoXYZ() throws Exception {
    Connection connection = new ConnectionFactory().getConnection();
    try {
        //sua lógica...
        //como já avisaram, não precisa de commit no caso de selects!
    }
} finally {
    connection.close();
}

Obs.: É um exemplo mais simples possível. Considerando que o seu método seja o método principal que atende a resposta final da funcionalidade. Desta forma haverá somente 1 conexão aberta durante todo o processamento da funcionalidade requisitada via HTTP pelo servlet.

Lembrando que naturalmente Web não tem estado, a cada requisição será aberta uma nova conexão com o banco, sendo que assim que terminar de acessar o banco deverá fechar a conexão.

É exatamente isso…a única diferença esta que :

Connection connection = new ConnectionFactory().getConnection();

eu instancio junto com a classe e não a cada método…

Então o problema pode ser em outro lugar que voce não tenha mostrado.

@javaflex Segue detalhes do código

Servlet

@WebServlet("/income")
public class IncomeServlet extends HttpServlet {
	private IncomeService incomeService = new IncomeService();

@Override
protected void doPost(HttpServletRequest req, HttpServletResponse res) throws ServletException, IOException {
    
          //SETs da income.....

          incomeService.insert(income);
    }
}

Service

public class IncomeService {
	
	private Connection connection = new ConnectionDBFactory().getConnection();
	private IncomeDao incomeDao = new IncomeDao(connection);
	private PeriodDao periodDao = new PeriodDao(connection);

	public Income insert(Income income) throws Exception{
		try{
			
			Period period = periodDao.findByMonthAndYear(income.getPeriod().getMonth(), income.getPeriod().getYear());
			
			if(period == null){
				period = periodDao.insertObject(income.getPeriod());
			}
			
			income.setPeriod(period);
			income = incomeDao.insertObject(income);
			connection.commit();
			return income;			
			
		}catch (Exception e) {
			connection.rollback();
			throw new RuntimeException(e);
		}
	}

}

Acredito que você tenha resolvido o meu problema quando sugeriu que eu fizesse Connection connection = new ConnectionDBFactory().getConnection(); e um connection.close() a cada método e não um Connection connection = new ConnectionDBFactory().getConnection(); na instancia do Service.

Só ficou uma pequena dúvida, quanto a repetição do Connection connection = new ConnectionDBFactory().getConnection(); a cada método. Acha que eu poderia melhorar isso?