This makes the target location consistent with other auto-generated
files like config_formats.h, config_core.h, and config_functions.h and
simplifies the build of clickhouse_common.
Before there was two incorrect checks:
- timer_id != nullptr, but first timer is 0x0 and the comparison does
not work
- previous_handler should also be reseted regardless it was nullptr or
not, plus it also incorrectly uses 'sigaction * previous_handler'
before.
Which leads to permanent query profiler enabled for particular thread,
this is especially visible if you have query profiler disabled and you
are running some query with profiler enabled, i.e.:
select 1 settings query_profiler_real_time_period_ns=1
After, QueryPipelineEx will eat lots of CPU due because of profiler.
