search

Tinkoff / invest-openapi-java-sdk

Apache License 2.0
167 stars 46 forks source link

Fix shutdown #125

Closed SergeevPavel closed 3 years ago

zlumyo commented 3 years ago

@SergeevPavel есть какой-то кейс, когда нужна жёсткая остановка?

SergeevPavel commented 3 years ago

Я ожидаю что если мы вызвали api.close() все не демон треды остановятся и программа сможет завершиться (сейчас на тредпуле зависают таски которые обслуживают вебсокеты). Этот фикс немного неправильный, по хорошему нужно сделать StreamingContextImpl Closeable и закрывать там вебсокеты

zlumyo commented 3 years ago

Подход с Closable звучит интересно. Получается для интерфейса OpenApi это уже будет не столь актуально. Хотелось бы посмотреть на ваше видиние реализации.