импорт из текстового файла в базу данных SQL Server, слишком медленно ADO.NET?

Question

Моя программа все еще работает для импорта данных из файла журнала в удаленную базу данных SQL Server. Размер файла журнала составляет около 80 МБ, он содержит около 470000 строк и около 25000 строк данных. Моя программа может импортировать только 300 строк в секунду, что очень плохо. (

public static int ImportData(string strPath)
{
    //NameValueCollection collection = ConfigurationManager.AppSettings;

    using (TextReader sr = new StreamReader(strPath))
    {
        sr.ReadLine(); //ignore three first lines of log file
        sr.ReadLine();
        sr.ReadLine();
        string strLine;
        var cn = new SqlConnection(ConnectionString);
        cn.Open();

        while ((strLine = sr.ReadLine()) != null)
        {
            {
                if (strLine.Trim() != "") //if not a blank line, then import into database
                {
                    InsertData(strLine, cn);
                    _count++;
                }
            }
        }
        cn.Close();
        sr.Close();

        return _count;
    }
}

InsertData - это обычный метод вставки, использующий ADO.NET. Используется метод разбора:

public Data(string strLine)
{
    string[] list = strLine.Split(new[] {'\t'});
    try
    {
        Senttime = DateTime.Parse(list[0] + " " + list[1]);
    }
    catch (Exception)
    {
    }

    Clientip = list[2];
    Clienthostname = list[3];

    Partnername = list[4];
    Serverhostname = list[5];
    Serverip = list[6];

    Recipientaddress = list[7];
    Eventid = Convert.ToInt16(list[8]);
    Msgid = list[9];
    Priority = Convert.ToInt16(list[10]);
    Recipientreportstatus = Convert.ToByte(list[11]);
    Totalbytes = Convert.ToInt32(list[12]);
    Numberrecipient = Convert.ToInt16(list[13]);
    DateTime temp;
    if (DateTime.TryParse(list[14], out temp))
    {
        OriginationTime = temp;
    }
    else
    {
        OriginationTime = null;
    }
    Encryption = list[15];
    ServiceVersion = list[16];
    LinkedMsgid = list[17];
    MessageSubject = list[18];
    SenderAddress = list[19];
}

Метод InsertData:

private static void InsertData(string strLine, SqlConnection cn)
{
    var dt = new Data(strLine); //parse the log line into proper fields 
    const string cnnStr =
        "INSERT INTO LOGDATA ([SentTime]," + "[client-ip]," +
        "[Client-hostname]," + "[Partner-Name]," + "[Server-hostname]," +
        "[server-IP]," + "[Recipient-Address]," + "[Event-ID]," + "[MSGID]," +
        "[Priority]," + "[Recipient-Report-Status]," + "[total-bytes]," +
        "[Number-Recipients]," + "[Origination-Time]," + "[Encryption]," +
        "[service-Version]," + "[Linked-MSGID]," + "[Message-Subject]," +
        "[Sender-Address]) " + " VALUES (     " + "@Senttime," + "@Clientip," +
        "@Clienthostname," + "@Partnername," + "@Serverhostname," + "@Serverip," +
        "@Recipientaddress," + "@Eventid," + "@Msgid," + "@Priority," +
        "@Recipientreportstatus," + "@Totalbytes," + "@Numberrecipient," +
        "@OriginationTime," + "@Encryption," + "@ServiceVersion," +
        "@LinkedMsgid," + "@MessageSubject," + "@SenderAddress)";


    var cmd = new SqlCommand(cnnStr, cn) {CommandType = CommandType.Text};

    cmd.Parameters.AddWithValue("@Senttime", dt.Senttime);
    cmd.Parameters.AddWithValue("@Clientip", dt.Clientip);
    cmd.Parameters.AddWithValue("@Clienthostname", dt.Clienthostname);
    cmd.Parameters.AddWithValue("@Partnername", dt.Partnername);
    cmd.Parameters.AddWithValue("@Serverhostname", dt.Serverhostname);
    cmd.Parameters.AddWithValue("@Serverip", dt.Serverip);
    cmd.Parameters.AddWithValue("@Recipientaddress", dt.Recipientaddress);
    cmd.Parameters.AddWithValue("@Eventid", dt.Eventid);
    cmd.Parameters.AddWithValue("@Msgid", dt.Msgid);
    cmd.Parameters.AddWithValue("@Priority", dt.Priority);
    cmd.Parameters.AddWithValue("@Recipientreportstatus", dt.Recipientreportstatus);
    cmd.Parameters.AddWithValue("@Totalbytes", dt.Totalbytes);
    cmd.Parameters.AddWithValue("@Numberrecipient", dt.Numberrecipient);
    if (dt.OriginationTime != null)
        cmd.Parameters.AddWithValue("@OriginationTime", dt.OriginationTime);
    else
        cmd.Parameters.AddWithValue("@OriginationTime", DBNull.Value);
            //if OriginationTime was null, then insert with null value to this column
    cmd.Parameters.AddWithValue("@Encryption", dt.Encryption);
    cmd.Parameters.AddWithValue("@ServiceVersion", dt.ServiceVersion);
    cmd.Parameters.AddWithValue("@LinkedMsgid", dt.LinkedMsgid);
    cmd.Parameters.AddWithValue("@MessageSubject", dt.MessageSubject);
    cmd.Parameters.AddWithValue("@SenderAddress", dt.SenderAddress);
    cmd.ExecuteNonQuery();
}

Как моя программа может работать быстрее? Большое вам спасибо!

Answer 1

Использование SqlBulkCopy .

Редактировать: я создал минимальную реализацию IDataReader и создал тип Batch, чтобы можно было вставлять произвольные данные в памяти, используя SqlBulkCopy. Вот важный бит:

IDataReader dr = batch.GetDataReader();
using (SqlTransaction tx = _connection.BeginTransaction())
{
    try
    {
        using (SqlBulkCopy sqlBulkCopy =
            new SqlBulkCopy(_connection, SqlBulkCopyOptions.Default, tx))
        {
            sqlBulkCopy.DestinationTableName = TableName;
            SetColumnMappings(sqlBulkCopy.ColumnMappings);
            sqlBulkCopy.WriteToServer(dr);
            tx.Commit();
        }
    }
    catch
    {
        tx.Rollback();
        throw;
    }
}

Остальная часть реализации оставлена ​​в качестве упражнения для читателя:)

