acelyc111 opened a new pull request, #1311: URL: https://github.com/apache/incubator-pegasus/pull/1311
https://github.com/apache/incubator-pegasus/issues/1305 This patch aim to use LOG_INFO_F instead of LOG_INFO for all the remain parts: - Use `LOG_INFO_F` instead of `LOG_INFO` - Use `LOG_INFO_PREFIX` instead of `LOG_INFO` for class `proxy_session`, `pegasus_server_impl ` - Use upper case for commands in class `redis_parser` - Remove some commentted `LOG_INFO` code TODO: 1. print dsn_primary_address in log is useless, since it must be itself in whose log, e.g. in `src/common/fs_manager.cpp`, `src/replica/replica_stub.cpp` 2. try to use `LOG_*_PREFIX` for class `policy_context`, `cold_backup_context` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org For additional commands, e-mail: dev-h...@pegasus.apache.org
