Unverified Commit fb8755a6 authored by Filipe Oliveira (Redis)'s avatar Filipe Oliveira (Redis) Committed by GitHub
Browse files

changed addReplyHumanLongDouble to addReplyDouble in georadiusGeneric and geoposCommand (#13494)

# Summary 

- Addresses https://github.com/redis/redis/issues/11565
- Measured improvements of 30% and 37% on the simple use-case (GEOSEARCH
and GEOPOS) (check
https://github.com/redis/redis/pull/13494#issuecomment-2313668934), and
of 66% on a dataset with >60M datapoints and pipeline 10 benchmark.
parent d3d94ccf
...@@ -796,8 +796,8 @@ void georadiusGeneric(client *c, int srcKeyIndex, int flags) { ...@@ -796,8 +796,8 @@ void georadiusGeneric(client *c, int srcKeyIndex, int flags) {
if (withcoords) { if (withcoords) {
addReplyArrayLen(c, 2); addReplyArrayLen(c, 2);
addReplyHumanLongDouble(c, gp->longitude); addReplyDouble(c,gp->longitude);
addReplyHumanLongDouble(c, gp->latitude); addReplyDouble(c,gp->latitude);
} }
} }
} else { } else {
...@@ -959,8 +959,8 @@ void geoposCommand(client *c) { ...@@ -959,8 +959,8 @@ void geoposCommand(client *c) {
continue; continue;
} }
addReplyArrayLen(c,2); addReplyArrayLen(c,2);
addReplyHumanLongDouble(c,xy[0]); addReplyDouble(c,xy[0]);
addReplyHumanLongDouble(c,xy[1]); addReplyDouble(c,xy[1]);
} }
} }
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment