Ошибка в клиринг. StockSharp

Ошибка в клиринг

04.10.2011

FiNick

Во время клиринга вылетает огромное колличество ошибок "Ошибка проверки потока репликации. Код -2147184638", описание "P2ERR_SERV_NO_SERVICE", пока не переполнится стек.
Источник ошибки:

Code

() => // pollAction
{
	if (Streams.Count == 0 && _streamsToRemove.Count == 0)
		_sleepInterval.Sleep();

	Streams.SyncDo(c =>
	{
		foreach (var stream in Streams)
		{
			try
			{
				stream.CheckConnection(_connection);
			}
			catch (COMException e)
			{
				System.Diagnostics.Trace.WriteLine("stream.CheckConnection(_connection) - COMException " + e.ErrorCode.ToString());
				Error.SafeInvoke(new PlazaException("Ошибка проверки потока репликации.", e));
			}
		}
	});

В методе stream.CheckConnection(_connection), вызывается DataStream.Open(connection);, что и дает ошибку.

клиринг ошибка.JPG 40 KB (170)

Теги:

Plaza 2

Спасибо:

1 2 3 > >>

Mikhail Sukhov

Дата: 04.10.2011

FiNick

пока не переполнится стек.

Что за стек?

Спасибо:

FiNick

Дата: 04.10.2011

Mikhail Sukhov

FiNick

пока не переполнится стек.

Что за стек?

Ну как, стек программы, который переполняется если, например, бесконечно рекурсивно вызывать какую-либо функцию. У меня на экране быстро появляются десятки окошек с ошибкой (см прикрепленную ошибку), все подвисает, пока не вылетит StackOverflow Exception.
Попробуйте запустить SimpleGUI в промклиринг.

Спасибо:

Alexander

Дата: 04.10.2011

Форум ртс
Насколько я понял - exception выкидывать не надо. просто делать Sleep если для всех стримов обломилась проверка.
какие ещё идеи?

Спасибо:

Mikhail Sukhov

Дата: 04.10.2011

FiNick

Ну как, стек программы, который переполняется если, например, бесконечно рекурсивно вызывать какую-либо функцию.

Рекурсии не увидел.

FiNick

У меня на экране быстро появляются десятки окошек с ошибкой (см прикрепленную ошибку), все подвисает, пока не вылетит StackOverflow Exception.
Попробуйте запустить SimpleGUI в промклиринг.

Пример на то и пример, что показывает основное. Гидра прекрасно и пром и основной клиринг переживает. Видимо окошки не нужно выбрасывать.

Спасибо:

Alexander

Дата: 04.10.2011

Mikhail Sukhov

Видимо окошки не нужно выбрасывать.

Я думаю данный exception вообще лишний для ProcessDataError - т.к. это стандартная ситуация.
Потому предлагаю чтоб pollAction возвращал bool флаг - success или нет (чтоб не грузили ProcessMessage).
Соответственно предлагаю изменить его на:

Code

				if (Streams.Count == 0 && _streamsToRemove.Count == 0)
					_sleepInterval.Sleep();

                var checkSuccess = true;
				Streams.SyncDo(c =>
				{
					foreach (var stream in Streams)
					{
						try
						{
							stream.CheckConnection(_connection);
						}
						catch (COMException e)
						{
							System.Diagnostics.Trace.WriteLine("stream.CheckConnection(_connection) - COMException " + e.ErrorCode.ToString());
                            checkSuccess = false;
                            return;
						}
					}
				});

                if (!checkSuccess)
                {
                    _sleepInterval.Sleep();
                    return false;
                }

				_streamsToRemove.SyncDo(c =>
				{
					foreach (var stream in _streamsToRemove)
					{
						try
						{
							stream.Close();
						}
						catch (COMException e)
						{
							System.Diagnostics.Trace.WriteLine("stream.Close() - COMException " + e.ErrorCode.ToString());
							Error.SafeInvoke(new PlazaException("Ошибка закрытия потока репликации.", e));
						}
					}

					_streamsToRemove.Clear();
				});

                return true;

Ну и в PlazaConnection соответственно:

Code

                            if (pollAction != null && !pollAction())
                                return;

Мысли \ возражения?

Спасибо:

FiNick

Дата: 04.10.2011

Мне кажется, или такой код должен грузить проц под 100%? Там же нигде Sleep нету.
Надо как-то сделать, чтобы если !pollAction() то поспать:

Code

if (pollAction != null && !pollAction())
{
    PollTimeOut.Sleep();
    return;
}

Спасибо:

Alexander

Дата: 05.10.2011

да, верно. подправил вариант ваше.

Спасибо:

Alexander

Дата: 05.10.2011

Положил. Хотя бы 1 поток должен быть открыт, так правильнее. В этом случае ProcessMessage имеет смысл делать.

Спасибо:

FiNick

Дата: 05.10.2011

Code

var success = true;
if (Streams.Count == 0 && _streamsToRemove.Count == 0)
	success = false;

Streams.SyncDo(c =>
{
	var anySuccess = false;
	foreach (var stream in Streams)
	{
		try
		{
			stream.CheckConnection(_connection);
			anySuccess = true;
		}
		catch (COMException e)
		{
			System.Diagnostics.Trace.WriteLine("Ошибка проверки потока репликации - stream.CheckConnection(_connection) - COMException " + e.ErrorCode.ToString());
		}
	}

	success = anySuccess;
});

anySuccess лишняя переменная имхо, почему бы сразу везде не использовать success?
Сегодня на клиринге протестирую, вроде должно работать.

Кстати, после этих изменений у меня не коннектится. И проблема именно в PlazaStreamManager, потому что со старым вариантом все норм

Спасибо:

Alexander

Дата: 05.10.2011

FiNick

Code

var success = true;
if (Streams.Count == 0 && _streamsToRemove.Count == 0)
	success = false;

Streams.SyncDo(c =>
{
	var anySuccess = false;
	foreach (var stream in Streams)
	{
		try
		{
			stream.CheckConnection(_connection);
			anySuccess = true;
		}
		catch (COMException e)
		{
			System.Diagnostics.Trace.WriteLine("Ошибка проверки потока репликации - stream.CheckConnection(_connection) - COMException " + e.ErrorCode.ToString());
		}
	}

	success = anySuccess;
});

Исправил. По ночам лучше спать :)
Мы должны дёргать ProcessMessage чтоб знать подключились или нет

Спасибо: