empiredan commented on issue #1305: URL: https://github.com/apache/incubator-pegasus/issues/1305#issuecomment-1377437413
> > > After all `LOG_*` macros have been replaced with `LOG_*_F`, would `LOG_*_F` be renamed as `LOG_*`? For example, `LOG_INFO_F` is renamed as `LOG_INFO` ? > > > > > > Yes, I'm planning to still use `LOG_*_F` before all refactor work been finished, then remove `_F` postfix at last in one patch. > > In this way, all macros with or without `_F` postfix are both in the same semantics in the progress. I think it is important to keep all macros consistent without intermediate stages. Let's go ahead ! -- 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
