-
Notifications
You must be signed in to change notification settings - Fork 10.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[grpc][Gpr_To_Absl_Logging] Migrating from gpr to absl logging - gpr_log #36595
[grpc][Gpr_To_Absl_Logging] Migrating from gpr to absl logging - gpr_log #36595
Conversation
@@ -59,7 +60,7 @@ | |||
#include "src/core/lib/resource_quota/api.h" | |||
#include "test/core/test_util/port.h" | |||
|
|||
#define LOG_TEST(x) gpr_log(GPR_INFO, "%s", #x) | |||
#define LOG_TEST(x) LOG(INFO) << #x |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: can we remove this macro? I see there's no good point now that we have LOG(INFO).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually a very nice catch . Multiple files are doing this. I will address all files in a single PR.
Thanks for catching this.
😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit comment, otherwise LGTM.
[grpc][Gpr_To_Absl_Logging] Migrating from gpr to absl logging - gpr_log
In this CL we are migrating from gRPCs own gpr logging mechanism to absl logging mechanism. The intention is to deprecate gpr_log in the future.
We have the following mapping
Reviewers need to check :
gpr_log format strings did not use string_view or std::string . absl LOG accepts these. So there will be some elimination of string_view and std::string related conversions. This is expected.