oracledb_cdc: handle row id based predicates

[josephwoodward]

In Oracle, a ROWID is Oracle's internal physical address for a row - it's not a column in the table; it's a pseudo-column Oracle maintains internally and under some circumstances it's possible that OracleDB will fall back to using it to locate rows (tables without primary keys, LOB based operations).

So this change handles such scenario, publishing the change as an event and logging the SQL via the DEBUG flag (it's possible it could contain sensitive information so should be an opt-in behaviour).

[claude]

Commits

1. Commit oracledb_cdc: Log SQL query causing failure via debug logging — the message subject must start lowercase (imperative mood). Per the commit message policy, in system: message form the message starts lowercase, so this should be oracledb_cdc: log SQL query causing failure via debug logging.

Review

Small, focused change to the Oracle LogMiner SQL_REDO scanner adding support for bare (unquoted) identifiers such as ROWID, plus a debug log on parse failure. The readIdent peek-ahead logic correctly distinguishes alias-prefixed quoted identifiers (a."COL") from bare identifiers (ROWID) by scanning for a " before any terminator, and both new test cases trace correctly through the shared ScanSQLCommand path used by ParseSQLCommand/ExtractValuesFromAST. The debug log format verbs are all valid. Change is well-tested and consistent with existing patterns.

LGTM

[Jeffail]

Ace, thanks @josephwoodward!