Conversation
| void BaseDaemon::onInterruptSignals(int signal_id) | ||
| { | ||
| is_cancelled = true; | ||
| LOG_INFO(&logger(), "Received termination signal(" << strsignal(signal_id) << ")"); |
There was a problem hiding this comment.
Нет пробела.
Отображается так:
Received termination signal(Interrupt)
|
Потеряна функциональность быстрого убиения по повторному нажатию Ctrl+C (очень важно). |
|
А такая функциональность была? в демонах только печаталось сообщение "Received second termination signal" |
|
а если вызвать два раза рестарт, то тоже девятка? вообщем стоит обсудить |
Это довольно забавно - такая функциональность была в clickhouse-server, но не было в других демонах Метрики. И сейчас мне её не хватает.
Даже не знаю, как лучше. Можно так же - это проще и понятнее, а можно отличить сигналы INT и TERM. |
|
Засчет чего она достигалась. я не могу найти код, который бы это делал |
|
За счёт того, что уже не ждём сигнал, и для него нет обработчика. |
…ngCloud' CLICKHOUSE-1 Support distributed ddl for qingcloud ClickHouse#1 See merge request !1
3 participants
Edited:
In 2016, we tried using git subtree instead of git submodules, but quickly found it difficult to use.
While git submodules is one of the worst examples of developer experience, subtree is even worse.