You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/09/20 05:36:14 UTC

[GitHub] [incubator-kvrocks] PragmaTwice commented on pull request #897: Replace std::stol with ParseInt

PragmaTwice commented on PR #897:
URL: https://github.com/apache/incubator-kvrocks/pull/897#issuecomment-1251863098

   > > That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?
   > 
   > @PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.
   
   I think it is ok to change it, since it is trivial.


-- 
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: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org