Minimal supplemental logging on Oracle connector

16 views
Skip to first unread message

Martin Perez

unread,
Apr 19, 2021, 9:52:31 AMApr 19
to debe...@googlegroups.com
Hello, 

I'm wondering, ant0nk tested minimal supplemental logging and raised PR #230 , then the team added the requirement for table level supplemental logging in #238 for backwards compatibility reasons. 

I think it makes sense to keep backwards compatibility. At the same time, allowing minimal logging could make sense for those cases where the consuming applications aren't interested at the before state. Would it make sense to make it optional with some connector configuration variable?

If it does I'm happy for us to contribute the change and test it. 

Martín

Chris Cranford

unread,
Apr 19, 2021, 11:18:29 AMApr 19
to debe...@googlegroups.com, Martin Perez
Hi Martin -

Overall, supplemental logging is very similar to REPLICA IDENTITY in PostgreSQL.  We don't use a configuration option for PostgreSQL and I don't think we'd need anything like that for Oracle. 

What we should do to support this besides the connector working as expected, we should also log the supplemental logging state for each table like we do in PostgreSQL, see PostgresSchema#printReplicaIdentityInfo as an example.  In addition to that, documentation should reflect the supplemental logging mappings of minimal, full, etc to what state is to be expected in the emitted events.  For documentation, see https://debezium.io/documentation/reference/1.5/connectors/postgresql.html#postgresql-replica-identity as an example.

If you have the bandwidth for this, that would be superb and we'd much welcome it!

Thanks,
Chris
--
You received this message because you are subscribed to the Google Groups "debezium" group.
To unsubscribe from this group and stop receiving emails from it, send an email to debezium+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/debezium/CAE6Yt4kAarschY%3DAx2_19Lk9fKGJMjUNa1EQLyY7Maw9UvKB7w%40mail.gmail.com.

Reply all
Reply to author
Forward
0 new messages