[GitHub] [incubator-kvrocks] tisonkun commented on a diff in pull request #897: Replace std::stol with ParseInt

[GitBox <gi...@apache.org>]

tisonkun commented on code in PR #897:
URL: https://github.com/apache/incubator-kvrocks/pull/897#discussion_r977195199


##########
tests/tcl/tests/unit/type/hash.tcl:
##########
@@ -335,8 +335,8 @@ start_server {tags {"hash"}} {
         catch {r hincrby smallhash str 1} smallerr
         catch {r hincrby smallhash str 1} bigerr
         set rv {}
-        lappend rv [string match "ERR*not an integer*" $smallerr]
-        lappend rv [string match "ERR*not an integer*" $bigerr]
+        lappend rv [string match "ERR*non-integer*" $smallerr]
+        lappend rv [string match "ERR*non-integer*" $bigerr]

Review Comment:
   why do you need these changes?



-- 
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