@robocoder opened this Issue on May 28th 2010 Contributor

In hindsight, the switch to Unsigned BIGINT (#668) won't accommodate an ipv6 address (see #1111).

Using a 32-bit INT would save 4 bytes per row in the log visit table and improve performance on 32-bit systems.

@halfdan commented on May 28th 2010 Member

Agreed, IPv6 would need 2xBIGINT, or alternatively DECIMAL(39,0) (http://www.koopman.me/2008/04/storing-ipv6-ips-in-mysql/)
Shouldn't this problem be adressed together with #1111?

@robocoder commented on May 28th 2010 Contributor

Truncating the unsigned bigint to unsigned int should be minimal effort and we already have unit tests for the conversion to/from dotted decimal form. #1111 requires more effort.

@robocoder commented on May 28th 2010 Contributor

(In [2236]) fixes #1390 - use UNSIGNED INT for ip; since the upper 32 bits is unused, the ALTER statement simply truncates to the lower 32 bits

This Issue was closed on May 28th 2010
Powered by GitHub Issue Mirror