Hi Sven, i remember having done a simple patch to address this issue in the past, so i perfectly see your point and encourage you to go ahead with it. On one side that patch was not really portable on the newer versions; also it seems to have totally gone lost now - can't find it anymore.
The work is not a big pain as internal structures are, you know, integers. So oversimplifying: having the right SQL schema in place, it's as simple as replacing a %s with a %d in the sprintf()s when building the SQL query. IPv6 addresses might pose a challenge. A separate set of MySQL schemas is a must - but that could be solved with just a bit of good documentation. And the mysql_numeric option is a good idea. Finally, yes, it makes sense working against the CVS so that merging the patch within the official version turns into a smooth bit - and yes, i would like the official version to include the feature. Cheers, Paolo On Fri, Aug 24, 2007 at 05:17:19PM +0200, Sven Anderson wrote: > Hi, > > I'm going to write a patch to have a numeric MySQL scheme, as I think it > is not very optimal to have mac_*, ip_* and ip_proto as strings. > PostgreSQL is better here with special types, but the performance of > MySQL is a lot better in my setup. > > Is it going to be very difficult? Any hints? Shall I write the patch > against SVN to include it in the official version? I think of a > config-key like "mysql_numeric: true" to use a numeric SQL scheme. > > > Cheers, > > Sven > > -- > Sven Anderson > Institute for Informatics - http://www.ifi.informatik.uni-goettingen.de > Georg-August-Universitaet Goettingen > Lotzestr. 16-18, 37083 Goettingen, Germany _______________________________________________ pmacct-discussion mailing list http://www.pmacct.net/#mailinglists
