Skip to content

Commit

Permalink
remove unrealistic tweaks
Browse files Browse the repository at this point in the history
  • Loading branch information
itrofimow committed Sep 11, 2024
1 parent 338122a commit a7e66b0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 20 deletions.
19 changes: 2 additions & 17 deletions frameworks/C++/userver/userver_benchmark/userver_techempower.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -46,20 +46,6 @@ class NoopTracingManager final
userver::server::http::HttpResponse&) const final {}
};

class MinimalMiddlewarePipelineBuilder final
: public userver::server::middlewares::PipelineBuilder {
public:
static constexpr std::string_view kName{
"minimal-middleware-pipeline-builder"};
using userver::server::middlewares::PipelineBuilder::PipelineBuilder;

private:
userver::server::middlewares::MiddlewaresList BuildPipeline(
userver::server::middlewares::MiddlewaresList) const override {
return {"userver-unknown-exceptions-handling-middleware"};
}
};

int Main(int argc, char* argv[]) {
auto component_list =
userver::components::MinimalServerComponentList()
Expand All @@ -78,10 +64,9 @@ int Main(int argc, char* argv[]) {
.Append<cached_queries::WorldCacheComponent>() // cache component
.Append<cached_queries::Handler>()
.Append<fortunes::Handler>()
// tracing and metrics tweaks
// tracing tweaks
.Append<NoopTracingManager>()
.Append<MinimalMiddlewarePipelineBuilder>()
// bare
// bare (not used in the benchmark currently)
.Append<bare::SimpleRouter>()
.Append<bare::SimpleServer>();

Expand Down
3 changes: 0 additions & 3 deletions frameworks/C++/userver/userver_configs/static_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ components_manager:
thread_name: main-worker # OS will show the threads of this task processor with 'main-worker' prefix.
worker_threads: 48
guess-cpu-limit: true
task-processor-queue: work-stealing-task-queue

fs-task-processor: # Make a separate task processor for filesystem bound tasks.
thread_name: fs-worker
Expand All @@ -29,7 +28,6 @@ components_manager:
handler-defaults:
set_tracing_headers: false
server-name: us
middleware-pipeline-builder: minimal-middleware-pipeline-builder
simple-router:
simple-server:
port: 8081
Expand Down Expand Up @@ -63,7 +61,6 @@ components_manager:
noop-tracing-manager:
tracing-manager-locator:
component-name: noop-tracing-manager
minimal-middleware-pipeline-builder:

plaintext-handler:
path: /plaintext
Expand Down

0 comments on commit a7e66b0

Please sign in to comment.