geoip GEOIP_MEMORY_CACHE vs GEOIP_STANDARD

[abgit]

Hi,
On latest unstable branch commit, GeoIP_new was changed from GeoIP_new(GEOIP_MEMORY_CACHE) to GeoIP_new(GEOIP_STANDARD).

Note that, GEOIP_MEMORY_CACHE is much much faster than GEOIP_STANDARD.
On my 2 million hits file, i get this parsing times:

• GEOIP_STANDARD: in 40 seconds
• GEOIP_MEMORY_CACHE: in 12 seconds

almost 4x faster.

best,

[allinurl]

@abgit Thanks for the heads up on this. On 2 million hits, how big is the difference in terms of memory consumption between GEOIP_MEMORY_CACHE and GEOIP_STANDARD?

[abgit]

After a top analysis, i really don't see a major difference.

• GEOIP_STANDARD: in 40 seconds, top-mem-virtual 12000, top-cpu 95%
• GEOIP_MEMORY_CACHE: in 12 seconds, top-mem-virtual 13000, top-cpu 65%

GEOIP_MEMORY_CACHE is much faster and with lower cpu :)

best,

[allinurl]

Added GeoIP database flag, defaults to GEOIP_MEMORY_CACHE. Thanks!