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


Во время клиринга вылетает огромное колличество ошибок "Ошибка проверки потока репликации. Код -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);, что и дает ошибку.

Теги:


Спасибо:




1 2 3  > >>
Mikhail Sukhov

Фотография
Дата: 04.10.2011
Ответить


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


Что за стек?
Спасибо:

FiNick

Фотография
Дата: 04.10.2011
Ответить


Mikhail Sukhov GoTo
FiNick GoTo
пока не переполнится стек.


Что за стек?


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

Alexander

Фотография
Дата: 04.10.2011
Ответить


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

Mikhail Sukhov

Фотография
Дата: 04.10.2011
Ответить


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


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

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


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

Alexander

Фотография
Дата: 04.10.2011
Ответить


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


Я думаю данный 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 GoTo
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, потому что со старым вариантом все норм


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

Добавить файлы через драг-н-дроп, , или вставить из буфера обмена.

loading
clippy