| Mikhail Sukhov 
 
   
 
						
						
					 | Date: 3/22/2012 
 
 
	
			ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься.
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| ra81 
 
   
 
						
						
					 | Date: 3/22/2012 
 
 
	
			Mikhail Sukhov ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься. Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все. А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :)
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| Sergey Masyura 
 
   
 
						
						
					 | Date: 3/22/2012 
						
							|  |  |  |   |  
 
 
	
			ra81 Mikhail Sukhov ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься. Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все. А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :) Скрещивать два коннектора смысла мало в силу различий в архитектуре и авторов - получится нечто еще более сложное и менее рабочее. У Родиона есть хорошия решения, которые можно позаимствовать. Предлагаю оставить два коннектора, а я по мере возможностей приведу коннектор из транка в порядок.
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| ra81 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			Новый коммит http://stocksharp.codepl...changeset/changes/15775 1) Исправил проблему озвученную выше с переносом на другой ПК. теперь проблемы быть не должно. 2) Исправил работу с CancelTime, ExpiryDate. Время истечения заявки задается в ExpiryDate . Время отмены заявки приходит в CancelTime. Было иначе. Следует учитывать что терминал не возвращает реальное время отмены, поэтому при получении ордера, коннектор ставит время отмены сам. Если коннектор был запущен и в терминале есть несколько ордеров отмененных, то у них у всех будет время отмены равно текущему времени. Но стоп/старт экспорта не будет менять время отмены ордера. Оно обновляется только один раз, когда коннектор получает данные о том что ордер отменился. По умолчанию время отмены равно null.
			
			
			
			
		
 | 
			
				|  |  | 
			
				|  | 
		
			| 
 | 
		
			
				| OvcharenkoVI 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			Спасибо, Родион. Будем тестить.
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| Mikhail Sukhov 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.
 Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.
 
 Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.
 
 Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить.
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| ra81 
 
   
 
						
						
					 | Date: 3/23/2012 
						
							|  |  |  |   |  
 
 
	
			Mikhail Sukhov Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.
 Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.
 
 Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.
 
 Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить.
 Овердизайн остался похоже от полностью асинхронной модели. Планировал так сделать. Локи есть только на метода терминала. Дедлок Не возможен при моей архитектуре асинхронной передачи данных и приема ордеров. Дедлоки были, но в другом месте. Я избавился о них. Да и они были временным решением. А без локов возможны зависания терминала. ЧТо я и наблюдал :).  Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :).  Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| Mikhail Sukhov 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			ra81 Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :).
 
 Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки. ra81 Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)
 Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать?
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| ra81 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			Quote:Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки. Это совсем грубо. И неудобно работать. И главное непрозрачно в коде. Quote:Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать? Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :). Сергей думал смерджить, посмотрел и понял, что не мерджится. Вот как-то так.
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  | 
		
			| 
 | 
		
			
				| Mikhail Sukhov 
 
   
 
						
						
					 | Date: 3/23/2012 
 
 
	
			ra81 Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :). Я это понял сразу.[rolleyes]  Почему это было неправильно: 1. Два коннектора, оба далеки от 100% готовности. 2. Ревьюровать есть время только один коннектор. И это скорее всего будет основной. 3. Для юзеров опять же недоступен сейчас альтернативный коннектор. 4. С учетом малого кол-ва рук, скорость разработки медленная для каждой из версий коннектора.
			
			
			
			
		
 | 
			
				|  | 
	
		| Thanks: |   |  | 
			
				|  |