А
Спасибо большое. Я понял, как организовать архитектуру нормально с помощью
coroutineScope. Всё работаетSize: a a a
2019 April 13
В
ну ладно ¯\_(ツ)_/¯
2019 April 16
BV
Кто-нибудь пробовал использовать это https://github.com/jasync-sql/jasync-sql или аналоги? Поделитесь опытом
Q
I have used my project
Q
Can I help u
2019 April 17
AO
Кто-нибудь пробовал использовать это https://github.com/jasync-sql/jasync-sql или аналоги? Поделитесь опытом
Скаловый первоисточник работает ок, думаю этот не отличается
BV
Скаловый первоисточник работает ок, думаю этот не отличается
Мне показалось, что это другой проект, на котлине писанный
AO
Ну да, там прям черным по белому написано, что автор ушёл в другую команду и портировал проект на котлин
BV
Ну да, там прям черным по белому написано, что автор ушёл в другую команду и портировал проект на котлин
Именно из за этого я и подумал, что это другой проект 🙈 потому что там так написано
AO
Ну как я понимаю он заменил scala Future на java CompletableFuture и как бы всё :)
VP
Kotlin и CompletableFuture? Ууу...
Нормальные же средства есть, а не это
Нормальные же средства есть, а не это
AO
Ну это как бы драйвер по сути, и его надо прикручивать уже к высокоуровневой либе
VP
Ну а этот драйвер умеет например отменять запрос по future.cancel()? У CompletableFuture обычно с этим проблемы.
AO
AO
эта функция вызывается если вылетает исключаение, а cancel как раз завершает с CanceledException
VP
Вот здесь я не вижу ничего похожего на обработку отмены
https://github.com/jasync-sql/jasync-sql/blob/8e168f442b3175c102bdbeb8e6e706f24962fe8a/postgresql-async/src/main/java/com/github/jasync/sql/db/postgresql/PostgreSQLConnection.kt#L146
https://github.com/jasync-sql/jasync-sql/blob/8e168f442b3175c102bdbeb8e6e706f24962fe8a/postgresql-async/src/main/java/com/github/jasync/sql/db/postgresql/PostgreSQLConnection.kt#L146
VP
Возможно, эта либа не на той стадии развития, чтобы заморачиваться о таких вещах, но по мне это не похоже на production-ready
AO
Видимо есть ограничения, на то что отменить можно только sendPreparedStatementDirect
AO
Я же говорю что это лоу левел, высокоуровневая либа практически 100% должна вызвать sendPreparedStatement
VP
Тут согласен, но по-моему с prepared statement тоже не всё ок.
В sendPreparedStatementDirect() возвращается future из releaseIfNeeded(), а оттуда возвращается результат flatMapTry, внутри которого flattern(), в котором возвращется свежесозданная CompletableFuture.
И я не вижу навешивание обработчика отмены этой future.
https://github.com/jasync-sql/jasync-sql/blob/8e168f442b3175c102bdbeb8e6e706f24962fe8a/db-async-common/src/main/java/com/github/jasync/sql/db/util/FutureUtils.kt#L36
Вангую, что
В sendPreparedStatementDirect() возвращается future из releaseIfNeeded(), а оттуда возвращается результат flatMapTry, внутри которого flattern(), в котором возвращется свежесозданная CompletableFuture.
И я не вижу навешивание обработчика отмены этой future.
https://github.com/jasync-sql/jasync-sql/blob/8e168f442b3175c102bdbeb8e6e706f24962fe8a/db-async-common/src/main/java/com/github/jasync/sql/db/util/FutureUtils.kt#L36
Вангую, что
sendPreparedStatementDirect().cancel() будет проигнорирован.