JiriOndrusek commented on PR #9006:
URL: https://github.com/apache/camel-quarkus/pull/9006#issuecomment-5293212675

   Thanks for the thorough review! All points are addressed in c0a0e17:
   - `writeIntent` now holds the sweep bound across uncommitted re-intents 
(dialect-free `CASE`); the dead `committedCount` parameter is gone
   - `tombstone` upserts a pure suppression row for never-ingested documents; 
`commit` without an intent now throws
   - `ensureSchema()` gained the `createTableIfNotExists` escape hatch with a 
`tableExists` probe
   - table renamed to `camel_quarkus_ingestion_tracker`, `doc_id` widened to 
1024, README notes on `doc_id` sensitivity and pre-provisioning
   
   Each fixed path is pinned by a new test (unit + PostgreSQL IT). Ready for 
another look.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to