Является ли эта фабричная нить NpgsqlConnection безопасной?

Question

Безопасен ли этот код?

public static NpgsqlConnection getConnection() 
{   
    NpgsqlConnectionStringBuilder csb = new NpgsqlConnectionStringBuilder(); 
    csb.Host = ConfigurationManager.AppSettings["server"]; 
    csb.Password = ConfigurationManager.AppSettings["password"]; 
    csb.Pooling = true; 
    csb.UserName = ConfigurationManager.AppSettings["username"]; 
    csb.Port = Convert.ToInt32(ConfigurationManager.AppSettings["port"]); 
    csb.Enlist = true; 
    //csb.Database = ConfigurationManager.AppSettings["databaseName"]; 

    return new NpgsqlConnection(csb.ConnectionString); 
} 

Я использую эту связь везде, но иногда я получаю TimeoutException.

Произошел перерыв. Если вы установили соединение, увеличьте значение Timeout в ConnectionString. Если вы выполняете команду, увеличьте значение CommandTimeout в ConnectionString или в объекте NpgsqlCommand.

Любые идеи?

Answer 1

Нет, это не потокобезопасно, потому что csb не является локальным и, вероятно, не является потокобезопасным. Используйте локальный csb. Проверьте, является ли ConfigurationManager потокобезопасным.

Инициирование самого соединения является потокобезопасным, поскольку другие потоки не могут «видеть» экземпляр. Проблемы с подключением, вероятно, происходят из другого места. Вы можете попробовать, что было дано в качестве подсказки в сообщении об ошибке.

Answer 2

Отвечая на ваш вопрос, NpgsqlConnection не является потокобезопасным. Вы не должны делиться NpgsqlConnection с несколькими потоками.

Но в отношении вашего исключения, если вы заметили, вы получите его, когда вы вызываете метод NpgsqlConnection.Open(). И в вашем методе getConnection() вы не вызываете Open. Вы открываете соединение где-то еще.

Теперь, чтобы получить это исключение при открытии соединения, вы должны достигнуть максимального размера пула подключений. Это означает, что вы не можете закрывать предыдущие открытые соединения.

Вам нужно либо закрыть, либо закрыть объект NpgsqlConnection, чтобы он возвращался в пул соединений. Если вы позволяете экземпляру NpgsqlConnection выйти из области видимости, он не будет возвращен в пул и будет считаться использованным соединением.

Для того, чтобы проверить, если это ваша проблема, у вас есть две возможности:

1. Отключить пул соединений. Вы больше не будете получать это исключение, но поскольку вы не закрываете соединения, вы можете в конечном итоге достичь максимальных подключений, разрешенных для каждой базы данных. Это параметр сервера postgresql.

2. Позвоните в NpgsqlConnection.ClearPool или ClearAllPool, чтобы удалить все соединения из пула.

Обратите внимание, что эти две возможности должны использоваться только для проверки наличия проблем с пулом подключений. Если ваше приложение начинает работать нормально после использования одного из этих методов, ваше приложение все еще имеет проблемы, которые необходимо устранить.

Надеюсь, это поможет.