fix style

dbms/src/Interpreters/Context.cpp

@@ -292,7 +292,8 @@ struct ContextShared
         trace_collector_thread.join();
     }
 
-    void initializeTraceCollector(TraceLog * trace_log) {
+    void initializeTraceCollector(TraceLog * trace_log)
+    {
         trace_pipe.open();
         trace_collector.reset(new TraceCollector(trace_log, trace_collector_stop.get_future()));
         trace_collector_thread.start(*trace_collector);

dbms/src/Interpreters/ThreadStatusExt.cpp

@@ -131,8 +131,10 @@ void ThreadStatus::finalizePerformanceCounters()
     }
 }
 
-namespace {
-    void queryProfilerTimerHandler(int /* sig */, siginfo_t * /* info */, void * context) {
+namespace
+{
+    void queryProfilerTimerHandler(int /* sig */, siginfo_t * /* info */, void * context)
+    {
         DB::WriteBufferFromFileDescriptor out(trace_pipe.fds_rw[1]);
 
         const std::string & query_id = CurrentThread::getQueryId();
@@ -143,59 +145,51 @@ namespace {
     }
 }
 
-void ThreadStatus::initQueryProfiler() {
-    if (!query_context) {
-        LOG_INFO(log, "Query profiler disabled - no context");
+void ThreadStatus::initQueryProfiler()
+{
+    if (!query_context)
         return;
-    }
 
     struct sigevent sev;
     sev.sigev_notify = SIGEV_THREAD_ID;
     sev.sigev_signo = pause_signal;
     sev._sigev_un._tid = os_thread_id;
     // TODO(laplab): get clock type from settings
-    if (timer_create(CLOCK_REALTIME, &sev, &query_profiler_timer_id)) {
+    if (timer_create(CLOCK_REALTIME, &sev, &query_profiler_timer_id))
         throw Poco::Exception("Failed to create query profiler timer");
-    }
 
     // TODO(laplab): get period from settings
     struct timespec period{.tv_sec = 0, .tv_nsec = 200000000};
     struct itimerspec timer_spec = {.it_interval = period, .it_value = period};
-    if (timer_settime(query_profiler_timer_id, 0, &timer_spec, nullptr)) {
+    if (timer_settime(query_profiler_timer_id, 0, &timer_spec, nullptr))
         throw Poco::Exception("Failed to set query profiler timer");
-    }
 
     struct sigaction sa{};
     sa.sa_sigaction = queryProfilerTimerHandler;
     sa.sa_flags = SA_SIGINFO;
 
-    if (sigemptyset(&sa.sa_mask)) {
+    if (sigemptyset(&sa.sa_mask))
         throw Poco::Exception("Failed to clean signal mask for query profiler");
-    }
 
-    if (sigaddset(&sa.sa_mask, pause_signal)) {
+    if (sigaddset(&sa.sa_mask, pause_signal))
         throw Poco::Exception("Failed to add signal to mask for query profiler");
-    }
 
-    if (sigaction(pause_signal, &sa, previous_handler)) {
+    if (sigaction(pause_signal, &sa, previous_handler))
         throw Poco::Exception("Failed to setup signal handler for query profiler");
-    }
 
     has_query_profiler = true;
 }
 
-void ThreadStatus::finalizeQueryProfiler() {
-    if (!has_query_profiler) {
+void ThreadStatus::finalizeQueryProfiler()
+{
+    if (!has_query_profiler)
         return;
-    }
 
-    if (timer_delete(query_profiler_timer_id)) {
+    if (timer_delete(query_profiler_timer_id))
         throw Poco::Exception("Failed to delete query profiler timer");
-    }
 
-    if (sigaction(pause_signal, previous_handler, nullptr)) {
+    if (sigaction(pause_signal, previous_handler, nullptr))
         throw Poco::Exception("Failed to restore signal handler after query profiler");
-    }
 
     has_query_profiler = false;
 }

dbms/src/Interpreters/TraceCollector.cpp

@@ -7,7 +7,8 @@
 #include <Common/Exception.h>
 
 
-namespace DB {
+namespace DB
+{
     LazyPipe trace_pipe;
 
     TraceCollector::TraceCollector(TraceLog * trace_log, std::future<void>&& stop_future)
@@ -29,19 +30,21 @@ namespace DB {
             try {
                 DB::readPODBinary(context, in);
                 DB::readStringBinary(query_id, in);
-            } catch (Exception) {
+            }
+            catch (Exception)
+            {
                 /// Pipe was closed - looks like server is about to shutdown
                 /// Let us wait for stop_future
                 continue;
             }
 
-            if (trace_log != nullptr) {
+            if (trace_log != nullptr)
+            {
                 std::vector<void *> frames = getBacktraceFrames(context);
                 std::vector<UInt64> trace;
                 trace.reserve(frames.size());
-                for (void * frame : frames) {
+                for (void * frame : frames)
                     trace.push_back(reinterpret_cast<uintptr_t>(frame));
-                }
 
                 TraceLogElement element{std::time(nullptr), query_id, trace};