Skip to content

Commit

Permalink
Mark {load,save}_http as detached
Browse files Browse the repository at this point in the history
The operators use blocking API calls, so on a system with little
parallelism they were able to get the node to a halt.
  • Loading branch information
dominiklohmann committed Dec 19, 2024
1 parent 793cf68 commit c31fe57
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions libtenzir/builtins/connectors/curl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -406,6 +406,14 @@ class load_http_operator final : public crtp_operator<load_http_operator> {
}
}

auto location() const -> operator_location override {
return operator_location::local;
}

auto detached() const -> bool override {
return true;
}

auto optimize(expression const& filter,
event_order order) const -> optimize_result override {
TENZIR_UNUSED(filter, order);
Expand Down Expand Up @@ -448,6 +456,14 @@ class save_http_operator final : public crtp_operator<save_http_operator> {
}
}

auto location() const -> operator_location override {
return operator_location::local;
}

auto detached() const -> bool override {
return true;
}

auto optimize(expression const& filter,
event_order order) const -> optimize_result override {
TENZIR_UNUSED(filter, order);
Expand Down

0 comments on commit c31fe57

Please sign in to comment.