Skip to content

Commit

Permalink
fix: Remove log in RemoteFunctionService.cpp (#11787)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #11787

This can spam the logs and make it hard to debug issues, e.g.

https://fburl.com/logarithm/nte8jcg8

Ideally we'd have a rate limiter like `LOG_EVERY_MS()` but not sure we want to add that as dependency to Velox.

Reviewed By: xiaoxmeng

Differential Revision: D66914913

fbshipit-source-id: 2a3aeea01b8376812e637ceef503cd1ad81877bc
  • Loading branch information
Guilherme Kunigami authored and facebook-github-bot committed Dec 10, 2024
1 parent 4edd506 commit 3315c04
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions velox/functions/remote/server/RemoteFunctionService.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,6 @@ void RemoteFunctionServiceHandler::invokeFunction(
const auto& functionHandle = request->get_remoteFunctionHandle();
const auto& inputs = request->get_inputs();

LOG(INFO) << "Got a request for '" << functionHandle.get_name()
<< "': " << inputs.get_rowCount() << " input rows.";

// Deserialize types and data.
auto inputType = deserializeArgTypes(functionHandle.get_argumentTypes());
auto outputType = deserializeType(functionHandle.get_returnType());
Expand Down

0 comments on commit 3315c04

Please sign in to comment.