[syslog-ng] [Bug 73] sql driver does not: perform commits, doesnt support session start statements, mangles table names

bugzilla at bugzilla.balabit.com bugzilla at bugzilla.balabit.com
Mon Apr 12 22:21:52 CEST 2010


https://bugzilla.balabit.com/show_bug.cgi?id=73


Balazs Scheidler <bazsi at balabit.hu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #2 from Balazs Scheidler <bazsi at balabit.hu>  2010-04-12 22:21:52 ---
Hi,

Just as stated on the mailing list, I've integrated your work into the 3.2 branch of syslog-ng, but I've some comments, please see inline:

(In reply to comment #0)
> Created an attachment (id=18)
 --> (https://bugzilla.balabit.com/attachment.cgi?id=18) [details]
> fixes
> 
> The main problem: When using the sql driver to output entries straight to a database, commits are not performed, so the data isnt visible outside the session
> used by syslog-ng.
> This was experienced when using the oracle driver of libdbi-drivers.

You might be using an older version of libdbi, in my version, the Oracle driver automatically commits after every query using OCI_COMMIT_ON_SUCCESS parameter:

        status = OCIStmtExecute(Oconn->svc, stmt, Oconn->err, 
                       (ub4) (stmttype == OCI_STMT_SELECT ? 0 : 1), 
                       (ub4) 0, (CONST OCISnapshot *) NULL, (OCISnapshot *) NULL, 
                       stmttype == OCI_STMT_SELECT ? MY_OCI_STMT_SCROLLABLE_READONLY : OCI_COMMIT_ON_SUCCESS);


But anyway, your patches are now integrated, but COMMITs need to be explicitly requested by the flags("explicit-commits") option to the SQL driver.


-- 
Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.


More information about the syslog-ng mailing list