Подсказка: единственные биты IDataReader, которые вам нужно реализовать, это Read, GetValue и FieldCount.

Answer 2

Хммм, давайте немного разберемся с этим.

В псевдокоде вы сделали ff:

1. Открыть файл
   • Открыть соединение
   • Для каждой строки, в которой есть данные:
   • Разобрать строку
   • Сохранение данных в SQL Server
   • Закрыть соединение
   • Закрыть файл

Теперь основные проблемы, возникающие при этом следующим образом:

• Вы держите соединение SQL открытым, ожидая разбора вашей строки (довольно восприимчивы к тайм-аутам и прочему)
• Вы могли бы сохранять данные построчно, каждая в своей собственной транзакции. Мы не узнаем, пока вы не покажете нам, что делает InsertData метод
• Следовательно, вы держите файл открытым, ожидая, пока SQL завершит вставку

Оптимальный способ сделать это - проанализировать файл целиком, а затем вставить их в большом количестве. Это можно сделать с помощью SqlBulkCopy (согласно предложению Мэтта Хауэллса) или с помощью служб интеграции SQL Server.

Если вы хотите придерживаться ADO.NET, вы можете объединить ваши операторы INSERT и затем передать их в один большой SQLCommand, вместо того, чтобы делать это таким образом, например, устанавливая один объект SQLCommand для каждого оператора вставки.

Answer 3

Вы создаете объект SqlCommand для каждой строки данных. Поэтому самое простое улучшение - создать

private static SqlCommand cmdInsert

и объявите параметры с помощью метода Parameters.Add (). Затем для каждой строки данных установите значения параметров, используя

cmdInsert.Parameters["@paramXXX"].Value = valueXXX;

Второе улучшение производительности может заключаться в том, чтобы пропускать создание объектов данных для каждой строки и назначать значения параметров непосредственно из массива list [].