ZABBIX BUGS AND ISSUES
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-8176

SQL error while mass update web-scenarios on Oracle database

    Details

      Description

      Updated: Web-scenario "***" at "***".
      Updated: Web-scenario "*****" at "***".
      SQL error [ORA-01002: fetch out of sequence] in [SELECT nextid FROM ids WHERE nodeid=0 AND table_name='auditlog' AND field_name='auditid' FOR UPDATE]
      oci_fetch_assoc(): ORA-01002: fetch out of sequence [httpconf.php:287 → add_audit() → DB::insert() → DB::reserveIds() → DBfetch() → oci_fetch_assoc() in /srv/www/htdocs/include/db.inc.php:589]

        Activity

        Hide
        Alexander Vladishev added a comment -

        Similar issue: ZBX-8100

        Show
        Alexander Vladishev added a comment - Similar issue: ZBX-8100
        Hide
        Krists Krigers (Inactive) added a comment - - edited

        Fixed and committed in r45143, branch svn://svn.zabbix.com/branches/dev/ZBX-8176.

        (1) Also did search and added DBStart() / DBEnd() where add_audit() is called in foreach - r45144.

        Ivo Kurzemnieks CLOSED.

        Show
        Krists Krigers (Inactive) added a comment - - edited Fixed and committed in r45143, branch svn://svn.zabbix.com/branches/dev/ZBX-8176. (1) Also did search and added DBStart() / DBEnd() where add_audit() is called in foreach - r45144. Ivo Kurzemnieks CLOSED.
        Hide
        Ivo Kurzemnieks added a comment - - edited

        (2) Same problem in screen list configuration.

        Krists Krigers RESOLVED in r45235.

        Ivo Kurzemnieks CLOSED.

        Show
        Ivo Kurzemnieks added a comment - - edited (2) Same problem in screen list configuration. Krists Krigers RESOLVED in r45235. Ivo Kurzemnieks CLOSED.
        Hide
        Ivo Kurzemnieks added a comment - - edited

        (3)

        • Let's use DBstart() instead of DBStart() and DBend() instead of DBEnd().
        • adm.regexps.php: 185 the line is empty. But you can move clearCookies($result) there.
        • maintenance.php: 220 would be good to start tranaction before get_maintenance_by_maintenanceid() so it's more consistent with other DBs and trunk version.
        • usergrps.php: 225 transaction should start inside the if clause at line 227.

        When in doubt, see trunk r42749 under ZBX-7749.

        Krists Krigers RESOLVED in r45233.

        Ivo Kurzemnieks CLOSED.

        Show
        Ivo Kurzemnieks added a comment - - edited (3) Let's use DBstart() instead of DBStart() and DBend() instead of DBEnd(). adm.regexps.php: 185 the line is empty. But you can move clearCookies($result) there. maintenance.php: 220 would be good to start tranaction before get_maintenance_by_maintenanceid() so it's more consistent with other DBs and trunk version. usergrps.php: 225 transaction should start inside the if clause at line 227. When in doubt, see trunk r42749 under ZBX-7749 . Krists Krigers RESOLVED in r45233. Ivo Kurzemnieks CLOSED.
        Hide
        Ivo Kurzemnieks added a comment -

        TESTED.

        Show
        Ivo Kurzemnieks added a comment - TESTED.
        Hide
        Krists Krigers (Inactive) added a comment - - edited

        (4) Need to check if this is an issue for trunk as well.

        Krists Krigers Checked trunk, no issues found. No need to merge to trunk. CLOSED.

        Show
        Krists Krigers (Inactive) added a comment - - edited (4) Need to check if this is an issue for trunk as well. Krists Krigers Checked trunk, no issues found. No need to merge to trunk. CLOSED.
        Hide
        Krists Krigers (Inactive) added a comment -

        Merged and committed to branch 2.2 in r45252.

        Show
        Krists Krigers (Inactive) added a comment - Merged and committed to branch 2.2 in r45252.

          People

          • Assignee:
            Unassigned
            Reporter:
            Alexey Pustovalov
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: