Re: [pmacct-discussion] The accounting of flows requires SQL table v4

2012-07-18 Thread Paolo Lucente
Hi Leonardo, Solution to not get the error is to set sql_optimize_clauses to true so to flag you are going to provide a custom table (which you do as you provide a sql_table_schema) and disable versioning checks. It can be an idea for the future (simple enough to code after all) to assume

Re: [pmacct-discussion] The accounting of flows requires SQL table v4

2012-07-16 Thread Karl O. Pinc
On 07/16/2012 04:28:25 PM, Leonardo Sápiras wrote: Hi everybody, I am trying to use the aggregate peer_src_ip in my pmacctd.conf. But when I start it the pmacct says: ERROR ( default/mysql ): The accounting of flows requires SQL table v4. Exiting. Without this directive, it works.

Re: [pmacct-discussion] The accounting of flows requires SQL table v4

2012-07-16 Thread Leonardo Sápiras
2012/7/16 Karl O. Pinc k...@meme.com On 07/16/2012 04:28:25 PM, Leonardo Sápiras wrote: Hi everybody, I am trying to use the aggregate peer_src_ip in my pmacctd.conf. But when I start it the pmacct says: ERROR ( default/mysql ): The accounting of flows requires SQL table v4.

[pmacct-discussion] The accounting of flows requires SQL table v4?

2010-02-08 Thread Zenon Mousmoulas
Hi, plugins: mysql[sqltest] sql_optimize_clauses[sqltest]: true sql_table[sqltest]: acct ... aggregate: src_as, [...] flows, tag I also tried aggregate[sqltest]: src_as, [...] flows, tag It is also in my schema: create table acct ( id INT(4) UNSIGNED NOT NULL AUTO_INCREMENT,

Re: [pmacct-discussion] The accounting of flows requires SQL table v4?

2010-02-08 Thread Paolo Lucente
Hi Zenon, Good to see you around again. Please add to your config: sql_table_version[sqltest]: 4 Reason being most of the primitives are still connected to the SQL table versioning concept (in essence: pmacct expects you to make explicit which SQL schema you are running). This is in the

Re: [pmacct-discussion] The accounting of flows requires SQL table v4?

2010-02-08 Thread Paolo Lucente
HI Zenon, It turned to be a bug. It's now fixed and change is already committed to the CVS. Please give it a try and let me know. Cheers, Paolo On Mon, Feb 08, 2010 at 02:22:41PM +0200, Zenon Mousmoulas wrote: Hi Paolo, it's nice to be back :) I had already tried adding this directive (and