sql_log

[Peter Nixon]

On Sat 02 Sep 2006 01:04, Peter Nixon wrote:
> On Fri 01 Sep 2006 23:53, Nicolas Baradakis wrote:
> > Peter Nixon wrote:
> > > Forgive me if this has been asked before, but shouldn't the sql_log
> > > module use the same SQL queries from as the normal sql module? Why
> > > does it have it's own set of queries in radiusd.conf?
> >
> > rlm_sql_log isn't really a SQL module, as it doesn't talk to a SQL
> > server. In fact, the module just feeds the radsqlrelay script and
> > it doesn't share any code with rlm_sql.
> >
> > The module has it's own section because you're not supposed to
> > configure both rlm_sql and rlm_sql_log at the same time. However,
> > according to your SQL clean up idea, this section could be in a
> > separate file etc/raddb/sql/sql_log.conf instead of radiusd.conf.
> >
> > Alan DeKok wrote:
> > >   The other thing is that the SQL module should be updated to
> > > auto-discover more queries like rlm_sql_log, rather than having them
> > > hard-coded.  This means that any new accounting status type can be
> > > supported by simply adding the queries.
> >
> > The only problem with the rlm_sql_log method is that it doesn't
> > support the accounting_???_query_alt yet.
> >
> > I'm unsure if we still need the "alt" queries: since MySQL 4.1 the
> > new syntax "INSERT ... ON DUPLICATE KEY UPDATE" allows to do something
> > similar with a single statement.
>
> And one the other 7 or 8 SQL variants that we support?

After reading the last 3 email I wrote to the list I figure it must be time to 
get some sleep as I don't appear to be making sense. This was supposed to be:

"And what about the other 7 or 8 SQL variants that we support?"

-- 

Peter Nixon
http://www.peternixon.net/
PGP Key: http://www.peternixon.net/public.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freeradius.org/pipermail/freeradius-devel/attachments/20060902/7f45f99d/attachment.pgp>